[PATCH 04/11] gpio: tegra: Iterate over the correct number of banks

Stephen Warren swarren at wwwdotorg.org
Wed Apr 4 14:41:54 EDT 2012


On 04/04/2012 11:58 AM, Olof Johansson wrote:
> On Fri, Mar 30, 2012 at 04:59:57PM -0600, Stephen Warren wrote:
>> From: Stephen Warren <swarren at nvidia.com>
>>
>> When Tegra30 support was added to the Tegra GPIO driver, a few places
>> which iterated over all banks were not converted to use the variable
>> tegra_gpio_bank_count rather than hard-coding the bank count. Fix this.
>>
>> Signed-off-by: Stephen Warren <swarren at nvidia.com>
> 
> Acked-by: Olof Johansson <olof at lixom.net>
> 
> Should this be fixed in 3.4 as well? Given the need for some of the other
> patches in this series I suppose gpio isn't really working on tegra30 there
> though, so it's not technically a regression...

It's probably not a bad idea, even if we don't expect many users; the
code may as well be correct. I assume since 3.4 is still in the rc
stage, I would send an explicit pull request to arm-soc for that rather
than just Cc: stable.

I wonder if "gpio: tegra: fix register address calculations for Tegra30"
shouldn't be applied (after addressing your comments) for 3.4 too? It
might be a little large of a change?



More information about the linux-arm-kernel mailing list