Re: [PATCH 1/8] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-11 Thread Ramon Fried
On Tue, Jul 5, 2022 at 11:57 AM Daniel Thompson wrote: > > On Tue, Jul 05, 2022 at 11:05:04AM +0530, Sumit Garg wrote: > > Hi Daniel, > > > > Thanks for your review. > > > > On Mon, 4 Jul 2022 at 21:28, Daniel Thompson > > wrote: > > > > > > On Mon, Jul 04, 2022 at 06:28:38PM +0530, Sumit Garg

Re: [PATCH 1/8] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-05 Thread Daniel Thompson
On Tue, Jul 05, 2022 at 11:05:04AM +0530, Sumit Garg wrote: > Hi Daniel, > > Thanks for your review. > > On Mon, 4 Jul 2022 at 21:28, Daniel Thompson > wrote: > > > > On Mon, Jul 04, 2022 at 06:28:38PM +0530, Sumit Garg wrote: > > > U-boot specific DT properties belong to *-uboot.dtsi > > > >

Re: [PATCH 1/8] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-04 Thread Sumit Garg
Hi Daniel, Thanks for your review. On Mon, 4 Jul 2022 at 21:28, Daniel Thompson wrote: > > On Mon, Jul 04, 2022 at 06:28:38PM +0530, Sumit Garg wrote: > > U-boot specific DT properties belong to *-uboot.dtsi > > ... and are already included in starqltechn-uboot.dtsi (which is the > only current

Re: [PATCH 1/8] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-04 Thread Daniel Thompson
On Mon, Jul 04, 2022 at 06:28:38PM +0530, Sumit Garg wrote: > U-boot specific DT properties belong to *-uboot.dtsi ... and are already included in starqltechn-uboot.dtsi (which is the only current consumer of sdm845.dtsi). Adding fuller comments, such as the above, makes things much easier to

[PATCH 1/8] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-04 Thread Sumit Garg
U-boot specific DT properties belong to *-uboot.dtsi, so remove corresponding redundant properties. Signed-off-by: Sumit Garg --- arch/arm/dts/sdm845.dtsi | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/arm/dts/sdm845.dtsi b/arch/arm/dts/sdm845.dtsi index 6f2fb20d68..88030156d9