linux-kernel - Re: [PATCH v4 0/3] Support non-default LVDS data mapping for simple panel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <c54d19b8-eb5c-9977-b7e4-d2806054b5ae@pengutronix.de>
Date:   Wed, 13 Sep 2023 11:33:15 +0200
From:   Johannes Zink <j.zink@...gutronix.de>
To:     David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Thierry Reding <thierry.reding@...il.com>,
        Neil Armstrong <neil.armstrong@...aro.org>,
        Sam Ravnborg <sam@...nborg.org>
Cc:     devicetree@...r.kernel.org,
        Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>,
        Dan Carpenter <error27@...il.com>,
        kernel test robot <lkp@...el.com>,
        linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        kernel@...gutronix.de, patchwork-jzi@...gutronix.de
Subject: Re: [PATCH v4 0/3] Support non-default LVDS data mapping for simple
 panel

Hi everyone,

gentle ping - is there any feedback you can give me for moving this series forward?

Best regards
Johannes

On 7/28/23 16:16, Johannes Zink wrote:
> Some LVDS panels, such as the innolux,g101ice-l01 support multiple LVDS
> data mapping modes, which can be configured by strapping a dataformat
> pin on the display to a specific voltage.
> 
> This can be particularly useful for using the jeida-18 format, which
> requires only 3 instead of 4 LVDS lanes.
> 
> This series moves the data-mapping property for LVDS panels in a
> separate file and optionally adds it to simple-panel when matching to
> the innolux,g101ice-l01 compatible. This property allows to override
> the default data mapping set in the panel description in simple-panel.
> 
> The last patch in this series actually adds the driver support for
> parsing the data format override device tree property and modifying the
> corresponding values for bit per color and media bus format in the panel
> descriptor.
> 
> Best regards
> Johannes
> 
> ---
> 
> Changelog:
> 
> v3 -> v4:  - driver: worked in Dan's Feedback:
>               - return with proper error in case the call into
> 	       panel_simple_override_nondefault_lvds_datamapping()
> 	       failed
> 	     - drop the unneeded and ambiguous ret local value
> 
> - Link to v3: https://lore.kernel.org/r/20230523-simplepanel_support_nondefault_datamapping-v3-0-78ede374d3d9@pengutronix.de
> v2 -> v3:  - dt bindings: Worked in Conor's and Laurent's Feedback
> 	     (thanks for your review): Drop the chomping indicator
> 	   - dt bindings: Worked in Laurent's Feedback: fix typos
> 	   - driver: worked in Laurent's review findings:
> 	     - extract function for fixing up the bus format
> 	     - only call this function on LVDS panels
> 	     - fix typo
>             - Link to v2: https://lore.kernel.org/r/20230523-simplepanel_support_nondefault_datamapping-v2-0-87196f0d0b64@pengutronix.de
> 
> v1 -> v2:  - dt bindings: Worked in Rob's review findings (thanks for your
>               review), refactored to use common include instead of duplication
>             - driver: added missing error unwinding goto, as found by Dan
>               Carpenter's test robot:
>               Reported-by: kernel test robot <lkp@...el.com>
>               Reported-by: Dan Carpenter <error27@...il.com>
>               Link: https://lore.kernel.org/r/202304160359.4LHmFOlU-lkp@intel.com/
> 
> To: David Airlie <airlied@...il.com>
> To: Daniel Vetter <daniel@...ll.ch>
> To: Rob Herring <robh+dt@...nel.org>
> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
> To: Conor Dooley <conor+dt@...nel.org>
> To: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> To: Thierry Reding <thierry.reding@...il.com>
> To: Neil Armstrong <neil.armstrong@...aro.org>
> To: Sam Ravnborg <sam@...nborg.org>
> Cc: patchwork-jzi@...gutronix.de
> Cc: kernel@...gutronix.de
> Cc: Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>
> Cc: dri-devel@...ts.freedesktop.org
> Cc: devicetree@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> Signed-off-by: Johannes Zink <j.zink@...gutronix.de>
> 
> ---
> 
> ---
> Johannes Zink (3):
>        dt-bindings: display: move LVDS data-mapping definition to separate file
>        dt-bindings: display: simple: support non-default data-mapping
>        drm/panel-simple: allow LVDS format override
> 
>   .../bindings/display/lvds-data-mapping.yaml        | 84 ++++++++++++++++++++++
>   .../devicetree/bindings/display/lvds.yaml          | 77 +++-----------------
>   .../bindings/display/panel/panel-simple.yaml       | 26 ++++++-
>   drivers/gpu/drm/panel/panel-simple.c               | 53 ++++++++++++++
>   4 files changed, 171 insertions(+), 69 deletions(-)
> ---
> base-commit: 52920704df878050123dfeb469aa6ab8022547c1
> change-id: 20230523-simplepanel_support_nondefault_datamapping-13c3f2ea28f8
> 
> Best regards,

-- 
Pengutronix e.K.                | Johannes Zink                  |
Steuerwalder Str. 21            | https://www.pengutronix.de/    |
31137 Hildesheim, Germany       | Phone: +49-5121-206917-0       |
Amtsgericht Hildesheim, HRA 2686| Fax:   +49-5121-206917-5555    |