On Wed, 04 Sep 2024 13:23:21 +1200, Ryan Walklin wrote:
> The WL-355608-A8 is a 3.5" 640x480@60Hz RGB LCD display from an unknown
> OEM used in a number of handheld gaming devices made by Anbernic.
> Previously committed using the OEM serial without a vendor prefix,
> however following subsequent discussion the preference is to use the
> integrating device vendor and name where the OEM is unknown.
>
> There are 4 RG35XX series devices from Anbernic based on an Allwinner
> H700 SoC using this panel, with the -Plus variant introduced first.
> Therefore the -Plus is used as the fallback for the subsequent -H,
> -2024, and -SP devices.
>
> Alter the filename and compatible string to reflect the convention.
>
> Fixes: f08aac40639c ("drm: panel: nv3052c: Add WL-355608-A8 panel")
> Signed-off-by: Ryan Walklin <ryan@xxxxxxxxxxxxx>
>
> --
> Changelog v1..v2:
> - Use known panel serial for compatible name rather than referencing
> integrating devices as per feedback [1] from DT maintainers.
>
> Changelog v2..v3:
> - Use integrating device names rather than panel serial number for
> compatible string as per further maintainer feedback [2].
>
> [1] https://lore.kernel.org/linux-devicetree/8b4519fc-0fba-48fe-bfb4-318818b47a65@xxxxxxxxxxxxxxxx/#t
>
> [2] https://lore.kernel.org/dri-devel/6ab54fb5-8723-457d-b5e6-483f82faf30e@xxxxxxxxxxxxxxxx/
> ---
> ...08-a8.yaml => anbernic,rg35xx-plus-panel.yaml} | 15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
> rename Documentation/devicetree/bindings/display/panel/{wl-355608-a8.yaml => anbernic,rg35xx-plus-panel.yaml} (67%)
>
Acked-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
