From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Nikula Subject: Re: [PATCH v1 1/1] i2c: designware-pci: clarify a comment for Merrifield Date: Mon, 20 Jun 2016 09:47:15 +0300 Message-ID: <555cf8c0-8e25-f839-1e7c-d6f2d083bfd9@linux.intel.com> References: <1466367716-89939-1-git-send-email-andriy.shevchenko@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mga01.intel.com ([192.55.52.88]:19333 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751781AbcFTGsH (ORCPT ); Mon, 20 Jun 2016 02:48:07 -0400 In-Reply-To: <1466367716-89939-1-git-send-email-andriy.shevchenko@linux.intel.com> Sender: linux-i2c-owner@vger.kernel.org List-Id: linux-i2c@vger.kernel.org To: Andy Shevchenko , Wolfram Sang , linux-i2c@vger.kernel.org On 06/19/2016 11:21 PM, Andy Shevchenko wrote: > There are more than 7 busses, but only 7 are user visible. Update comment > accordingly. > > Signed-off-by: Andy Shevchenko > --- > drivers/i2c/busses/i2c-designware-pcidrv.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c b/drivers/i2c/busses/i2c-designware-pcidrv.c > index b66c31a..7c5a340 100644 > --- a/drivers/i2c/busses/i2c-designware-pcidrv.c > +++ b/drivers/i2c/busses/i2c-designware-pcidrv.c > @@ -125,10 +125,10 @@ static int mfld_setup(struct pci_dev *pdev, struct dw_pci_controller *c) > static int mrfld_setup(struct pci_dev *pdev, struct dw_pci_controller *c) > { > /* > - * On Intel Merrifield the i2c busses are enumerated [1..7]. So, we add > - * 1 to shift the default range. Besides that the first PCI slot > - * provides 4 functions, that's why we have to add 0 to the head slot > - * and 4 to the tail one. > + * On Intel Merrifield the user visible i2c busses are enumerated > + * [1..7]. So, we add 1 to shift the default range. Besides that the > + * first PCI slot provides 4 functions, that's why we have to add 0 to > + * the fisrt slot and 4 to the next one. typo -> ---^ Otherwise Acked-by: Jarkko Nikula