On Fri, Apr 03, 2009 at 08:18:47PM +0400, Sergei Shtylyov wrote:
> Mark A. Greer wrote:

>> @@ -268,17 +259,14 @@ gpio_irq_handler(unsigned irq, struct irq_desc *desc)
>>  static int __init davinci_gpio_irq_setup(void)
>>  {
>>      unsigned        gpio, irq, bank;
>> -    unsigned        bank_irq;
>>      struct clk      *clk;
>>      u32             binten = 0;
>> +    unsigned        ngpio, bank_irq;
>> +
>> +    ngpio = davinci_soc_info->gpio_num;
>
>    Not seeing where 'ngpio' is used in this function...

That's because you're only viewing the patch and not the code it
applies to.  'ngpio' was a global used later on in that routine.
I replaced the global with a local with the same name and gave it
the value, davinci_soc_info->gpio_num.

Mark
--

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to