Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp5693279ybg; Tue, 22 Oct 2019 07:05:46 -0700 (PDT) X-Google-Smtp-Source: APXvYqwuu6HN3GbDxoANxwzR2hmX2aD8RRW25A62XOGBefIVYIrKMWESxJxMV3UyqbohhjtigaiR X-Received: by 2002:a17:906:edb7:: with SMTP id sa23mr27916946ejb.263.1571753146277; Tue, 22 Oct 2019 07:05:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1571753146; cv=none; d=google.com; s=arc-20160816; b=cqzuC6aTsHRj6jWbMiCd6ceMOWT/1EqI/ALNVdVaOOw7oH7Ao88swU3MG9dxzTy7yl YM6Igb5nXKbPz6YDJmr4WP/03XYQjkPAm/nscnhRSsICSb7wo5R1HppCIKw7EMVsbhG1 oh9sAbmtCrv3DaqTK2FAha8bvqFTtGY5vC0cJnPP3hmd/mKA7Zh7v/ssSiqZ0ccLdMJc kJEEnuh1/74mosUDyicqgy8OcWnmfdYbsWDbqSnRY7VvBtl9RItTqvYMTrdtNA23lOMV pmRmM7wKYa/FLRF10aRZ1IsDNmGAuS7okFkVnL8caZR+wiHIxsaf/BY7EFYfFncRvtEO nokg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=6OiFUl1fdaeRq1DoqLpiVMazeaoQpnE/nLtSGyzhd/8=; b=Mi/2zC6gt/Kj3wAtbK6eqWTherVQ12Mf6jP+iCnsAywSmNC+tQWoWXgjUCx8W8fwwX mXpd8zxjfV37ji0OA4ftEL34a441s8IYyBq9XDiBtElijgmeOTyibCsyfTpbd6teM/sj 7SA7UEhpkW3rhQzTR78NEaOM6+E8tawMEll22kb6sCzeh4EYUo5vSpuRv+thN1fwpKrd bwUoFxOwJAE/sY7YYa5nQuthWL2z+tMID/FtUyC41X/kij4rdI5kdsH/WEWIJe3lnO/j 9hLI2tYRkcutDYaD094Dq7CzTfY7Gl/W4PU5FOEqbXI8MkbuprzpfWTum1LcMvMK7IaA TaQQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@st.com header.s=STMicroelectronics header.b=lpZh7ZFe; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=st.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id d25si3804171edb.19.2019.10.22.07.05.20; Tue, 22 Oct 2019 07:05:46 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@st.com header.s=STMicroelectronics header.b=lpZh7ZFe; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=st.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388823AbfJVNmD (ORCPT + 99 others); Tue, 22 Oct 2019 09:42:03 -0400 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:43764 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388176AbfJVNmD (ORCPT ); Tue, 22 Oct 2019 09:42:03 -0400 Received: from pps.filterd (m0046668.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id x9MDfKcR013037; Tue, 22 Oct 2019 15:41:50 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=st.com; h=subject : to : cc : references : from : message-id : date : mime-version : in-reply-to : content-type : content-transfer-encoding; s=STMicroelectronics; bh=6OiFUl1fdaeRq1DoqLpiVMazeaoQpnE/nLtSGyzhd/8=; b=lpZh7ZFe5rXJeK1A8+nZpUxIAl5J+lyHSBolGt5yILdDdCixYyq0pd75X4DHt5qw4iN6 Chhy29zxH9wMp5nLYqg5X0Ww9SS4s/n1uZmnzN1jToNgKBMQ+/BqpK+sA7OTfgraKDJ1 EkESAXyPc64j9G8WxNAC4qz1xjqLBKlsot1KbBJ3kcEffBg6as2XW6HB0aOLkymTtF50 U1TjOid7B2eWC8OQO/cVGRICKfvuuL+DuXXydzPOFe2BDYCqh4EI1/O5kKBpyouROIyI Zzb2YWpKOBD8XLYWe1r2fnugylUGykNsBjGxdiT6XNV5O8/OXyuEz2f7/etuXGkSMu+N Ew== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com with ESMTP id 2vqqy17uyd-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 22 Oct 2019 15:41:50 +0200 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 6E57310002A; Tue, 22 Oct 2019 15:41:50 +0200 (CEST) Received: from Webmail-eu.st.com (sfhdag5node2.st.com [10.75.127.14]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id 5C9BB2AE210; Tue, 22 Oct 2019 15:41:50 +0200 (CEST) Received: from [10.48.1.171] (10.75.127.51) by SFHDAG5NODE2.st.com (10.75.127.14) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 22 Oct 2019 15:41:49 +0200 Subject: Re: [PATCH] i2c: i2c-stm32f7: rework slave_id allocation To: Alain Volmat , , CC: , , , , , , , , References: <1571144589-3066-1-git-send-email-alain.volmat@st.com> From: Pierre Yves MORDRET Message-ID: <7092a0d6-2f6a-fa1f-2999-17418b479c5f@st.com> Date: Tue, 22 Oct 2019 15:41:48 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <1571144589-3066-1-git-send-email-alain.volmat@st.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.75.127.51] X-ClientProxiedBy: SFHDAG5NODE2.st.com (10.75.127.14) To SFHDAG5NODE2.st.com (10.75.127.14) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.95,1.0.8 definitions=2019-10-22_03:2019-10-22,2019-10-22 signatures=0 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Looks good Reviewed-by: Pierre-Yves MORDRET Thx Regards On 10/15/19 3:03 PM, Alain Volmat wrote: > The IP can handle two slave addresses. One address can either be > 7 bits or 10 bits while the other can only be 7 bits. > In order to ensure that a 10 bits address can always be allocated > (assuming there is only one 7 bits address already allocated), > pick up the 7-bits only address slot in priority when performing a 7-bits > address allocation. > > Fixes: 60d609f3 ("i2c: i2c-stm32f7: Add slave support") > > Signed-off-by: Alain Volmat > --- > drivers/i2c/busses/i2c-stm32f7.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/i2c/busses/i2c-stm32f7.c b/drivers/i2c/busses/i2c-stm32f7.c > index d36cf08461f7..cfae01b15243 100644 > --- a/drivers/i2c/busses/i2c-stm32f7.c > +++ b/drivers/i2c/busses/i2c-stm32f7.c > @@ -1265,7 +1265,7 @@ static int stm32f7_i2c_get_free_slave_id(struct stm32f7_i2c_dev *i2c_dev, > * slave[0] supports 7-bit and 10-bit slave address > * slave[1] supports 7-bit slave address only > */ > - for (i = 0; i < STM32F7_I2C_MAX_SLAVE; i++) { > + for (i = STM32F7_I2C_MAX_SLAVE - 1; i >= 0; i--) { > if (i == 1 && (slave->flags & I2C_CLIENT_PEC)) > continue; > if (!i2c_dev->slave[i]) { >