* [PATCH] ARM: socfpga: dts: add fpga manager @ 2015-10-13 18:28 atull 2015-10-13 18:57 ` Steffen Trumtrar 2015-10-13 18:58 ` Moritz Fischer 0 siblings, 2 replies; 7+ messages in thread From: atull @ 2015-10-13 18:28 UTC (permalink / raw) To: dinguyen Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, devicetree, linux-arm-kernel, linux-kernel, delicious.quinoa, Alan Tull From: Alan Tull <atull@opensource.altera.com> Add FPGA manager to device tree for SoCFPGA. Signed-off-by: Alan Tull <atull@opensource.altera.com> --- arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi index 314e589..e37ba11 100644 --- a/arch/arm/boot/dts/socfpga.dtsi +++ b/arch/arm/boot/dts/socfpga.dtsi @@ -834,5 +834,12 @@ compatible = "altr,sys-mgr", "syscon"; reg = <0xffd08000 0x4000>; }; + + hps_0_fpgamgr: fpgamgr@0xff706000 { + compatible = "altr,socfpga-fpga-mgr"; + reg = <0xFF706000 0x1000 + 0xFFB90000 0x1000>; + interrupts = <0 175 4>; + }; }; }; -- 1.9.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: socfpga: dts: add fpga manager 2015-10-13 18:28 [PATCH] ARM: socfpga: dts: add fpga manager atull @ 2015-10-13 18:57 ` Steffen Trumtrar 2015-10-13 19:18 ` Dinh Nguyen 2015-10-13 18:58 ` Moritz Fischer 1 sibling, 1 reply; 7+ messages in thread From: Steffen Trumtrar @ 2015-10-13 18:57 UTC (permalink / raw) To: atull Cc: dinguyen, Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux-kernel, Rob Herring, Kumar Gala, delicious.quinoa, linux-arm-kernel Hi Alan! On Tue, Oct 13, 2015 at 01:28:20PM -0500, atull@opensource.altera.com wrote: > From: Alan Tull <atull@opensource.altera.com> > > Add FPGA manager to device tree for SoCFPGA. > > Signed-off-by: Alan Tull <atull@opensource.altera.com> > --- > arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi > index 314e589..e37ba11 100644 > --- a/arch/arm/boot/dts/socfpga.dtsi > +++ b/arch/arm/boot/dts/socfpga.dtsi > @@ -834,5 +834,12 @@ > compatible = "altr,sys-mgr", "syscon"; > reg = <0xffd08000 0x4000>; > }; > + > + hps_0_fpgamgr: fpgamgr@0xff706000 { > + compatible = "altr,socfpga-fpga-mgr"; > + reg = <0xFF706000 0x1000 > + 0xFFB90000 0x1000>; > + interrupts = <0 175 4>; > + }; > }; > }; Please keep the format in line with the rest of the file: Hexvalues are lowercase and no 0x after the "fpgamgr@". I'd say "And please put the node in its proper position according to its address", but I see just now, that the dtsi is all messed up right now anyway :-( Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: socfpga: dts: add fpga manager 2015-10-13 18:57 ` Steffen Trumtrar @ 2015-10-13 19:18 ` Dinh Nguyen 2015-10-13 19:46 ` atull 2015-10-13 20:20 ` Steffen Trumtrar 0 siblings, 2 replies; 7+ messages in thread From: Dinh Nguyen @ 2015-10-13 19:18 UTC (permalink / raw) To: Steffen Trumtrar Cc: atull, Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux-kernel, Rob Herring, Kumar Gala, delicious.quinoa, linux-arm-kernel On Tue, 13 Oct 2015, Steffen Trumtrar wrote: > Hi Alan! > > On Tue, Oct 13, 2015 at 01:28:20PM -0500, atull@opensource.altera.com wrote: > > From: Alan Tull <atull@opensource.altera.com> > > > > Add FPGA manager to device tree for SoCFPGA. > > > > Signed-off-by: Alan Tull <atull@opensource.altera.com> > > --- > > arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi > > index 314e589..e37ba11 100644 > > --- a/arch/arm/boot/dts/socfpga.dtsi > > +++ b/arch/arm/boot/dts/socfpga.dtsi > > @@ -834,5 +834,12 @@ > > compatible = "altr,sys-mgr", "syscon"; > > reg = <0xffd08000 0x4000>; > > }; > > + > > + hps_0_fpgamgr: fpgamgr@0xff706000 { > > + compatible = "altr,socfpga-fpga-mgr"; > > + reg = <0xFF706000 0x1000 > > + 0xFFB90000 0x1000>; > > + interrupts = <0 175 4>; > > + }; > > }; > > }; > > Please keep the format in line with the rest of the file: > Hexvalues are lowercase and no 0x after the "fpgamgr@". > I'd say "And please put the node in its proper position > according to its address", but I see just now, that the > dtsi is all messed up right now anyway :-( > How so? Patches are welcomed! Please fix the "fpgamgr@" and lower case the hex values. But I would prefer the nodes to be in alphabetical order. Also, you need to document the "altr,socfpga-fpga-mgr". Thanks, Dinh > Regards, > Steffen > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > BR, Dinh ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: socfpga: dts: add fpga manager 2015-10-13 19:18 ` Dinh Nguyen @ 2015-10-13 19:46 ` atull 2015-10-13 20:27 ` Steffen Trumtrar 2015-10-13 20:20 ` Steffen Trumtrar 1 sibling, 1 reply; 7+ messages in thread From: atull @ 2015-10-13 19:46 UTC (permalink / raw) To: Dinh Nguyen Cc: Steffen Trumtrar, Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux-kernel, Rob Herring, Kumar Gala, delicious.quinoa, linux-arm-kernel On Tue, 13 Oct 2015, Dinh Nguyen wrote: > On Tue, 13 Oct 2015, Steffen Trumtrar wrote: > > > Hi Alan! > > > > On Tue, Oct 13, 2015 at 01:28:20PM -0500, atull@opensource.altera.com wrote: > > > From: Alan Tull <atull@opensource.altera.com> > > > > > > Add FPGA manager to device tree for SoCFPGA. > > > > > > Signed-off-by: Alan Tull <atull@opensource.altera.com> > > > --- > > > arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ > > > 1 file changed, 7 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi > > > index 314e589..e37ba11 100644 > > > --- a/arch/arm/boot/dts/socfpga.dtsi > > > +++ b/arch/arm/boot/dts/socfpga.dtsi > > > @@ -834,5 +834,12 @@ > > > compatible = "altr,sys-mgr", "syscon"; > > > reg = <0xffd08000 0x4000>; > > > }; > > > + > > > + hps_0_fpgamgr: fpgamgr@0xff706000 { > > > + compatible = "altr,socfpga-fpga-mgr"; > > > + reg = <0xFF706000 0x1000 > > > + 0xFFB90000 0x1000>; > > > + interrupts = <0 175 4>; > > > + }; > > > }; > > > }; > > > > Please keep the format in line with the rest of the file: > > Hexvalues are lowercase and no 0x after the "fpgamgr@". > > I'd say "And please put the node in its proper position > > according to its address", but I see just now, that the > > dtsi is all messed up right now anyway :-( > > Hi Steffen, Thanks for the review. I'll fix these things you pointed out. "all messed up" seems kind of unclear and overstated here. From the context, I assume you mean that you are pointing out that the nodes are in more or less alphatetical order and not in order of address. Is that what you mean or is the dtsi really all messed up? Alan > > How so? Patches are welcomed! > > Please fix the "fpgamgr@" and lower case the hex values. But I would prefer the > nodes to be in alphabetical order. > > Also, you need to document the "altr,socfpga-fpga-mgr". > > Thanks, > Dinh > > Regards, > > Steffen > > > > -- > > Pengutronix e.K. | | > > Industrial Linux Solutions | http://www.pengutronix.de/ | > > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > > > BR, > Dinh > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: socfpga: dts: add fpga manager 2015-10-13 19:46 ` atull @ 2015-10-13 20:27 ` Steffen Trumtrar 0 siblings, 0 replies; 7+ messages in thread From: Steffen Trumtrar @ 2015-10-13 20:27 UTC (permalink / raw) To: atull Cc: Dinh Nguyen, Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux-kernel, Rob Herring, Kumar Gala, delicious.quinoa, linux-arm-kernel Hi! On Tue, Oct 13, 2015 at 09:46:15PM +0200, atull wrote: > On Tue, 13 Oct 2015, Dinh Nguyen wrote: > > > On Tue, 13 Oct 2015, Steffen Trumtrar wrote: > > > > > Hi Alan! > > > > > > On Tue, Oct 13, 2015 at 01:28:20PM -0500, atull@opensource.altera.com wrote: > > > > From: Alan Tull <atull@opensource.altera.com> > > > > > > > > Add FPGA manager to device tree for SoCFPGA. > > > > > > > > Signed-off-by: Alan Tull <atull@opensource.altera.com> > > > > --- > > > > arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ > > > > 1 file changed, 7 insertions(+) > > > > > > > > diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi > > > > index 314e589..e37ba11 100644 > > > > --- a/arch/arm/boot/dts/socfpga.dtsi > > > > +++ b/arch/arm/boot/dts/socfpga.dtsi > > > > @@ -834,5 +834,12 @@ > > > > compatible = "altr,sys-mgr", "syscon"; > > > > reg = <0xffd08000 0x4000>; > > > > }; > > > > + > > > > + hps_0_fpgamgr: fpgamgr@0xff706000 { > > > > + compatible = "altr,socfpga-fpga-mgr"; > > > > + reg = <0xFF706000 0x1000 > > > > + 0xFFB90000 0x1000>; > > > > + interrupts = <0 175 4>; > > > > + }; > > > > }; > > > > }; > > > > > > Please keep the format in line with the rest of the file: > > > Hexvalues are lowercase and no 0x after the "fpgamgr@". > > > I'd say "And please put the node in its proper position > > > according to its address", but I see just now, that the > > > dtsi is all messed up right now anyway :-( > > > > > "all messed up" seems kind of unclear and overstated here. From the > context, I assume you mean that you are pointing out that the nodes > are in more or less alphatetical order and not in order of address. Is > that what you mean or is the dtsi really all messed up? > "more or less" was the thing that bugged me. And for the record: I didn't try to blame anyone for this. I put many of the nodes there myself ;-) Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: socfpga: dts: add fpga manager 2015-10-13 19:18 ` Dinh Nguyen 2015-10-13 19:46 ` atull @ 2015-10-13 20:20 ` Steffen Trumtrar 1 sibling, 0 replies; 7+ messages in thread From: Steffen Trumtrar @ 2015-10-13 20:20 UTC (permalink / raw) To: Dinh Nguyen Cc: atull, Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux-kernel, Rob Herring, Kumar Gala, delicious.quinoa, linux-arm-kernel On Tue, Oct 13, 2015 at 02:18:22PM -0500, Dinh Nguyen wrote: > On Tue, 13 Oct 2015, Steffen Trumtrar wrote: > > > Hi Alan! > > > > On Tue, Oct 13, 2015 at 01:28:20PM -0500, atull@opensource.altera.com wrote: > > > From: Alan Tull <atull@opensource.altera.com> > > > > > > Add FPGA manager to device tree for SoCFPGA. > > > > > > Signed-off-by: Alan Tull <atull@opensource.altera.com> > > > --- > > > arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ > > > 1 file changed, 7 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi > > > index 314e589..e37ba11 100644 > > > --- a/arch/arm/boot/dts/socfpga.dtsi > > > +++ b/arch/arm/boot/dts/socfpga.dtsi > > > @@ -834,5 +834,12 @@ > > > compatible = "altr,sys-mgr", "syscon"; > > > reg = <0xffd08000 0x4000>; > > > }; > > > + > > > + hps_0_fpgamgr: fpgamgr@0xff706000 { > > > + compatible = "altr,socfpga-fpga-mgr"; > > > + reg = <0xFF706000 0x1000 > > > + 0xFFB90000 0x1000>; > > > + interrupts = <0 175 4>; > > > + }; > > > }; > > > }; > > > > Please keep the format in line with the rest of the file: > > Hexvalues are lowercase and no 0x after the "fpgamgr@". > > I'd say "And please put the node in its proper position > > according to its address", but I see just now, that the > > dtsi is all messed up right now anyway :-( > > > > How so? Patches are welcomed! Done :-) > > Please fix the "fpgamgr@" and lower case the hex values. But I would prefer the > nodes to be in alphabetical order. > Ah, that explains it, I guess. I always sorted my patches by address :-) The thing is, now all nodes are sorted alphabetically by phandle, the clkmgr however seems to be ordered by address. I prefer address btw because it is a fixed hardware constant. For the dts files sorting via phandle is again fixed via the dtsi. Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: socfpga: dts: add fpga manager 2015-10-13 18:28 [PATCH] ARM: socfpga: dts: add fpga manager atull 2015-10-13 18:57 ` Steffen Trumtrar @ 2015-10-13 18:58 ` Moritz Fischer 1 sibling, 0 replies; 7+ messages in thread From: Moritz Fischer @ 2015-10-13 18:58 UTC (permalink / raw) To: Alan Tull Cc: dinguyen@opensource.altera.com, Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux-kernel, Rob Herring, Kumar Gala, Alan Tull, linux-arm-kernel On Tue, Oct 13, 2015 at 11:28 AM, <atull@opensource.altera.com> wrote: > From: Alan Tull <atull@opensource.altera.com> > > Add FPGA manager to device tree for SoCFPGA. > > Signed-off-by: Alan Tull <atull@opensource.altera.com> Reviewed-by: Moritz Fischer <moritz.fischer@ettus.com> > --- > arch/arm/boot/dts/socfpga.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi > index 314e589..e37ba11 100644 > --- a/arch/arm/boot/dts/socfpga.dtsi > +++ b/arch/arm/boot/dts/socfpga.dtsi > @@ -834,5 +834,12 @@ > compatible = "altr,sys-mgr", "syscon"; > reg = <0xffd08000 0x4000>; > }; > + > + hps_0_fpgamgr: fpgamgr@0xff706000 { > + compatible = "altr,socfpga-fpga-mgr"; > + reg = <0xFF706000 0x1000 > + 0xFFB90000 0x1000>; > + interrupts = <0 175 4>; > + }; > }; > }; > -- > 1.9.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-10-13 20:48 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-10-13 18:28 [PATCH] ARM: socfpga: dts: add fpga manager atull 2015-10-13 18:57 ` Steffen Trumtrar 2015-10-13 19:18 ` Dinh Nguyen 2015-10-13 19:46 ` atull 2015-10-13 20:27 ` Steffen Trumtrar 2015-10-13 20:20 ` Steffen Trumtrar 2015-10-13 18:58 ` Moritz Fischer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox