From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Hunter Subject: Re: [PATCHv5 16/18] I2C: OMAP: fix missing handling of errata I2C_OMAP3_1P153 Date: Wed, 4 Apr 2012 14:22:07 -0500 Message-ID: <4F7C9F5F.8010206@ti.com> References: <1333468957-5999-1-git-send-email-shubhrajyoti@ti.com> <1333468957-5999-17-git-send-email-shubhrajyoti@ti.com> <4F7C7780.7040807@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from comal.ext.ti.com ([198.47.26.152]:35097 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932645Ab2DDTWg (ORCPT ); Wed, 4 Apr 2012 15:22:36 -0400 In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Shubhrajyoti Datta Cc: Shubhrajyoti D , linux-omap@vger.kernel.org, tony@atomide.com, Tasslehoff Kjappfot , w.sang@pengutronix.de, linux-i2c@vger.kernel.org, ben-linux@fluff.org, linux-arm-kernel@lists.infradead.org Hi Shubhro, On 04/04/2012 12:45 PM, Shubhrajyoti Datta wrote: > Hi Jon, > > >>> + if (dev->flags& OMAP_I2C_FLAG_APPLY_ERRATA_I207) >>> >>> + dev->errata |= I2C_OMAP_ERRATA_I207; >>> + >>> if (dev->rev<= OMAP_I2C_REV_ON_3430_3530) >>> dev->errata |= I2C_OMAP3_1P153; >> >> >> The errata ID is not correct. I believe that 1P153 is referring to the >> section number in the errata doc and not the errata ID. The errata ID should >> be i451 in the latest OMAP34xx and OMAP36xx docs. >> >> Please can you update this? The section number changed in the latest errata >> docs and it took me a minute to find this. > > Do you mind if that is a separate patch? Not at all and in fact it probably should be. Thanks Jon