Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-06 Thread Stephen Warren
On 03/06/2014 08:43 PM, Chen-Yu Tsai wrote: > On Fri, Mar 7, 2014 at 11:41 AM, Linus Walleij > wrote: >> On Wed, Mar 5, 2014 at 10:59 AM, Stephen Warren >> wrote: >>> On 03/04/2014 07:37 PM, Linus Walleij wrote: On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren wrote: > On

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-06 Thread Chen-Yu Tsai
On Fri, Mar 7, 2014 at 11:41 AM, Linus Walleij wrote: > On Wed, Mar 5, 2014 at 10:59 AM, Stephen Warren wrote: >> On 03/04/2014 07:37 PM, Linus Walleij wrote: >>> On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren >>> wrote: On 03/04/2014 06:43 PM, Linus Walleij wrote: >>> > If I

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-06 Thread Linus Walleij
On Wed, Mar 5, 2014 at 10:59 AM, Stephen Warren wrote: > On 03/04/2014 07:37 PM, Linus Walleij wrote: >> On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren >> wrote: >>> On 03/04/2014 06:43 PM, Linus Walleij wrote: >> If I understand the situation correctly it's like ACPI does not have named

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-06 Thread Linus Walleij
On Wed, Mar 5, 2014 at 10:59 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014 07:37 PM, Linus Walleij wrote: On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014 06:43 PM, Linus Walleij wrote: If I understand the situation correctly it's

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-06 Thread Chen-Yu Tsai
On Fri, Mar 7, 2014 at 11:41 AM, Linus Walleij linus.wall...@linaro.org wrote: On Wed, Mar 5, 2014 at 10:59 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014 07:37 PM, Linus Walleij wrote: On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-06 Thread Stephen Warren
On 03/06/2014 08:43 PM, Chen-Yu Tsai wrote: On Fri, Mar 7, 2014 at 11:41 AM, Linus Walleij linus.wall...@linaro.org wrote: On Wed, Mar 5, 2014 at 10:59 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014 07:37 PM, Linus Walleij wrote: On Wed, Mar 5, 2014 at 10:18 AM, Stephen

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Stephen Warren
On 03/04/2014 07:37 PM, Linus Walleij wrote: > On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren wrote: >> On 03/04/2014 06:43 PM, Linus Walleij wrote: > >>> If I understand the situation correctly it's like ACPI does not have named >>> GPIOs so keeping specifying this in DT GPIO bindings is

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Linus Walleij
On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren wrote: > On 03/04/2014 06:43 PM, Linus Walleij wrote: >> If I understand the situation correctly it's like ACPI does not have named >> GPIOs so keeping specifying this in DT GPIO bindings is counter-productive >> to the work of abstracting the

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Stephen Warren
On 03/04/2014 06:43 PM, Linus Walleij wrote: > On Wed, Feb 26, 2014 at 7:04 AM, Stephen Warren wrote: > >>> - gpio = devm_gpiod_get_index(>dev, rfkill->reset_name, 0); >>> + gpio = devm_gpiod_get_index(>dev, NULL, 0); >> >> I think the correct fix here is to look up the GPIO by name

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Linus Walleij
On Wed, Feb 26, 2014 at 7:04 AM, Stephen Warren wrote: >> - gpio = devm_gpiod_get_index(>dev, rfkill->reset_name, 0); >> + gpio = devm_gpiod_get_index(>dev, NULL, 0); > > I think the correct fix here is to look up the GPIO by name rather than > by index, but simply hard-code the name

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Linus Walleij
On Wed, Feb 26, 2014 at 7:04 AM, Stephen Warren swar...@wwwdotorg.org wrote: - gpio = devm_gpiod_get_index(pdev-dev, rfkill-reset_name, 0); + gpio = devm_gpiod_get_index(pdev-dev, NULL, 0); I think the correct fix here is to look up the GPIO by name rather than by index, but simply

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Stephen Warren
On 03/04/2014 06:43 PM, Linus Walleij wrote: On Wed, Feb 26, 2014 at 7:04 AM, Stephen Warren swar...@wwwdotorg.org wrote: - gpio = devm_gpiod_get_index(pdev-dev, rfkill-reset_name, 0); + gpio = devm_gpiod_get_index(pdev-dev, NULL, 0); I think the correct fix here is to look up the

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Linus Walleij
On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014 06:43 PM, Linus Walleij wrote: If I understand the situation correctly it's like ACPI does not have named GPIOs so keeping specifying this in DT GPIO bindings is counter-productive to the work of

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-03-04 Thread Stephen Warren
On 03/04/2014 07:37 PM, Linus Walleij wrote: On Wed, Mar 5, 2014 at 10:18 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 03/04/2014 06:43 PM, Linus Walleij wrote: If I understand the situation correctly it's like ACPI does not have named GPIOs so keeping specifying this in DT GPIO

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-02-25 Thread Stephen Warren
On 02/25/2014 05:22 AM, Heikki Krogerus wrote: > There is no use for them in this driver. This will fix a > static checker warning.. > > net/rfkill/rfkill-gpio.c:144 rfkill_gpio_probe() > warn: variable dereferenced before check 'rfkill->name' > > This will also make sure that when DT

[PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-02-25 Thread Heikki Krogerus
There is no use for them in this driver. This will fix a static checker warning.. net/rfkill/rfkill-gpio.c:144 rfkill_gpio_probe() warn: variable dereferenced before check 'rfkill->name' This will also make sure that when DT support is added, "gpios" property can be used as no con_id

[PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-02-25 Thread Heikki Krogerus
There is no use for them in this driver. This will fix a static checker warning.. net/rfkill/rfkill-gpio.c:144 rfkill_gpio_probe() warn: variable dereferenced before check 'rfkill-name' This will also make sure that when DT support is added, gpios property can be used as no con_id labels

Re: [PATCHv2 3/5] net: rfkill: gpio: remove gpio names

2014-02-25 Thread Stephen Warren
On 02/25/2014 05:22 AM, Heikki Krogerus wrote: There is no use for them in this driver. This will fix a static checker warning.. net/rfkill/rfkill-gpio.c:144 rfkill_gpio_probe() warn: variable dereferenced before check 'rfkill-name' This will also make sure that when DT support is