Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <3a4eb13d-f764-f32d-02a5-38f62ce2a3f2@linaro.org>
Date:   Thu, 11 Aug 2022 13:50:11 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.12.0
Subject: Re: [PATCH v2 2/3] ARM: dts: imx6ul-kontron: Simplify devicetree
 structure
Content-Language: en-US
To:     Frieder Schrempf <frieder@fris.de>, Arnd Bergmann <arnd@arndb.de>,
        devicetree@vger.kernel.org,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Olof Johansson <olof@lixom.net>,
        Rob Herring <robh+dt@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Shawn Guo <shawnguo@kernel.org>, soc@kernel.org
Cc:     Frieder Schrempf <frieder.schrempf@kontron.de>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Denys Drozdov <denys.drozdov@toradex.com>,
        Dmitry Osipenko <digetx@gmail.com>,
        Fabio Estevam <festevam@gmail.com>,
        Joel Stanley <joel@jms.id.au>,
        Konrad Dybcio <konrad.dybcio@somainline.org>,
        Luca Weiss <luca@z3ntu.xyz>,
        NXP Linux Team <linux-imx@nxp.com>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Thierry Reding <treding@nvidia.com>
References: <20220810081825.19051-1-frieder@fris.de>
 <20220810081825.19051-3-frieder@fris.de>
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <20220810081825.19051-3-frieder@fris.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 10/08/2022 11:18, Frieder Schrempf wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> 
> There is no need in differentiating between the different SoM versions
> that only differ in memory size, etc. It's much cleaner to simplify
> the devicetrees and let them be more generic.
> 
> If necessary the bootloader will update the devicetree dynamically
> to contain the correct size of the memory.
> 
> We also get rid of the N6xxx notation in the compatibles and file
> names, as they are not really used anymore and often result in
> confusion.
> 
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> ---
>  arch/arm/boot/dts/Makefile                    |  4 +-
>  ...6310-s-43.dts => imx6ul-kontron-bl-43.dts} |  8 ++--
>  ...x-s.dtsi => imx6ul-kontron-bl-common.dtsi} |  0
>  ...tron-n6310-s.dts => imx6ul-kontron-bl.dts} |  9 ++--
>  .../boot/dts/imx6ul-kontron-n6310-som.dtsi    | 41 -------------------
>  arch/arm/boot/dts/imx6ul-kontron-n6311-s.dts  | 16 --------
>  .../boot/dts/imx6ul-kontron-n6311-som.dtsi    | 40 ------------------
>  ...mon.dtsi => imx6ul-kontron-sl-common.dtsi} | 15 +++++++
>  arch/arm/boot/dts/imx6ul-kontron-sl.dtsi      | 14 +++++++
>  arch/arm/boot/dts/imx6ull-kontron-bl.dts      | 15 +++++++
>  arch/arm/boot/dts/imx6ull-kontron-n6411-s.dts | 16 --------
>  .../boot/dts/imx6ull-kontron-n6411-som.dtsi   | 40 ------------------
>  arch/arm/boot/dts/imx6ull-kontron-sl.dtsi     | 13 ++++++

Most of these should be renames with small amount of changes. You should
experiment with -B/-M/-C format-patch arguments so the end diff is easy
to read.


Best regards,
Krzysztof