staging: iio: ak8975: make gpio platdata mandatory
Naveen Krishna Ch
naveenkrishna.ch at gmail.com
Sat Mar 5 20:40:33 EST 2011
Hi Russell King,
On 6 March 2011 09:58, Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
> On Sun, Mar 06, 2011 at 09:31:41AM +0900, Naveen Krishna Ch wrote:
>> Hi Kyungmin Park,
>>
>> On 4 March 2011 14:51, Kyungmin Park <kmpark at infradead.org> wrote:
>> > Hi,
>> >
>> > On Fri, Mar 4, 2011 at 2:34 PM, Naveen Krishna Ch
>> > <naveenkrishna.ch at gmail.com> wrote:
>> >> Issue:
>> >> For some architectures CONFIG_GENERIC_GPIO is defined,
>> >> leaving irq_to_gpio undefined. Causing build break.
>> >
>> > It's better to implement the irq_to_gpio. Since gpio_to_irq is already
>> > implemented. it's just vice versa.
>> Implementing such macros in machine header files leads to inclusion of
>> mach/gpio.h or irqs.h
>> Russel King, seems against to the same...
>
> Four reasons:
>
> 1. including mach/gpio.h means that the code is non-portable to other
> architectures.
> 2. linux/gpio.h defines helpers for the !GENERIC_GPIO case.
> 3. it's good practice to always use linux/ includes rather than asm/ or
> mach/ where-ever possible.
> 4. over time, standard helpers tend to get split out into the common
> architecture-independent header files. Who would like to volunteer
> to fix up all the mach/gpio.h->linux/gpio.h when that happens.
I understand, the complication created by using machine specific code.
>
> However:
>> >> For some architectures CONFIG_GENERIC_GPIO is defined,
>> >> leaving irq_to_gpio undefined. Causing build break.
>
> I read this as saying: "There are architectures which set
> CONFIG_GENERIC_GPIO but do not provide a definition for irq_to_gpio()".
> If that's true, then that's a problem for those architectures to solve.
I will start looking into fixing the problem, instead of this issue.
>
> However, first I'd like you to clear up some confusion: what do you mean
> by architecture? ARM/Sparc/x86? Or S3C2410 vs OMAP4430 vs Orion vs
> Tegra? The former are architectures. The latter are SoCs or families
> of SoCs/machines/platforms.
I mean, SoCs or families.
>
> Last point: do _not_ cc: mailing list managers for normal discussion
> email - I'm talking about the majordomo at vger.kernel.org which was in the
> CC in this thread.
Sorry my bad, Will correct it here on.
> Mailing list managers take specially formatted email
> messages in order to take automated action. They do not recognise plain
> language, and you'll probably get a verbose error reply.
Yes, I Did..
>
Thanks, for your prompt reply and the explanation.
--
Shine bright,
(: Nav :)
More information about the linux-arm-kernel
mailing list