Re: [PATCH 0/2] arm64: dts: (imx8mm|imx8mp)-verdin-dev: Do not include dahlia dtsi

From: Shawn Guo
Date: Thu Jan 26 2023 - 03:06:30 EST


On Wed, Jan 18, 2023 at 11:52:49AM +0100, Philippe Schenker wrote:
> From: Philippe Schenker <philippe.schenker@xxxxxxxxxxx>
>
>
> We so far included the Dahlia device-tree in the development board
> devicetree in order to try to minimize code duplication.
>
> This mainly gets rid of ''status = "okay"' lines. Plus we saw that some
> settings in the audio-codec of verdin-imx8mm are inherited that are
> wrong for the development board. Those did also lead to the DT Schema
> checker to fail on multiple settings.
>
> We decided to remove this include in order to get the audio-codec
> settings correct and make the DT Schema checker return less warnings.
> This introduces some code duplication which we consider being mostly
> only configuration of the actual development/dahlia boards.
>
> This series of two patches removes the include for both Verdin boards we
> have at the moment.
>
> This issue was spotted by Rob Herring. [1]
>
> [1] https://lore.kernel.org/all/167182349408.373865.10339766752503211557.robh@xxxxxxxxxx/
>
>
> Philippe Schenker (2):
> arm64: dts: imx8mm-verdin-dev: Do not include dahlia dtsi
> arm64: dts: imx8mp-verdin-dev: Do not include dahlia dtsi

Applied both, thanks!