On Fri, Aug 13, 2021 at 11:01:55AM +0800, hongbo.wang@xxxxxxx wrote: > From: hongbo wang <hongbo.wang@xxxxxxx> > > some use cases want to use swp4-eno2 link as ordinary data path, > so we can enable swp5 as dsa master, the data from kernel can > be transmitted to eno3, then send to swp5 via internal link, switch will > forward it to swp0-3. > > the data to kernel will come from swp0-3, and received by kernel > via swp5-eno3 link. > new file mode 100644 > index 000000000000..a88396c137a1 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb-dsa-swp5-eno3.dts > @@ -0,0 +1,27 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Device Tree file for NXP LS1028A RDB with dsa master swp5-eno3. > + * > + * Copyright 2018-2021 NXP > + * > + * Hongbo Wang <hongbo.wang@xxxxxxx> > + * > + */ > + > +/dts-v1/; > +#include "fsl-ls1028a-rdb.dts" You will end up with two DT blobs with the same top level compatible. This is going to cause confusion. I suggest you add an additional top level compatible to make it clear this differs from the compatible = "fsl,ls1028a-rdb", "fsl,ls1028a" blob. Andrew