Re: [PATCH v10 4/4] ARM: dts: mxs: Add descriptions for imx287 based btt3-[012] devices

[Lukasz Majewski <lukma@denx.de>]

[-- Attachment #1: Type: text/plain, Size: 2413 bytes --]

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@denx.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 :-).


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@denx.de

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]