[PATCH v5 3/3] drm/panel-simple: allow LVDS format override

2023-09-29 Thread Johannes Zink
: Johannes Zink --- Changes: v4 -> v5: none v3 -> v4: - worked in Dan's feedback (thanks for reviewing my work): - return with a proper error in case the call to panel_simple_override_nondefault_lvds_datamapping() fails

[PATCH v5 2/3] dt-bindings: display: simple: support non-default data-mapping

2023-09-29 Thread Johannes Zink
bus, use it selectively on the innolux,g101ice-l01, which supports changing the data mapping via a strapping pin. Reviewed-by: Conor Dooley Signed-off-by: Johannes Zink --- Changes: v4 -> v5: none, re-added lost reviewed-by tag v3 -> v4: none v2 -> v3: - worked in Laurent's r

[PATCH v5 1/3] dt-bindings: display: move LVDS data-mapping definition to separate file

2023-09-29 Thread Johannes Zink
As the LVDS data-mapping property is required in multiple bindings: move it to separate file and include instead of duplicating it. Reviewed-by: Conor Dooley Reviewed-by: Laurent Pinchart Signed-off-by: Johannes Zink --- Changes: v4 -> v5: none, but are-dded the reviewed-bys from v2, t

[PATCH v5 0/3] Support non-default LVDS data mapping for simple panel

2023-09-29 Thread Johannes Zink
tronix.de Cc: ker...@pengutronix.de Cc: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org Cc: devicet...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Johannes Zink --- --- Johannes Zink (3): dt-bindings: display: move LVDS data-mapping definition to separate file dt

Re: [PATCH v4 3/3] drm/panel-simple: allow LVDS format override

2023-09-13 Thread Johannes Zink
Hi Dan, On 9/13/23 13:14, Dan Carpenter wrote: On Fri, Aug 18, 2023 at 09:04:34AM +0200, Johannes Zink wrote: Hi Dan, do you have any input on this for me? Sorry, I was out of office and the truth is that I'm never going to catch up on all the email I missed. :/ nevermind, that&#x

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

2023-09-13 Thread Johannes Zink
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

Re: [PATCH v4 3/3] drm/panel-simple: allow LVDS format override

2023-08-18 Thread Johannes Zink
Hi Dan, do you have any input on this for me? Best regards Johannes On 7/28/23 16:16, Johannes Zink wrote: Some panels support multiple LVDS data mapping formats, which can be used e.g. run displays on jeida-18 format when only 3 LVDS lanes are available. Add parsing of an optional data

Re: [PATCH v4 2/3] dt-bindings: display: simple: support non-default data-mapping

2023-07-30 Thread Johannes Zink
Hi Conor, On 7/29/23 12:19, Conor Dooley wrote: On Fri, Jul 28, 2023 at 04:16:56PM +0200, Johannes Zink wrote: Some Displays support more than just a single default LVDS data mapping, which can be used to run displays on only 3 LVDS lanes in the jeida-18 data-mapping mode. Add an optional

[PATCH v4 1/3] dt-bindings: display: move LVDS data-mapping definition to separate file

2023-07-28 Thread Johannes Zink
As the LVDS data-mapping property is required in multiple bindings: move it to separate file and include instead of duplicating it. Signed-off-by: Johannes Zink --- Changes: v3 -> v4: none v2 -> v3: worked in Conor's and Laurent's review findings (thank you for r

[PATCH v4 3/3] drm/panel-simple: allow LVDS format override

2023-07-28 Thread Johannes Zink
: Johannes Zink --- Changes: v3 -> v4: - worked in Dan's feedback (thanks for reviewing my work): - return with a proper error in case the call to panel_simple_override_nondefault_lvds_datamapping() fails - drop the unne

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

2023-07-28 Thread Johannes Zink
l Armstrong To: Sam Ravnborg Cc: patchwork-...@pengutronix.de Cc: ker...@pengutronix.de Cc: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org Cc: devicet...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Johannes Zink --- --- Johannes Zink (3): dt-bindings: display: move LVDS data-

[PATCH v4 2/3] dt-bindings: display: simple: support non-default data-mapping

2023-07-28 Thread Johannes Zink
bus, use it selectively on the innolux,g101ice-l01, which supports changing the data mapping via a strapping pin. Signed-off-by: Johannes Zink --- Changes: v3 -> v4: none v2 -> v3: - worked in Laurent's review findings (thanks for reviewing my work): fix typos in commit

Re: [PATCH v3 3/3] drm/panel-simple: allow LVDS format override

2023-07-28 Thread Johannes Zink
Hi Dan, thanks for your feedback. On 7/28/23 14:03, Dan Carpenter wrote: On Fri, Jul 28, 2023 at 01:54:40PM +0200, Johannes Zink wrote: @@ -556,7 +602,7 @@ static int panel_simple_probe(struct device *dev, const struct panel_desc *desc) struct device_node *ddc; int

[PATCH v3 0/3] Support non-default LVDS data mapping for simple panel

2023-07-28 Thread Johannes Zink
Cc: dri-devel@lists.freedesktop.org Cc: devicet...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Johannes Zink --- Johannes Zink (3): dt-bindings: display: move LVDS data-mapping definition to separate file dt-bindings: display: simple: support non-default data-m

[PATCH v3 1/3] dt-bindings: display: move LVDS data-mapping definition to separate file

2023-07-28 Thread Johannes Zink
As the LVDS data-mapping property is required in multiple bindings: move it to separate file and include instead of duplicating it. Signed-off-by: Johannes Zink --- Changes: v2 -> v3: worked in Conor's and Laurent's review findings (thank you for reviewing my work

[PATCH v3 2/3] dt-bindings: display: simple: support non-default data-mapping

2023-07-28 Thread Johannes Zink
bus, use it selectively on the innolux,g101ice-l01, which supports changing the data mapping via a strapping pin. Signed-off-by: Johannes Zink --- Changes: v2 -> v3: - worked in Laurent's review findings (thanks for reviewing my work): fix typos in commit message v1 -> v

[PATCH v3 3/3] drm/panel-simple: allow LVDS format override

2023-07-28 Thread Johannes Zink
: Johannes Zink --- Changes: v2 -> v3: - worked in Laurent's review findings (thanks for reviewing my work): - extract fixing up the bus format to separate function - only call function on LVDS panels - fix typos

Re: [PATCH v2 2/3] dt-bindings: display: simple: support non-default data-mapping

2023-07-28 Thread Johannes Zink
Hi Laurent, thank you for your review. On 6/2/23 17:35, Laurent Pinchart wrote: Hi Johannes, Thank you for the patch. On Tue, May 23, 2023 at 10:19:42AM +0200, Johannes Zink wrote: Some Displays support more than just a single default lvds data mapping, s/lvds/LVDS/ ack, gonna fix in V3

Re: [PATCH v2 1/3] dt-bindings: display: move LVDS data-mapping definition to separate file

2023-07-28 Thread Johannes Zink
Hi Laurent, thank you for your review. On 6/2/23 17:32, Laurent Pinchart wrote: Hello Johannes, Thank you for the patch. On Tue, May 23, 2023 at 10:19:41AM +0200, Johannes Zink wrote: As the LVDS data-mapping property is required in multiple bindings: move it to separate file and include

Re: [PATCH v2 3/3] drm/panel-simple: allow LVDS format override

2023-06-04 Thread Johannes Zink
Hi Laurent, thanks for your review and your feedback. On 6/2/23 17:39, Laurent Pinchart wrote: Hi Johannes, Thank you for the patch. On Tue, May 23, 2023 at 10:19:43AM +0200, Johannes Zink wrote: Some panels support multiple LVDS data mapping formats, which can be used e.g. run displays on

Re: [PATCH v2 3/3] drm/panel-simple: allow LVDS format override

2023-06-02 Thread Johannes Zink
Hi, gentle ping here - Do you have any feedback for me on this patch? Thanks and best regards Johannes On 5/23/23 10:19, Johannes Zink wrote: Some panels support multiple LVDS data mapping formats, which can be used e.g. run displays on jeida-18 format when only 3 LVDS lanes are available

[PATCH v2 2/3] dt-bindings: display: simple: support non-default data-mapping

2023-05-23 Thread Johannes Zink
bus: use it selectively on the innolux,g101ice-l01, which supports changing the data mapping via a strapping pin. Signed-off-by: Johannes Zink --- Changes: v1 -> v2: - worked in Rob's review findings (thanks for reviewing my work): use extracted common property in

[PATCH v2 3/3] drm/panel-simple: allow LVDS format override

2023-05-23 Thread Johannes Zink
: Johannes Zink --- Changes: v1 -> v2: - fix missing unwind goto found by test robot Reported-by: kernel test robot Reported-by: Dan Carpenter Link: https://lore.kernel.org/r/202304160359.4lhmfolu-...@intel.com/ --- drivers/gpu/drm/panel/panel-simpl

[PATCH v2 0/3] Support non-default LVDS data mapping for simple panel

2023-05-23 Thread Johannes Zink
ix.de Cc: ker...@pengutronix.de Cc: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org Cc: devicet...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Johannes Zink --- Johannes Zink (3): dt-bindings: display: move LVDS data-mapping definition to separate file dt-bindings: di

[PATCH v2 1/3] dt-bindings: display: move LVDS data-mapping definition to separate file

2023-05-23 Thread Johannes Zink
As the LVDS data-mapping property is required in multiple bindings: move it to separate file and include instead of duplicating it. Signed-off-by: Johannes Zink --- Changes: v1 -> v2: worked in Rob's review findings (thank you for reviewing my work): extract common prope

Re: [PATCH 1/2] dt-bindings: display: simple: support non-default data-mapping

2023-05-04 Thread Johannes Zink
Hi Rob, On 4/19/23 08:29, Johannes Zink wrote: [snip] Why do you duplicate what's in display/lvds.yaml? I don't think any of the other properties from display/lvds.yaml currently make any sense to be put into simple-panel, thus I selectively picked this one. If there i

Re: [PATCH 1/2] dt-bindings: display: simple: support non-default data-mapping

2023-04-20 Thread Johannes Zink
Hi Rob, On 4/18/23 23:20, Rob Herring wrote: On Fri, Apr 14, 2023 at 06:11:15PM +0200, Johannes Zink wrote: Some Displays support more than just a single default lvds data mapping, which can be used to run displays on only 3 LVDS lanes in the jeida-18 data-mapping mode. Add an optional data

[PATCH 0/2] Support non-default LVDS data mapping for simple

2023-04-17 Thread Johannes Zink
. This series adds an optional data-mapping property, analogously to the property on lvds-panel, which overrides the default data mapping set in the panel description in simple-panel. Best regards Johannes Johannes Zink (2): dt-bindings: display: simple: support non-default data-mapping drm/panel

[PATCH 2/2] drm/panel-simple: allow LVDS format override

2023-04-17 Thread Johannes Zink
: Johannes Zink --- drivers/gpu/drm/panel/panel-simple.c | 37 +++- 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index 065f378bba9d..7366ad13b3f1 100644 --- a/drivers/gpu/drm/panel/panel

[PATCH 1/2] dt-bindings: display: simple: support non-default data-mapping

2023-04-17 Thread Johannes Zink
: Johannes Zink --- .../bindings/display/panel/panel-simple.yaml | 51 +++ 1 file changed, 51 insertions(+) diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml index 18241f4051d2..6e219f67dd67