Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-23 Thread Linus Walleij
On Thu, Jun 23, 2016 at 10:18 AM, Neil Armstrong wrote: > Would you agree to : > - Push the minimal code to make OF work again, at least for 4.7 ? > - Engage complete refactoring to transform it in a real gpio+pinctrl driver ? Sounds like a plan, as long as the DT maintainers agree. Yours, Li

Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-23 Thread Neil Armstrong
On 06/23/2016 10:08 AM, Linus Walleij wrote: > On Mon, Jun 20, 2016 at 5:57 PM, Rob Herring wrote: >> On Fri, Jun 17, 2016 at 11:51:03AM +0200, Neil Armstrong wrote: > >>> +Optional Properties: >>> +- oscio-is-gpo: Boolean, Indicated the oscio pin can be used as additional >>> + outpu

Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-23 Thread Linus Walleij
On Mon, Jun 20, 2016 at 5:57 PM, Rob Herring wrote: > On Fri, Jun 17, 2016 at 11:51:03AM +0200, Neil Armstrong wrote: >> +Optional Properties: >> +- oscio-is-gpo: Boolean, Indicated the oscio pin can be used as additional >> + output gpo port. >> + > >> +- pull-up-ports: Array of port

Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-20 Thread Rob Herring
On Fri, Jun 17, 2016 at 11:51:03AM +0200, Neil Armstrong wrote: > In case of OF probing, the driver fails to initialize : > - gpio_chip.base must be -1 > - irq_summary must be either -1 or valid > - There is no way to use the other configurations > > Add OF parsing function to complete the HW conf

Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-20 Thread Neil Armstrong
On 06/18/2016 11:02 AM, Linus Walleij wrote: > On Fri, Jun 17, 2016 at 11:51 AM, Neil Armstrong > wrote: > >> In case of OF probing, the driver fails to initialize : >> - gpio_chip.base must be -1 >> - irq_summary must be either -1 or valid >> - There is no way to use the other configurations >>

Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-18 Thread Linus Walleij
On Fri, Jun 17, 2016 at 11:51 AM, Neil Armstrong wrote: > In case of OF probing, the driver fails to initialize : > - gpio_chip.base must be -1 > - irq_summary must be either -1 or valid > - There is no way to use the other configurations > > Add OF parsing function to complete the HW configurati

Re: [PATCH] gpio: sx150x: Update OF configuration

2016-06-17 Thread kbuild test robot
Hi, [auto build test ERROR on gpio/for-next] [also build test ERROR on v4.7-rc3 next-20160617] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Neil-Armstrong/gpio-sx150x-Update-OF-configuration/2

[PATCH] gpio: sx150x: Update OF configuration

2016-06-17 Thread Neil Armstrong
In case of OF probing, the driver fails to initialize : - gpio_chip.base must be -1 - irq_summary must be either -1 or valid - There is no way to use the other configurations Add OF parsing function to complete the HW configuration, make OF configuration dynamic instead of static with #defines and