On Tue, 2019-02-26 at 07:58 -0800, Dalon L Westergreen wrote: > 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?
Yeah, how this breaking all current users? This property in only used for the A10 fpga driver with fit implementation. > > --dalon > > > > > > > M > > > > > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot