From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Kocialkowski Date: Fri, 26 Apr 2019 15:42:38 +0200 Subject: [U-Boot] [PATCH v3.1 03/13] rockchip: dts: rk3399: Create initial rk3399-u-boot.dtsi In-Reply-To: References: <20190426131817.16776-1-jagan@amarulasolutions.com> Message-ID: <4dc7e2d0f4f0da508224e3ffa089f8780395493f.camel@bootlin.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On Fri, 2019-04-26 at 19:07 +0530, Jagan Teki wrote: > On Fri, Apr 26, 2019 at 7:04 PM Paul Kocialkowski > 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 > > > Reviewed-by: Paul Kocialkowski > > > > 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. Could you clean that up and make a proper standalone series that creates and populates rk3399-u-boot.dtsi? Cheers, Paul > [1] https://patchwork.ozlabs.org/patch/1091543/ -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com