On Fri, Apr 26, 2019 at 7:12 PM Paul Kocialkowski <paul.kocialkowski@xxxxxxxxxxx> wrote: > > Hi, > > On Fri, 2019-04-26 at 19:07 +0530, Jagan Teki wrote: > > On Fri, Apr 26, 2019 at 7:04 PM Paul Kocialkowski > > <paul.kocialkowski@xxxxxxxxxxx> wrote: > > > Hi, > > > > > > On Fri, 2019-04-26 at 18:48 +0530, Jagan Teki wrote: > > > > u-boot,dm-pre-reloc is required for SDMMC booted rk3399 boards and > > > > which is U-Boot specific devicetrees binding. > > > > > > > > Move it on global rk3399-u-boot.dtsi file and rest of the U-Boot > > > > bindings will move it future based on the requirement. > > > > > > > > This would help to sync the devicetrees from Linux whenever required > > > > instead of adding specific nodes. > > > > > > > > Signed-off-by: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> > > > > Reviewed-by: Paul Kocialkowski <paul.kocialkowski@xxxxxxxxxxx> > > > > > > Since your patch has changed significantly, you can no longer include a > > > Reviewed-by that was given on a previous iteration. > > > > > > It looks like we are still missing bits to have rk3399-u-boot.dtsi > > > included, so this patch does not get my Reviewed-by tag. > > > > It will include rockchip-u-boot.dtsi automatically and I made the > > receptive changes to include other files in this patch [1] and sure I > > will ask Philipp to remove the reviewed-by tag, thanks. > > Most importantly (now that I see the new series), you should certainly > have sent that patch as part of your new series because we now have an > inter-dependency between both series. It is not like an inter-dependency the previous rk3399-u-boot.dtsi is not included by these files so they would work as before. and now this series would need this change to include rk3399-u-boot.dtsi since the goal here to add binman node to common for all rk3399 dts files. _______________________________________________ Linux-rockchip mailing list Linux-rockchip@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-rockchip