Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On Rab, 2017-09-27 at 10:29 +0200, Marek Vasut wrote: > On 09/27/2017 05:12 AM, Chee, Tien Fong wrote: > > > > On Sel, 2017-09-26 at 12:30 +0200, Marek Vasut wrote: > > > > > > On 09/26/2017 10:54 AM, Chee, Tien Fong wrote: > > > > > > > > > > > > On Isn, 2017-09-25 at 11:00 +0200, Marek Vasut wrote: > > > > > > > > > > > > > > > On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > > > > > > > > > > > > > > > > > > > > > > > > From: Tien Fong Chee> > > > > > > > > > > > This patch adds description on properties about location of > > > > > > FPGA > > > > > > RBFs are > > > > > > stored, type and functionality of RBF used to configure > > > > > > FPGA. > > > > > > > > > > > > Signed-off-by: Tien Fong Chee > > > > > Why does this patch have different tags than 1/19 ? Please > > > > > keep > > > > > things > > > > > consistent ... > > > > > > > > > Not get you. What's you means for tags? > > > ARM: socfpga: , not the random doc: dtbinding: . > > > Heck, the first and second patch change the same file, yet have > > > different tags, why ? > > > > > I ported patch 01 from Linux, so i keep everything intact. For > > patch > > 02, i put doc:dtbinding because i think that is more descriptive to > > the > > file i changed. > > I can change to ARM:socfpga . > The tags are standardized ... inventing new random ones only messes > things up. > Okay. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --- > > > > > > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > > > > > mgr.txt > > > > > > > > > > > > > > 11 > > > > > > +++ > > > > > > 1 file changed, 11 insertions(+) > > > > > > > > > > > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga- > > > > > > a10- > > > > > > fpga- > > > > > > mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10- > > > > > > fpga- > > > > > > mgr.txt > > > > > > index 2fd8e7a..7abb746 100644 > > > > > > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10- > > > > > > fpga- > > > > > > mgr.txt > > > > > > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10- > > > > > > fpga- > > > > > > mgr.txt > > > > > > @@ -7,6 +7,14 @@ Required properties: > > > > > > - The second index is for writing FPGA > > > > > > configuration data. > > > > > > - resets : Phandle and reset specifier for the > > > > > > device's > > > > > > reset. > > > > > > - clocks : Clocks used by the device. > > > > > > +- bitstream_periph : FPGA peripheral raw binary file which > > > > > > is > > > > > > used > > > > > > to > > > > > > + initialize FPGA IOs, PLL, IO48 and > > > > > > DDR. > > > > > > +- bitstream_core : FPGA core raw binary file contains FPGA > > > > > > design > > > > > > which is used > > > > > > + to program FPGA CRAM and ERAM. > > > > > > +- bitstream_devpart : Partition of flash device where > > > > > > bitstream > > > > > > files are > > > > > > + stored. > > > > > > + - dev is flash device > > > > > > number, > > > > > > part is flash > > > > > > + device partition. > > > > > > > > > > > > Example: > > > > > > > > > > > > @@ -16,4 +24,7 @@ Example: > > > > > > 0xffcfe400 0x20>; > > > > > > clocks = <_mp_clk>; > > > > > > resets = < FPGAMGR_RESET>; > > > > > > + bitstream_periph = > > > > > > "ghrd_10as066n2.periph.rbf.mkimage"; > > > > > > + bitstream_core = > > > > > > "ghrd_10as066n2.core.rbf.mkimage"; > > > > > > + bitstream_devpart = "0:1"; > > > > > > }; > > > > > > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On 09/27/2017 05:12 AM, Chee, Tien Fong wrote: > On Sel, 2017-09-26 at 12:30 +0200, Marek Vasut wrote: >> On 09/26/2017 10:54 AM, Chee, Tien Fong wrote: >>> >>> On Isn, 2017-09-25 at 11:00 +0200, Marek Vasut wrote: On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > > > From: Tien Fong Chee> > This patch adds description on properties about location of > FPGA > RBFs are > stored, type and functionality of RBF used to configure FPGA. > > Signed-off-by: Tien Fong Chee Why does this patch have different tags than 1/19 ? Please keep things consistent ... >>> Not get you. What's you means for tags? >> ARM: socfpga: , not the random doc: dtbinding: . >> Heck, the first and second patch change the same file, yet have >> different tags, why ? >> > I ported patch 01 from Linux, so i keep everything intact. For patch > 02, i put doc:dtbinding because i think that is more descriptive to the > file i changed. > I can change to ARM:socfpga . The tags are standardized ... inventing new random ones only messes things up. >>> > > --- > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > | 11 > +++ > 1 file changed, 11 insertions(+) > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10- > fpga- > mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10- > fpga- > mgr.txt > index 2fd8e7a..7abb746 100644 > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > mgr.txt > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > mgr.txt > @@ -7,6 +7,14 @@ Required properties: > - The second index is for writing FPGA > configuration data. > - resets : Phandle and reset specifier for the device's > reset. > - clocks : Clocks used by the device. > +- bitstream_periph : FPGA peripheral raw binary file which is > used > to > + initialize FPGA IOs, PLL, IO48 and DDR. > +- bitstream_core : FPGA core raw binary file contains FPGA > design > which is used > + to program FPGA CRAM and ERAM. > +- bitstream_devpart : Partition of flash device where > bitstream > files are > + stored. > + - dev is flash device > number, > part is flash > + device partition. > > Example: > > @@ -16,4 +24,7 @@ Example: > 0xffcfe400 0x20>; > clocks = <_mp_clk>; > resets = < FPGAMGR_RESET>; > + bitstream_periph = > "ghrd_10as066n2.periph.rbf.mkimage"; > + bitstream_core = > "ghrd_10as066n2.core.rbf.mkimage"; > + bitstream_devpart = "0:1"; > }; > -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On Sel, 2017-09-26 at 12:30 +0200, Marek Vasut wrote: > On 09/26/2017 10:54 AM, Chee, Tien Fong wrote: > > > > On Isn, 2017-09-25 at 11:00 +0200, Marek Vasut wrote: > > > > > > On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > > > > > > > > > > > > From: Tien Fong Chee> > > > > > > > This patch adds description on properties about location of > > > > FPGA > > > > RBFs are > > > > stored, type and functionality of RBF used to configure FPGA. > > > > > > > > Signed-off-by: Tien Fong Chee > > > Why does this patch have different tags than 1/19 ? Please keep > > > things > > > consistent ... > > > > > Not get you. What's you means for tags? > ARM: socfpga: , not the random doc: dtbinding: . > Heck, the first and second patch change the same file, yet have > different tags, why ? > I ported patch 01 from Linux, so i keep everything intact. For patch 02, i put doc:dtbinding because i think that is more descriptive to the file i changed. I can change to ARM:socfpga . > > > > > > > > > > > > > --- > > > > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > > > > | 11 > > > > +++ > > > > 1 file changed, 11 insertions(+) > > > > > > > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10- > > > > fpga- > > > > mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10- > > > > fpga- > > > > mgr.txt > > > > index 2fd8e7a..7abb746 100644 > > > > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > > > mgr.txt > > > > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > > > mgr.txt > > > > @@ -7,6 +7,14 @@ Required properties: > > > > - The second index is for writing FPGA > > > > configuration data. > > > > - resets : Phandle and reset specifier for the device's > > > > reset. > > > > - clocks : Clocks used by the device. > > > > +- bitstream_periph : FPGA peripheral raw binary file which is > > > > used > > > > to > > > > + initialize FPGA IOs, PLL, IO48 and DDR. > > > > +- bitstream_core : FPGA core raw binary file contains FPGA > > > > design > > > > which is used > > > > + to program FPGA CRAM and ERAM. > > > > +- bitstream_devpart : Partition of flash device where > > > > bitstream > > > > files are > > > > + stored. > > > > + - dev is flash device > > > > number, > > > > part is flash > > > > + device partition. > > > > > > > > Example: > > > > > > > > @@ -16,4 +24,7 @@ Example: > > > > 0xffcfe400 0x20>; > > > > clocks = <_mp_clk>; > > > > resets = < FPGAMGR_RESET>; > > > > + bitstream_periph = > > > > "ghrd_10as066n2.periph.rbf.mkimage"; > > > > + bitstream_core = > > > > "ghrd_10as066n2.core.rbf.mkimage"; > > > > + bitstream_devpart = "0:1"; > > > > }; > > > > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On 09/26/2017 10:54 AM, Chee, Tien Fong wrote: > On Isn, 2017-09-25 at 11:00 +0200, Marek Vasut wrote: >> On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: >>> >>> From: Tien Fong Chee>>> >>> This patch adds description on properties about location of FPGA >>> RBFs are >>> stored, type and functionality of RBF used to configure FPGA. >>> >>> Signed-off-by: Tien Fong Chee >> Why does this patch have different tags than 1/19 ? Please keep >> things >> consistent ... >> > Not get you. What's you means for tags? ARM: socfpga: , not the random doc: dtbinding: . Heck, the first and second patch change the same file, yet have different tags, why ? >>> --- >>> doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt | 11 >>> +++ >>> 1 file changed, 11 insertions(+) >>> >>> diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- >>> mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- >>> mgr.txt >>> index 2fd8e7a..7abb746 100644 >>> --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt >>> +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt >>> @@ -7,6 +7,14 @@ Required properties: >>> - The second index is for writing FPGA >>> configuration data. >>> - resets : Phandle and reset specifier for the device's reset. >>> - clocks : Clocks used by the device. >>> +- bitstream_periph : FPGA peripheral raw binary file which is used >>> to >>> + initialize FPGA IOs, PLL, IO48 and DDR. >>> +- bitstream_core : FPGA core raw binary file contains FPGA design >>> which is used >>> + to program FPGA CRAM and ERAM. >>> +- bitstream_devpart : Partition of flash device where bitstream >>> files are >>> + stored. >>> + - dev is flash device number, >>> part is flash >>> + device partition. >>> >>> Example: >>> >>> @@ -16,4 +24,7 @@ Example: >>> 0xffcfe400 0x20>; >>> clocks = <_mp_clk>; >>> resets = < FPGAMGR_RESET>; >>> + bitstream_periph = >>> "ghrd_10as066n2.periph.rbf.mkimage"; >>> + bitstream_core = >>> "ghrd_10as066n2.core.rbf.mkimage"; >>> + bitstream_devpart = "0:1"; >>> }; >>> -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On Isn, 2017-09-25 at 11:00 +0200, Marek Vasut wrote: > On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > > > > From: Tien Fong Chee> > > > This patch adds description on properties about location of FPGA > > RBFs are > > stored, type and functionality of RBF used to configure FPGA. > > > > Signed-off-by: Tien Fong Chee > Why does this patch have different tags than 1/19 ? Please keep > things > consistent ... > Not get you. What's you means for tags? > > > > --- > > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt | 11 > > +++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > mgr.txt > > index 2fd8e7a..7abb746 100644 > > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > > @@ -7,6 +7,14 @@ Required properties: > > - The second index is for writing FPGA > > configuration data. > > - resets : Phandle and reset specifier for the device's reset. > > - clocks : Clocks used by the device. > > +- bitstream_periph : FPGA peripheral raw binary file which is used > > to > > + initialize FPGA IOs, PLL, IO48 and DDR. > > +- bitstream_core : FPGA core raw binary file contains FPGA design > > which is used > > + to program FPGA CRAM and ERAM. > > +- bitstream_devpart : Partition of flash device where bitstream > > files are > > + stored. > > + - dev is flash device number, > > part is flash > > + device partition. > > > > Example: > > > > @@ -16,4 +24,7 @@ Example: > > 0xffcfe400 0x20>; > > clocks = <_mp_clk>; > > resets = < FPGAMGR_RESET>; > > + bitstream_periph = > > "ghrd_10as066n2.periph.rbf.mkimage"; > > + bitstream_core = > > "ghrd_10as066n2.core.rbf.mkimage"; > > + bitstream_devpart = "0:1"; > > }; > > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On Isn, 2017-09-25 at 11:01 +0200, Marek Vasut wrote: > On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > > > > From: Tien Fong Chee> > > > This patch adds description on properties about location of FPGA > > RBFs are > > stored, type and functionality of RBF used to configure FPGA. > > > > Signed-off-by: Tien Fong Chee > > --- > > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt | 11 > > +++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga- > > mgr.txt > > index 2fd8e7a..7abb746 100644 > > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > > @@ -7,6 +7,14 @@ Required properties: > > - The second index is for writing FPGA > > configuration data. > > - resets : Phandle and reset specifier for the device's reset. > > - clocks : Clocks used by the device. > > +- bitstream_periph : FPGA peripheral raw binary file which is used > > to > > + initialize FPGA IOs, PLL, IO48 and DDR. > > +- bitstream_core : FPGA core raw binary file contains FPGA design > > which is used > > + to program FPGA CRAM and ERAM. > > +- bitstream_devpart : Partition of flash device where bitstream > > files are > > + stored. > > + - dev is flash device number, > > part is flash > > + device partition. > > > > Example: > > > > @@ -16,4 +24,7 @@ Example: > > 0xffcfe400 0x20>; > > clocks = <_mp_clk>; > > resets = < FPGAMGR_RESET>; > > + bitstream_periph = > > "ghrd_10as066n2.periph.rbf.mkimage"; > > + bitstream_core = > > "ghrd_10as066n2.core.rbf.mkimage"; > > + bitstream_devpart = "0:1"; > These should probably be altr,something-something ... they are > definitely not generic bindings > Okay. > > > > }; > > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee> > This patch adds description on properties about location of FPGA RBFs are > stored, type and functionality of RBF used to configure FPGA. > > Signed-off-by: Tien Fong Chee > --- > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt | 11 > +++ > 1 file changed, 11 insertions(+) > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > index 2fd8e7a..7abb746 100644 > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > @@ -7,6 +7,14 @@ Required properties: > - The second index is for writing FPGA configuration data. > - resets : Phandle and reset specifier for the device's reset. > - clocks : Clocks used by the device. > +- bitstream_periph : FPGA peripheral raw binary file which is used to > + initialize FPGA IOs, PLL, IO48 and DDR. > +- bitstream_core : FPGA core raw binary file contains FPGA design which is > used > + to program FPGA CRAM and ERAM. > +- bitstream_devpart : Partition of flash device where bitstream files are > +stored. > + - dev is flash device number, part is > flash > + device partition. > > Example: > > @@ -16,4 +24,7 @@ Example: > 0xffcfe400 0x20>; > clocks = <_mp_clk>; > resets = < FPGAMGR_RESET>; > + bitstream_periph = "ghrd_10as066n2.periph.rbf.mkimage"; > + bitstream_core = "ghrd_10as066n2.core.rbf.mkimage"; > + bitstream_devpart = "0:1"; These should probably be altr,something-something ... they are definitely not generic bindings > }; > -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
On 09/25/2017 10:39 AM, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee> > This patch adds description on properties about location of FPGA RBFs are > stored, type and functionality of RBF used to configure FPGA. > > Signed-off-by: Tien Fong Chee Why does this patch have different tags than 1/19 ? Please keep things consistent ... > --- > doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt | 11 > +++ > 1 file changed, 11 insertions(+) > > diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > index 2fd8e7a..7abb746 100644 > --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt > @@ -7,6 +7,14 @@ Required properties: > - The second index is for writing FPGA configuration data. > - resets : Phandle and reset specifier for the device's reset. > - clocks : Clocks used by the device. > +- bitstream_periph : FPGA peripheral raw binary file which is used to > + initialize FPGA IOs, PLL, IO48 and DDR. > +- bitstream_core : FPGA core raw binary file contains FPGA design which is > used > + to program FPGA CRAM and ERAM. > +- bitstream_devpart : Partition of flash device where bitstream files are > +stored. > + - dev is flash device number, part is > flash > + device partition. > > Example: > > @@ -16,4 +24,7 @@ Example: > 0xffcfe400 0x20>; > clocks = <_mp_clk>; > resets = < FPGAMGR_RESET>; > + bitstream_periph = "ghrd_10as066n2.periph.rbf.mkimage"; > + bitstream_core = "ghrd_10as066n2.core.rbf.mkimage"; > + bitstream_devpart = "0:1"; > }; > -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH v2 02/19] doc: dtbinding: Description on FPGA RBF properties at Arria 10 FPGA manager
From: Tien Fong CheeThis patch adds description on properties about location of FPGA RBFs are stored, type and functionality of RBF used to configure FPGA. Signed-off-by: Tien Fong Chee --- doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt | 11 +++ 1 file changed, 11 insertions(+) diff --git a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt index 2fd8e7a..7abb746 100644 --- a/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt +++ b/doc/device-tree-bindings/fpga/altera-socfpga-a10-fpga-mgr.txt @@ -7,6 +7,14 @@ Required properties: - The second index is for writing FPGA configuration data. - resets : Phandle and reset specifier for the device's reset. - clocks : Clocks used by the device. +- bitstream_periph : FPGA peripheral raw binary file which is used to + initialize FPGA IOs, PLL, IO48 and DDR. +- bitstream_core : FPGA core raw binary file contains FPGA design which is used + to program FPGA CRAM and ERAM. +- bitstream_devpart : Partition of flash device where bitstream files are + stored. + - dev is flash device number, part is flash + device partition. Example: @@ -16,4 +24,7 @@ Example: 0xffcfe400 0x20>; clocks = <_mp_clk>; resets = < FPGAMGR_RESET>; + bitstream_periph = "ghrd_10as066n2.periph.rbf.mkimage"; + bitstream_core = "ghrd_10as066n2.core.rbf.mkimage"; + bitstream_devpart = "0:1"; }; -- 2.2.0 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot