Hi Shawn,
> -----Original Message-----
> From: Shawn Guo <shawnguo@xxxxxxxxxx>
> Sent: 2019年9月14日 10:43
> To: Andy Tang <andy.tang@xxxxxxx>
> Cc: Leo Li <leoyang.li@xxxxxxx>; robh+dt@xxxxxxxxxx;
> mark.rutland@xxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx
> Subject: [EXT] Re: [PATCH] arm64: dts: ls1028a: fix a compatible issue
>
> Caution: EXT Email
>
> On Thu, Aug 29, 2019 at 03:34:39PM +0800, Yuantian Tang wrote:
> > The I2C multiplexer used on ls1028aqds is PCA9547, not PCA9847.
> > So correct it.
>
> Can elaborate the fix a little bit, saying what's the issue exactly with using
> that incorrect compatible?
Ok.
>
> >
> > Signed-off-by: Yuantian Tang <andy.tang@xxxxxxx>
>
> Do we need a fix tag?
What does "fix tag" mean?
BR,
Andy
>
> Shawn
>
> > ---
> > arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > b/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > index 5e14e5a19744..f5da9e8b0d9d 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > @@ -107,7 +107,7 @@
> > status = "okay";
> >
> > i2c-mux@77 {
> > - compatible = "nxp,pca9847";
> > + compatible = "nxp,pca9547";
> > reg = <0x77>;
> > #address-cells = <1>;
> > #size-cells = <0>;
> > --
> > 2.17.1
> >
