Re: [PATCH 2/2] ARM: dts: imx6ull: add seeed studio NPi dev board

From: Parthiban
Date: Fri Mar 29 2024 - 05:40:38 EST


Dear Shawn,

On 3/29/24 09:30, Shawn Guo wrote:
> On Thu, Feb 29, 2024 at 01:53:37PM +0530, Parthiban Nallathambi wrote:
>> Add support for Seed Stuidos NPi i.MX6ULL SoM equipped with
>> 512MB RAM and 8GB eMMC or 512MB NAND flash. Development
>> board comes with either eMMC or NAND based SoM with peripheral
>> interfaces like 2 x ethernet, 2 x USB, LCD, CSI and more.
>>
>> Signed-off-by: Parthiban Nallathambi <parthiban@xxxxxxxxxxx>
>> ---
>> arch/arm/boot/dts/nxp/imx/Makefile | 2 +
>> .../imx/imx6ull-seeed-npi-dev-board-emmc.dts | 19 +
>> .../imx/imx6ull-seeed-npi-dev-board-nand.dts | 19 +
>> .../nxp/imx/imx6ull-seeed-npi-dev-board.dtsi | 424 ++++++++++++++++++
>> .../boot/dts/nxp/imx/imx6ull-seeed-npi.dtsi | 155 +++++++
>> 5 files changed, 619 insertions(+)
>> create mode 100644 arch/arm/boot/dts/nxp/imx/imx6ull-seeed-npi-dev-board-emmc.dts
>> create mode 100644 arch/arm/boot/dts/nxp/imx/imx6ull-seeed-npi-dev-board-nand.dts
>> create mode 100644 arch/arm/boot/dts/nxp/imx/imx6ull-seeed-npi-dev-board.dtsi
>> create mode 100644 arch/arm/boot/dts/nxp/imx/imx6ull-seeed-npi.dtsi
>
> Could you elaborate a bit the point of splitting imx6ull-seeed-npi.dtsi
> and imx6ull-seeed-npi-dev-board.dtsi? Why cannot they be a single file?
imx6ull-seeed-npi.dtsi is for the npi SoM and -dev-board is for the board which uses the SoM.

Thanks,
Parthiban N


>
> Shawn
>