Re: linux-next: manual merge of the thermal tree with the imx-mxs tree

[Zhang Rui]

Hi, Mark,

thanks for the fix.

On Mon, 2017-10-16 at 10:17 +0100, Mark Brown wrote:
> Hi Zhang,
> 
> Today's linux-next merge of the thermal tree got a conflict in:
> 
> Â arch/arm/boot/dts/imx6ul.dtsi
> 
> between commit:
> 
> ÂÂÂefb9adb274754 ("ARM: dts: imx6ul: Remove leading zeroes from unit
> addresses")
> 
> from the imx-mxs tree and commit:
> 
> ÂÂÂ1dc31d4981dd9 ("ARM: dts: imx6ul: Add imx6ul-tempmon")
> 
> from the thermal tree.
> 
> I fixed it up (see below) and can carry the fix as necessary. This
> is now fixed as far as linux-next is concerned, but any non trivial
> conflicts should be mentioned to your upstream maintainer when your
> tree
> is submitted for merging.ÂÂYou may also want to consider cooperating
> with the maintainer of the conflicting tree to minimise any
> particularly
> complex conflicts.ÂÂUsually DTS changes go through arm-soc to avoid
> issues like this.

Shawn,

https://patchwork.kernel.org/patch/9841191/
https://patchwork.kernel.org/patch/9841193/
Should I drop these two dts patches and leave them to you?

thanks,
rui

> 
> diff --cc arch/arm/boot/dts/imx6ul.dtsi
> index 2057ee695a66,51717d54f285..000000000000
> --- a/arch/arm/boot/dts/imx6ul.dtsi
> +++ b/arch/arm/boot/dts/imx6ul.dtsi
> @@@ -860,13 -869,23 +869,23 @@@
> ÂÂ				reg = <0x021b0000 0x4000>;
> ÂÂ			};
> ÂÂ
> Â-			ocotp: ocotp-ctrl@021bc000 {
> Â+			ocotp: ocotp-ctrl@21bc000 {
> +Â				#address-cells = <1>;
> +Â				#size-cells = <1>;
> ÂÂ				compatible = "fsl,imx6ul-ocotp",
> "syscon";
> ÂÂ				reg = <0x021bc000 0x4000>;
> ÂÂ				clocks = <&clks IMX6UL_CLK_OCOTP>;
> +Â
> +Â				tempmon_calib: calib@38 {
> +Â					reg = <0x38 4>;
> +Â				};
> +Â
> +Â				tempmon_temp_grade: temp-grade@20 {
> +Â					reg = <0x20 4>;
> +Â				};
> ÂÂ			};
> ÂÂ
> Â-			lcdif: lcdif@021c8000 {
> Â+			lcdif: lcdif@21c8000 {
> ÂÂ				compatible = "fsl,imx6ul-lcdif",
> "fsl,imx28-lcdif";
> ÂÂ				reg = <0x021c8000 0x4000>;
> ÂÂ				interrupts = <GIC_SPI 5
> IRQ_TYPE_LEVEL_HIGH>;