On Tue, 2019-02-26 at 16:42 +0100, Michal Simek wrote: > On 26. 02. 19 15:28, Chee, Tien Fong wrote: > > On Tue, 2019-02-26 at 15:06 +0100, Michal Simek wrote: > > > On 19. 02. 19 4:47, tien.fong.c...@intel.com wrote: > > > > From: Tien Fong Chee <tien.fong.c...@intel.com> > > > > > > > > This patch adds description on properties about file name used for > > > > both > > > > peripheral bitstream and core bitstream. > > > > > > > > Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com> > > > > > > > > --- > > > > > > > > changes for v8 > > > > - Removed explanation about support for altr,bitstream-core > > > > > > > > changes for v7 > > > > - Provided example of setting FPGA FIT image for both early IO > > > > release > > > > and full release FPGA configuration. > > > > --- > > > > .../fpga/altera-socfpga-a10-fpga-mgr.txt | 26 > > > > +++++++++++++++++++++- > > > > 1 file changed, 25 insertions(+), 1 deletion(-) > > > > > > > > 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..da210bf 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,8 +7,31 @@ 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. > > > > +- altr,bitstream : Fit image file name for both FPGA peripheral > > > > bitstream, > > > > + FPGA core bitstream and full bitstream. > > > > > > > By adding new required property you are automatically saying that you > > > want to break all current users. > > This is company's product specific property, that's why with prefix > > "altr". DT allows that ,right? > > no issue with altr prefix. Issue is that you add a required property and > breaking all current users. > It should be optional.
This parameter is only for Arria10, which at this point is not fully supported in mainline uboot. So this doesnt affect any existing designs, no? --dalon > > M > > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
