From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: dri-devel@lists.freedesktop.org
Cc: devicetree@vger.kernel.org, "Marek Vasut" <marex@denx.de>,
"Stefan Agner" <stefan@agner.ch>,
"Rob Herring" <robh+dt@kernel.org>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
linux-arm-kernel@lists.infradead.org,
"Guido Günther" <agx@sigxcpu.org>
Subject: [PATCH 0/8] drm: mxsfb: Allow overriding bus width
Date: Thu, 13 Aug 2020 04:29:02 +0300 [thread overview]
Message-ID: <20200813012910.13576-1-laurent.pinchart@ideasonboard.com> (raw)
Hello,
This patch series adds support to the mxsfb driver for bus width
override. The need came from a hardware platform where a 18-bpp panel
had the R[5:0], G[5:0] and B[5:0] signals connected to LCD_DATA[7:2],
LCD_DATA[15:10] and LCD_DATA[23:18] instead of LCD_DATA[5:0],
LCD_DATA[11:6] and LCD_DATA[17:12]. The bus width, automatically
configured to 18 by querying the panel, is incorrect in this case, and
needs to be set to 24.
To solve this issue, a new bus-width DT property is added to the mxsfb
DT binding. Patch 1/8 first converts the binding to YAML, with a fix for
the compatible string values in 2/8. Patch 3/8 then adds the new
property, and 4/8 renames the binding file to fsl,lcdif.yaml to match
the usual naming convention. I've kept that patch last to make it easy
to drop should should mxsfb.yaml be preferred.
Patches 5/8 to 6/8 then fix the DT sources to match the LCDIF bindings,
as I noticed during the conversion that the compatible strings were
badly managed (see patch 2/8 for a longer explanation). Patch 7/8 drops
an unused clock from DT sources.
Patch 8/8 finally adds support for the bus-width property to the mxsfb
driver.
Laurent Pinchart (8):
dt-bindings: display: mxsfb: Convert binding to YAML
dt-bindings: display: mxsfb: Add and fix compatible strings
dt-bindings: display: mxsfb: Add a bus-width endpoint property
dt-bindings: display: mxsfb: Rename to fsl,lcdif.yaml
ARM: dts: imx: Fix LCDIF compatible strings
arm64: dts: imx8mq: Fix LCDIF compatible strings
ARM: dts: imx: Remove unneeded LCDIF disp_axi clock
drm: mxsfb: Add support for the bus-width DT property
.../bindings/display/fsl,lcdif.yaml | 135 ++++++++++++++++++
.../devicetree/bindings/display/mxsfb.txt | 87 -----------
MAINTAINERS | 2 +-
arch/arm/boot/dts/imx6sl.dtsi | 7 +-
arch/arm/boot/dts/imx6sll.dtsi | 7 +-
arch/arm/boot/dts/imx6sx.dtsi | 4 +-
arch/arm/boot/dts/imx6ul.dtsi | 7 +-
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +-
drivers/gpu/drm/mxsfb/mxsfb_drv.c | 26 ++++
drivers/gpu/drm/mxsfb/mxsfb_drv.h | 2 +
drivers/gpu/drm/mxsfb/mxsfb_kms.c | 8 +-
11 files changed, 182 insertions(+), 105 deletions(-)
create mode 100644 Documentation/devicetree/bindings/display/fsl,lcdif.yaml
delete mode 100644 Documentation/devicetree/bindings/display/mxsfb.txt
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: dri-devel@lists.freedesktop.org
Cc: "Marek Vasut" <marex@denx.de>,
devicetree@vger.kernel.org, "Guido Günther" <agx@sigxcpu.org>,
"Stefan Agner" <stefan@agner.ch>,
"Rob Herring" <robh+dt@kernel.org>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/8] drm: mxsfb: Allow overriding bus width
Date: Thu, 13 Aug 2020 04:29:02 +0300 [thread overview]
Message-ID: <20200813012910.13576-1-laurent.pinchart@ideasonboard.com> (raw)
Hello,
This patch series adds support to the mxsfb driver for bus width
override. The need came from a hardware platform where a 18-bpp panel
had the R[5:0], G[5:0] and B[5:0] signals connected to LCD_DATA[7:2],
LCD_DATA[15:10] and LCD_DATA[23:18] instead of LCD_DATA[5:0],
LCD_DATA[11:6] and LCD_DATA[17:12]. The bus width, automatically
configured to 18 by querying the panel, is incorrect in this case, and
needs to be set to 24.
To solve this issue, a new bus-width DT property is added to the mxsfb
DT binding. Patch 1/8 first converts the binding to YAML, with a fix for
the compatible string values in 2/8. Patch 3/8 then adds the new
property, and 4/8 renames the binding file to fsl,lcdif.yaml to match
the usual naming convention. I've kept that patch last to make it easy
to drop should should mxsfb.yaml be preferred.
Patches 5/8 to 6/8 then fix the DT sources to match the LCDIF bindings,
as I noticed during the conversion that the compatible strings were
badly managed (see patch 2/8 for a longer explanation). Patch 7/8 drops
an unused clock from DT sources.
Patch 8/8 finally adds support for the bus-width property to the mxsfb
driver.
Laurent Pinchart (8):
dt-bindings: display: mxsfb: Convert binding to YAML
dt-bindings: display: mxsfb: Add and fix compatible strings
dt-bindings: display: mxsfb: Add a bus-width endpoint property
dt-bindings: display: mxsfb: Rename to fsl,lcdif.yaml
ARM: dts: imx: Fix LCDIF compatible strings
arm64: dts: imx8mq: Fix LCDIF compatible strings
ARM: dts: imx: Remove unneeded LCDIF disp_axi clock
drm: mxsfb: Add support for the bus-width DT property
.../bindings/display/fsl,lcdif.yaml | 135 ++++++++++++++++++
.../devicetree/bindings/display/mxsfb.txt | 87 -----------
MAINTAINERS | 2 +-
arch/arm/boot/dts/imx6sl.dtsi | 7 +-
arch/arm/boot/dts/imx6sll.dtsi | 7 +-
arch/arm/boot/dts/imx6sx.dtsi | 4 +-
arch/arm/boot/dts/imx6ul.dtsi | 7 +-
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +-
drivers/gpu/drm/mxsfb/mxsfb_drv.c | 26 ++++
drivers/gpu/drm/mxsfb/mxsfb_drv.h | 2 +
drivers/gpu/drm/mxsfb/mxsfb_kms.c | 8 +-
11 files changed, 182 insertions(+), 105 deletions(-)
create mode 100644 Documentation/devicetree/bindings/display/fsl,lcdif.yaml
delete mode 100644 Documentation/devicetree/bindings/display/mxsfb.txt
--
Regards,
Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: dri-devel@lists.freedesktop.org
Cc: "Marek Vasut" <marex@denx.de>,
devicetree@vger.kernel.org, "Guido Günther" <agx@sigxcpu.org>,
"Rob Herring" <robh+dt@kernel.org>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/8] drm: mxsfb: Allow overriding bus width
Date: Thu, 13 Aug 2020 04:29:02 +0300 [thread overview]
Message-ID: <20200813012910.13576-1-laurent.pinchart@ideasonboard.com> (raw)
Hello,
This patch series adds support to the mxsfb driver for bus width
override. The need came from a hardware platform where a 18-bpp panel
had the R[5:0], G[5:0] and B[5:0] signals connected to LCD_DATA[7:2],
LCD_DATA[15:10] and LCD_DATA[23:18] instead of LCD_DATA[5:0],
LCD_DATA[11:6] and LCD_DATA[17:12]. The bus width, automatically
configured to 18 by querying the panel, is incorrect in this case, and
needs to be set to 24.
To solve this issue, a new bus-width DT property is added to the mxsfb
DT binding. Patch 1/8 first converts the binding to YAML, with a fix for
the compatible string values in 2/8. Patch 3/8 then adds the new
property, and 4/8 renames the binding file to fsl,lcdif.yaml to match
the usual naming convention. I've kept that patch last to make it easy
to drop should should mxsfb.yaml be preferred.
Patches 5/8 to 6/8 then fix the DT sources to match the LCDIF bindings,
as I noticed during the conversion that the compatible strings were
badly managed (see patch 2/8 for a longer explanation). Patch 7/8 drops
an unused clock from DT sources.
Patch 8/8 finally adds support for the bus-width property to the mxsfb
driver.
Laurent Pinchart (8):
dt-bindings: display: mxsfb: Convert binding to YAML
dt-bindings: display: mxsfb: Add and fix compatible strings
dt-bindings: display: mxsfb: Add a bus-width endpoint property
dt-bindings: display: mxsfb: Rename to fsl,lcdif.yaml
ARM: dts: imx: Fix LCDIF compatible strings
arm64: dts: imx8mq: Fix LCDIF compatible strings
ARM: dts: imx: Remove unneeded LCDIF disp_axi clock
drm: mxsfb: Add support for the bus-width DT property
.../bindings/display/fsl,lcdif.yaml | 135 ++++++++++++++++++
.../devicetree/bindings/display/mxsfb.txt | 87 -----------
MAINTAINERS | 2 +-
arch/arm/boot/dts/imx6sl.dtsi | 7 +-
arch/arm/boot/dts/imx6sll.dtsi | 7 +-
arch/arm/boot/dts/imx6sx.dtsi | 4 +-
arch/arm/boot/dts/imx6ul.dtsi | 7 +-
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +-
drivers/gpu/drm/mxsfb/mxsfb_drv.c | 26 ++++
drivers/gpu/drm/mxsfb/mxsfb_drv.h | 2 +
drivers/gpu/drm/mxsfb/mxsfb_kms.c | 8 +-
11 files changed, 182 insertions(+), 105 deletions(-)
create mode 100644 Documentation/devicetree/bindings/display/fsl,lcdif.yaml
delete mode 100644 Documentation/devicetree/bindings/display/mxsfb.txt
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next reply other threads:[~2020-08-13 1:29 UTC|newest]
Thread overview: 93+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-13 1:29 Laurent Pinchart [this message]
2020-08-13 1:29 ` [PATCH 0/8] drm: mxsfb: Allow overriding bus width Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` [PATCH 1/8] dt-bindings: display: mxsfb: Convert binding to YAML Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 6:22 ` Sam Ravnborg
2020-08-16 6:22 ` Sam Ravnborg
2020-08-16 6:22 ` Sam Ravnborg
2020-08-17 0:00 ` Laurent Pinchart
2020-08-17 0:00 ` Laurent Pinchart
2020-08-17 0:00 ` Laurent Pinchart
2020-08-24 23:59 ` Rob Herring
2020-08-24 23:59 ` Rob Herring
2020-08-24 23:59 ` Rob Herring
2020-08-13 1:29 ` [PATCH 2/8] dt-bindings: display: mxsfb: Add and fix compatible strings Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 6:39 ` Sam Ravnborg
2020-08-16 6:39 ` Sam Ravnborg
2020-08-16 6:39 ` Sam Ravnborg
2020-08-17 0:04 ` Laurent Pinchart
2020-08-17 0:04 ` Laurent Pinchart
2020-08-17 0:04 ` Laurent Pinchart
2020-08-24 23:57 ` Rob Herring
2020-08-24 23:57 ` Rob Herring
2020-08-24 23:57 ` Rob Herring
2020-08-21 14:53 ` Stefan Agner
2020-08-21 14:53 ` Stefan Agner
2020-08-21 14:53 ` Stefan Agner
2020-08-23 23:26 ` Laurent Pinchart
2020-08-23 23:26 ` Laurent Pinchart
2020-08-23 23:26 ` Laurent Pinchart
2020-08-24 14:19 ` Stefan Agner
2020-08-24 14:19 ` Stefan Agner
2020-08-24 14:19 ` Stefan Agner
2020-10-07 1:12 ` Laurent Pinchart
2020-10-07 1:12 ` Laurent Pinchart
2020-10-07 1:12 ` Laurent Pinchart
2020-08-13 1:29 ` [PATCH 3/8] dt-bindings: display: mxsfb: Add a bus-width endpoint property Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-15 21:28 ` Guido Günther
2020-08-15 21:28 ` Guido Günther
2020-08-15 21:28 ` Guido Günther
2020-08-17 0:09 ` Laurent Pinchart
2020-08-17 0:09 ` Laurent Pinchart
2020-08-17 0:09 ` Laurent Pinchart
2020-08-16 7:25 ` Sam Ravnborg
2020-08-16 7:25 ` Sam Ravnborg
2020-08-16 7:25 ` Sam Ravnborg
2020-08-17 0:17 ` Laurent Pinchart
2020-08-17 0:17 ` Laurent Pinchart
2020-08-17 0:17 ` Laurent Pinchart
2020-08-13 1:29 ` [PATCH 4/8] dt-bindings: display: mxsfb: Rename to fsl,lcdif.yaml Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 7:27 ` Sam Ravnborg
2020-08-16 7:27 ` [PATCH 4/8] dt-bindings: display: mxsfb: Rename to fsl, lcdif.yaml Sam Ravnborg
2020-08-16 7:27 ` Sam Ravnborg
2020-08-21 14:55 ` [PATCH 4/8] dt-bindings: display: mxsfb: Rename to fsl,lcdif.yaml Stefan Agner
2020-08-21 14:55 ` [PATCH 4/8] dt-bindings: display: mxsfb: Rename to fsl, lcdif.yaml Stefan Agner
2020-08-21 14:55 ` Stefan Agner
2020-08-23 23:27 ` [PATCH 4/8] dt-bindings: display: mxsfb: Rename to fsl,lcdif.yaml Laurent Pinchart
2020-08-23 23:27 ` [PATCH 4/8] dt-bindings: display: mxsfb: Rename to fsl, lcdif.yaml Laurent Pinchart
2020-08-23 23:27 ` Laurent Pinchart
2020-08-13 1:29 ` [PATCH 5/8] ARM: dts: imx: Fix LCDIF compatible strings Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 7:28 ` Sam Ravnborg
2020-08-16 7:28 ` Sam Ravnborg
2020-08-16 7:28 ` Sam Ravnborg
2020-08-13 1:29 ` [PATCH 6/8] arm64: dts: imx8mq: " Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 7:28 ` Sam Ravnborg
2020-08-16 7:28 ` Sam Ravnborg
2020-08-16 7:28 ` Sam Ravnborg
2020-08-13 1:29 ` [PATCH 7/8] ARM: dts: imx: Remove unneeded LCDIF disp_axi clock Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 7:28 ` Sam Ravnborg
2020-08-16 7:28 ` Sam Ravnborg
2020-08-16 7:28 ` Sam Ravnborg
2020-08-13 1:29 ` [PATCH 8/8] drm: mxsfb: Add support for the bus-width DT property Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-13 1:29 ` Laurent Pinchart
2020-08-16 7:46 ` Sam Ravnborg
2020-08-16 7:46 ` Sam Ravnborg
2020-08-16 7:46 ` Sam Ravnborg
2020-08-17 0:29 ` Laurent Pinchart
2020-08-17 0:29 ` Laurent Pinchart
2020-08-17 0:29 ` Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200813012910.13576-1-laurent.pinchart@ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=agx@sigxcpu.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=marex@denx.de \
--cc=robh+dt@kernel.org \
--cc=stefan@agner.ch \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.