[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0cb1e74f-1163-4a80-8c62-9c7b0a0584f9@gmx.net>
Date: Thu, 31 Oct 2024 11:57:22 +0100
From: Stefan Wahren <wahrenst@....net>
To: Lukasz Majewski <lukma@...x.de>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>, devicetree@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v10 4/4] ARM: dts: mxs: Add descriptions for imx287 based
btt3-[012] devices
Hi Lukasz,
Am 31.10.24 um 11:34 schrieb Lukasz Majewski:
> Hi Stefan,
>
>> Am 31.10.24 um 09:42 schrieb Lukasz Majewski:
>>> Dear Community,
>>>
>>>> The btt3 device' HW revisions from 0 to 2 use imx287 SoC and are to
>>>> some extend similar to already upstreamed XEA devices, hence are
>>>> using common imx28-lwe.dtsi file.
>>>>
>>>> New, imx28-btt3.dtsi has been added to embrace common DTS
>>>> properties for different HW revisions for this device.
>>>>
>>>> As a result - changes introduced in imx28-btt3-[012].dts are
>>>> minimal.
>>>>
>>>> Signed-off-by: Lukasz Majewski <lukma@...x.de>
>>>>
>>>> ---
>>>> Changes for v2:
>>>> - Rename dts file from btt3-[012] to imx28-btt3-[012] to match
>>>> current linux kernel naming convention
>>>> - Remove 'wlf,wm8974' from compatible for codec@1a
>>>>
>>>> Changes for v3:
>>>> - Keep alphabethical order for Makefile entries
>>>>
>>>> Changes for v4:
>>>> - Change compatible for btt3 board (to 'lwn,imx28-btt3')
>>>>
>>>> Changes for v5:
>>>> - Combine patch, which adds btt3-[012] with one adding board entry
>>>> to fsl.yaml
>>>>
>>>> Changes for v6:
>>>> - Make the patch series for adding entry in fsl.yaml and btt3
>>>>
>>>> Changes for v7:
>>>> - Use "panel" property as suggested by the community
>>>> - Use panel-timing to specify the display parameters
>>>> - Update subject line with correct tags
>>>>
>>>> Changes for v8:
>>>> - Use GPIO_ACTIVE_HIGH instead of '0'
>>>> - Add the comment regarding mac address specification
>>>> - Remove superfluous comment
>>>> - Change wifi-en-pin node name
>>>>
>>>> Changes for v9:
>>>> - Remove not used 'pm-ignore-notify'
>>>> - Add display names for 'panel-dpi' compatible to avoid Schema
>>>> warnings
>>>>
>>>> Changes for v10:
>>>> - Drop new line with panel-timing definitions
>>>> - Add new lines with 'sound' node
>>>> - Change 'codec' to 'audio-codec'
>>>> - Change order of properties for saif1 node
>>> Are there any more comments regarding this patch?
>> Sorry, i wasn't aware that you are waiting for reviews after Rob's
>> request.
> I did not received any mail from Rob's automated DTS test script, so
> this is a good thing :-).
I meant his comment to patch 2 V10. This patch "depends" on the binding.
>
>
> Best regards,
>
> Lukasz Majewski
>
> --
>
> DENX Software Engineering GmbH, Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@...x.de
Powered by blists - more mailing lists