Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-15 Thread Y Vo
On Mon, Sep 14, 2015 at 10:23 PM, Marc Zyngier wrote: > On 14/09/15 16:06, Y Vo wrote: >> On Mon, Sep 14, 2015 at 9:47 PM, Arnd Bergmann wrote: >>> On Monday 14 September 2015 16:39:43 Y Vo wrote: On Mon, Sep 14, 2015 at 4:11 PM, Arnd Bergmann wrote: > On Saturday 12 September 2015 12:5

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-14 Thread Marc Zyngier
On 14/09/15 16:06, Y Vo wrote: > On Mon, Sep 14, 2015 at 9:47 PM, Arnd Bergmann wrote: >> On Monday 14 September 2015 16:39:43 Y Vo wrote: >>> On Mon, Sep 14, 2015 at 4:11 PM, Arnd Bergmann wrote: On Saturday 12 September 2015 12:55:55 Y Vo wrote: > On Fri, Sep 11, 2015 at 11:45 PM, Arnd

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-14 Thread Arnd Bergmann
On Monday 14 September 2015 22:06:15 Y Vo wrote: > > > > Did 1b7047edfcfb25 ("genirq: Allow the irqchip state of an IRQ to be > > save/restored") not address the problem for you? You were on > > Cc to that patch and should have spoken up when the code that was > > merged was not sufficient. > > Ye

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-14 Thread Y Vo
On Mon, Sep 14, 2015 at 9:47 PM, Arnd Bergmann wrote: > On Monday 14 September 2015 16:39:43 Y Vo wrote: >> On Mon, Sep 14, 2015 at 4:11 PM, Arnd Bergmann wrote: >> > On Saturday 12 September 2015 12:55:55 Y Vo wrote: >> >> On Fri, Sep 11, 2015 at 11:45 PM, Arnd Bergmann wrote: >> >> > On Friday

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-14 Thread Arnd Bergmann
On Monday 14 September 2015 16:39:43 Y Vo wrote: > On Mon, Sep 14, 2015 at 4:11 PM, Arnd Bergmann wrote: > > On Saturday 12 September 2015 12:55:55 Y Vo wrote: > >> On Fri, Sep 11, 2015 at 11:45 PM, Arnd Bergmann wrote: > >> > On Friday 11 September 2015 22:06:58 Y Vo wrote: > >> > >> Example for

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-14 Thread Y Vo
On Mon, Sep 14, 2015 at 4:11 PM, Arnd Bergmann wrote: > On Saturday 12 September 2015 12:55:55 Y Vo wrote: >> On Fri, Sep 11, 2015 at 11:45 PM, Arnd Bergmann wrote: >> > On Friday 11 September 2015 22:06:58 Y Vo wrote: >> >> On Fri, Sep 11, 2015 at 9:47 PM, Arnd Bergmann wrote: >> >> > On Friday

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-14 Thread Arnd Bergmann
On Saturday 12 September 2015 12:55:55 Y Vo wrote: > On Fri, Sep 11, 2015 at 11:45 PM, Arnd Bergmann wrote: > > On Friday 11 September 2015 22:06:58 Y Vo wrote: > >> On Fri, Sep 11, 2015 at 9:47 PM, Arnd Bergmann wrote: > >> > On Friday 11 September 2015 21:23:31 Y Vo wrote: > >> >> Is that possi

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Y Vo
On Fri, Sep 11, 2015 at 11:45 PM, Arnd Bergmann wrote: > On Friday 11 September 2015 22:06:58 Y Vo wrote: >> On Fri, Sep 11, 2015 at 9:47 PM, Arnd Bergmann wrote: >> > On Friday 11 September 2015 21:23:31 Y Vo wrote: >> >> Is that possible if we change to this approach: >> >> - Keep all GPIO as G

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Arnd Bergmann
On Friday 11 September 2015 22:06:58 Y Vo wrote: > On Fri, Sep 11, 2015 at 9:47 PM, Arnd Bergmann wrote: > > On Friday 11 September 2015 21:23:31 Y Vo wrote: > >> Is that possible if we change to this approach: > >> - Keep all GPIO as GPIO mode by default. > >> - If anyone want to use them as inte

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Y Vo
On Fri, Sep 11, 2015 at 9:47 PM, Arnd Bergmann wrote: > On Friday 11 September 2015 21:23:31 Y Vo wrote: >> >> /* Board-specific: exclude GPIO_DS8 to leave it as GPIO */ >> >> &sbgpio { >> >>interrupts =<0x0 0x29 0x1>, /* GPIO_DS9 */ >> >><0x0 0x2a 0x1>,

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Arnd Bergmann
On Friday 11 September 2015 21:23:31 Y Vo wrote: > >> /* Board-specific: exclude GPIO_DS8 to leave it as GPIO */ > >> &sbgpio { > >>interrupts =<0x0 0x29 0x1>, /* GPIO_DS9 */ > >><0x0 0x2a 0x1>, /* GPIO_DS10 */ > >><0x0 0x2

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Y Vo
On Fri, Sep 11, 2015 at 7:46 PM, Arnd Bergmann wrote: > On Friday 11 September 2015 18:24:47 Y Vo wrote: >> On Fri, Sep 11, 2015 at 4:35 PM, Arnd Bergmann wrote: >> > On Friday 11 September 2015 16:22:12 Y Vo wrote: >> >> There are 20 GPIO pins from 0..21. There is no GPIO_DS14 or GPIO_DS15, >>

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Arnd Bergmann
On Friday 11 September 2015 18:24:47 Y Vo wrote: > On Fri, Sep 11, 2015 at 4:35 PM, Arnd Bergmann wrote: > > On Friday 11 September 2015 16:22:12 Y Vo wrote: > >> There are 20 GPIO pins from 0..21. There is no GPIO_DS14 or GPIO_DS15, > >> -only GPIO_DS8..GPIO_DS13 support interrupts. The IRQ mapp

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Y Vo
On Fri, Sep 11, 2015 at 4:35 PM, Arnd Bergmann wrote: > On Friday 11 September 2015 16:22:12 Y Vo wrote: >> There are 20 GPIO pins from 0..21. There is no GPIO_DS14 or GPIO_DS15, >> -only GPIO_DS8..GPIO_DS13 support interrupts. The IRQ mapping >> +only GPIO_DS8..GPIO_DS13 support external interru

Re: [PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Arnd Bergmann
On Friday 11 September 2015 16:22:12 Y Vo wrote: > There are 20 GPIO pins from 0..21. There is no GPIO_DS14 or GPIO_DS15, > -only GPIO_DS8..GPIO_DS13 support interrupts. The IRQ mapping > +only GPIO_DS8..GPIO_DS13 support external interrupts. The IRQ mapping > is currently 1-to-1 on interrupts 0x

[PATCH 2/2] Documentation: gpio: Update description for X-Gene standby GPIO controller DTS binding

2015-09-11 Thread Y Vo
Signed-off-by: Y Vo --- .../devicetree/bindings/gpio/gpio-xgene-sb.txt | 26 ++- 1 files changed, 19 insertions(+), 7 deletions(-) diff --git a/Documentation/devicetree/bindings/gpio/gpio-xgene-sb.txt b/Documentation/devicetree/bindings/gpio/gpio-xgene-sb.txt index dae1300