Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Jon Hunter
On 11/05/16 18:28, Mark Rutland wrote: > On Wed, May 11, 2016 at 05:53:43PM +0100, Jon Hunter wrote: >> Hi Rob, Mark, >> >> On 11/05/16 17:30, Rob Herring wrote: >>> A platform driver for just gic-400 is wrong IMO until we have platform >>> drivers for all interrupt controllers. >> >> Yes, that is

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Mark Rutland
On Wed, May 11, 2016 at 05:53:43PM +0100, Jon Hunter wrote: > Hi Rob, Mark, > > On 11/05/16 17:30, Rob Herring wrote: > > A platform driver for just gic-400 is wrong IMO until we have platform > > drivers for all interrupt controllers. > > Yes, that is fine with me, but can we decide on whether t

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Jon Hunter
Hi Rob, Mark, On 11/05/16 17:30, Rob Herring wrote: > A platform driver for just gic-400 is wrong IMO until we have platform > drivers for all interrupt controllers. Yes, that is fine with me, but can we decide on whether the platform driver should match "tegra210-agic" or the early driver should

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Rob Herring
On Wed, May 11, 2016 at 11:16 AM, Jon Hunter wrote: > > On 11/05/16 17:08, Jon Hunter wrote: >> Hi Rob, >> >> On 11/05/16 16:51, Rob Herring wrote: >>> On Sat, May 7, 2016 at 9:10 AM, Geert Uytterhoeven >>> wrote: Hi Jon, On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: >>

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Jon Hunter
On 11/05/16 17:08, Jon Hunter wrote: > Hi Rob, > > On 11/05/16 16:51, Rob Herring wrote: >> On Sat, May 7, 2016 at 9:10 AM, Geert Uytterhoeven >> wrote: >>> Hi Jon, >>> >>> On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: > The "nvidia,tegra210-agic" string can be taken as describing any

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Jon Hunter
On 11/05/16 17:08, Jon Hunter wrote: > Hi Rob, > > On 11/05/16 16:51, Rob Herring wrote: >> On Sat, May 7, 2016 at 9:10 AM, Geert Uytterhoeven >> wrote: >>> Hi Jon, >>> >>> On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: > The "nvidia,tegra210-agic" string can be taken as describing any

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Jon Hunter
Hi Rob, On 11/05/16 16:51, Rob Herring wrote: > On Sat, May 7, 2016 at 9:10 AM, Geert Uytterhoeven > wrote: >> Hi Jon, >> >> On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: The "nvidia,tegra210-agic" string can be taken as describing any Tegra-210 specific integration quirks, thoug

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-11 Thread Rob Herring
On Sat, May 7, 2016 at 9:10 AM, Geert Uytterhoeven wrote: > Hi Jon, > > On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: >>> The "nvidia,tegra210-agic" string can be taken as describing any >>> Tegra-210 specific integration quirks, though I agree that's also not >>> fantastic for extending PM

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-09 Thread Marc Zyngier
On 08/05/16 13:25, Jon Hunter wrote: > Hi Geert, > > On 07/05/16 15:10, Geert Uytterhoeven wrote: >> Hi Jon, >> >> On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: The "nvidia,tegra210-agic" string can be taken as describing any Tegra-210 specific integration quirks, though I agree th

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-08 Thread Jon Hunter
Hi Geert, On 07/05/16 15:10, Geert Uytterhoeven wrote: > Hi Jon, > > On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: >>> The "nvidia,tegra210-agic" string can be taken as describing any >>> Tegra-210 specific integration quirks, though I agree that's also not >>> fantastic for extending PM su

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-07 Thread Geert Uytterhoeven
Hi Jon, On Fri, May 6, 2016 at 10:32 AM, Jon Hunter wrote: >> The "nvidia,tegra210-agic" string can be taken as describing any >> Tegra-210 specific integration quirks, though I agree that's also not >> fantastic for extending PM support beyond Tegra 210 and variants >> thereof. >> >> So maybe th

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-05-06 Thread Jon Hunter
Hi Mark, On 28/04/16 10:55, Mark Rutland wrote: [...] > The "nvidia,tegra210-agic" string can be taken as describing any > Tegra-210 specific integration quirks, though I agree that's also not > fantastic for extending PM support beyond Tegra 210 and variants > thereof. > > So maybe the best ap

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-28 Thread Mark Rutland
On Thu, Apr 28, 2016 at 09:11:03AM +0100, Jon Hunter wrote: > > On 27/04/16 18:38, Mark Rutland wrote: > > On Wed, Apr 27, 2016 at 04:34:53PM +0100, Jon Hunter wrote: > >> > >> On 22/04/16 12:22, Mark Rutland wrote: > >> > >> [snip] > >> > >> I am not sure if it will be popular to add Tegra sp

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-28 Thread Geert Uytterhoeven
Hi Jon, On Thu, Apr 28, 2016 at 10:11 AM, Jon Hunter wrote: > On 27/04/16 18:38, Mark Rutland wrote: >> On Wed, Apr 27, 2016 at 04:34:53PM +0100, Jon Hunter wrote: >>> On 22/04/16 12:22, Mark Rutland wrote: >>> [snip] >>> I am not sure if it will be popular to add Tegra specific clock names >

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-28 Thread Jon Hunter
On 27/04/16 18:38, Mark Rutland wrote: > On Wed, Apr 27, 2016 at 04:34:53PM +0100, Jon Hunter wrote: >> >> On 22/04/16 12:22, Mark Rutland wrote: >> >> [snip] >> >> I am not sure if it will be popular to add Tegra specific clock names >> to the GIC DT docs. However, in that case, then poss

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-27 Thread Geert Uytterhoeven
On Wed, Apr 27, 2016 at 7:38 PM, Mark Rutland wrote: > On Wed, Apr 27, 2016 at 04:34:53PM +0100, Jon Hunter wrote: >> On 22/04/16 12:22, Mark Rutland wrote: >> [snip] >> >> I am not sure if it will be popular to add Tegra specific clock names >> to the GIC DT docs. However, in that case,

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-27 Thread Mark Rutland
On Wed, Apr 27, 2016 at 04:34:53PM +0100, Jon Hunter wrote: > > On 22/04/16 12:22, Mark Rutland wrote: > > [snip] > > I am not sure if it will be popular to add Tegra specific clock names > to the GIC DT docs. However, in that case, then possibly the only > alternative is to move

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-27 Thread Jon Hunter
On 22/04/16 12:22, Mark Rutland wrote: [snip] I am not sure if it will be popular to add Tegra specific clock names to the GIC DT docs. However, in that case, then possibly the only alternative is to move the Tegra AGIC driver into its own file and expose the GIC APIs for it

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-22 Thread Jon Hunter
On 22/04/16 12:22, Mark Rutland wrote: > On Fri, Apr 22, 2016 at 12:12:57PM +0100, Jon Hunter wrote: >> >> On 22/04/16 11:00, Mark Rutland wrote: >>> On Wed, Apr 20, 2016 at 12:03:56PM +0100, Jon Hunter wrote: The Tegra AGIC interrupt controller is compatible with the ARM GIC-400 interru

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-22 Thread Mark Rutland
On Fri, Apr 22, 2016 at 12:12:57PM +0100, Jon Hunter wrote: > > On 22/04/16 11:00, Mark Rutland wrote: > > On Wed, Apr 20, 2016 at 12:03:56PM +0100, Jon Hunter wrote: > >> The Tegra AGIC interrupt controller is compatible with the ARM GIC-400 > >> interrupt controller. > > > > The cover letter sa

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-22 Thread Jon Hunter
On 22/04/16 11:00, Mark Rutland wrote: > On Wed, Apr 20, 2016 at 12:03:56PM +0100, Jon Hunter wrote: >> The Tegra AGIC interrupt controller is compatible with the ARM GIC-400 >> interrupt controller. > > The cover letter says it _is_ a GIC-400, just used in a slightly unusual > manner (i.e. not d

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-22 Thread Mark Rutland
On Wed, Apr 20, 2016 at 12:03:56PM +0100, Jon Hunter wrote: > The Tegra AGIC interrupt controller is compatible with the ARM GIC-400 > interrupt controller. The cover letter says it _is_ a GIC-400, just used in a slightly unusual manner (i.e. not directly connected to CPUs). > The Tegra AGIC requ

Re: [PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-22 Thread Marc Zyngier
On 20/04/16 12:03, Jon Hunter wrote: > The Tegra AGIC interrupt controller is compatible with the ARM GIC-400 > interrupt controller. The Tegra AGIC requires two clocks, namely the > "ape" (functional) and "apb2ape" (interface) clocks, to operate. Add > the compatible string and clock information f

[PATCH V2 13/14] dt-bindings: arm-gic: Add documentation for Tegra210 AGIC

2016-04-20 Thread Jon Hunter
The Tegra AGIC interrupt controller is compatible with the ARM GIC-400 interrupt controller. The Tegra AGIC requires two clocks, namely the "ape" (functional) and "apb2ape" (interface) clocks, to operate. Add the compatible string and clock information for the AGIC to the GIC device-tree binding do