MIPS: JZ4780 and CI20 HDMI
Subsystems (5)
v18 cover MIPS: JZ4780 and CI20 HDMI
PATCH V18 2022-04-07 13:16:10:
* removed CSC filter patches (3-5/6) by Neil Armstrong <[email protected]> as they no longer seem to be required
* added "Reviewed-by: Laurent Pinchart <[email protected]>" to ddc-en patch (6/6)
PATCH V17 2022-04-06 18:26:08:
* removed our local attempts to enable HPD because "drm/bridge_connector: enable HPD by default if supported"
became available and solves the issue globally (by [email protected])
* add new mechanism to disable Color Space Conversion (by [email protected])
* rebased to v5.18-rc1 based drm-misc/drm-misc-next
PATCH V16 2022-02-26 18:13:02:
* fixed and renamed dw-hdmi bus negotiation patch (by [email protected])
* reordered and merged HPD fix (suggested by [email protected])
* fixed MODULE_ALIAS for dw-hdmi-ingenic (reported by [email protected])
* dropped some already merged commits from the series
PATCH V15 2022-02-12 16:50:54:
* remove already (elsewhere) merged commits (suggested by [email protected])
* clarify commit message for (now) 1/7 ((suggested by [email protected]))
PATCH V14 2022-02-12 15:19:25:
* make compatible to c03d0b52ff71d5 ("drm/connector: Fix typo in output format")
* move "dw-hdmi/ingenic-dw-hdmi: repair interworking with hdmi-connector" before drm/ingenic: Add dw-hdmi driver specialization for jz4780 (by [email protected])
* split introduction of dw_hdmi_enable_poll() into separate patch
* explicitly mark plane f0 as not working in jz4780 (suggested by [email protected])
* drop 1/9 since it is now in drm-misc/drm-misc-next
PATCH V13 2022-02-02 17:31:22:
* 7/9: remove call to gpiod_set_value() because of GPIOD_OUT_HIGH (by [email protected])
* 4/9: replace ".." by "." (by [email protected])
* 3/9: remove old hdmi-5v-power in the example (by [email protected])
* 2/9: disable handling of plane f0 only for jz4780 (by [email protected])
PATCH V12 2022-01-31 13:26:54:
This version reworks how hdmi ddc power is controlled by connector and not by ddc/hdmi bridge driver.
Also some patches of the previous version of this series have been removed since they are already applied to mips-next/linux/next/v5.17-rc1.
Fixes and changes:
- repair interworking of dw-hdmi with connector-hdmi (by [email protected])
- fix JZ_REG_LCD_OSDC setup for jz4780 (by [email protected] and [email protected])
- adjustments for ci20.dts to use connector gpio for +5v (suggested by several)
- to add control of "ddc-en-gpios" to hdmi-connector driver (by [email protected])
- regulator code removed because we now use the "ddc-en-gpios" of the connector driver (suggested by [email protected])
- bindings: addition of "ddc-i2c-bus" and "hdmi-5v-supply" removed (suggested by [email protected])
- rebase on v5.17-rc2
PATCH V11 2021-12-02 19:39:52:
- patch 4/8: change devm_regulator_get_optional to devm_regulator_get and remove NULL check (requested by [email protected])
- patch 3/8: make hdmi-5v-supply required (requested by [email protected])
PATCH V10 2021-11-30 22:26:41:
- patch 3/8: fix $id and $ref paths (found by [email protected])
PATCH V9 2021-11-24 22:29:14:
- patch 6/8: remove optional <0> for assigned-clocks and unintentionally included "unwedge" setup (found by [email protected])
- patch 4/8: some cosmetics
make regulator enable/disable only if not NULL (found by [email protected])
simplify/fix error handling and driver cleanup on remove (proposed by [email protected])
- patch 3/8: fix #include path in example (found by [email protected])
fix missing "i" in unevaluatedProperties (found by [email protected])
fix 4 spaces indentation for required: property (found by [email protected])
PATCH V8 2021-11-23 19:14:00:
- fix a bad editing result from patch 2/8 (found by [email protected])
PATCH V7 2021-11-23 18:46:23:
- changed gpio polarity of hdmi_power to 0 (suggested by [email protected])
- fixed LCD1 irq number (bug found by [email protected])
- removed "- 4" for calculating max_register (suggested by [email protected])
- use unevaluatedPropertes instead of additionalProperties (suggested by [email protected])
- moved and renamed ingenic,jz4780-hdmi.yaml (suggested by [email protected])
- adjusted assigned-clocks changes to upstream which added some for SSI (by [email protected])
- rebased and tested with v5.16-rc2 + patch set drm/ingenic by [email protected] (by [email protected])
PATCH V6 2021-11-10 20:43:33:
- changed CONFIG_DRM_INGENIC_DW_HDMI to "m" (by [email protected])
- made ingenic-dw-hdmi an independent platform driver which can be compiled as module and removed error patch fixes for IPU (suggested by [email protected])
- moved assigned-clocks from jz4780.dtsi to ci20.dts (suggested by [email protected])
- fixed reg property in jz4780.dtsi to cover all registers incl. gamma and vee (by [email protected])
- added a base patch to calculate regmap size from DTS reg property (requested by [email protected])
- restored resetting all bits except one in LCDOSDC (requested by [email protected])
- clarified setting of cpos (suggested by [email protected])
- moved bindings definition for ddc-i2c-bus (suggested by [email protected])
- simplified mask definitions for JZ_LCD_DESSIZE (requested by [email protected])
- removed setting alpha premultiplication (suggested by [email protected])
- removed some comments (suggested by [email protected])
PATCH V5 2021-10-05 14:28:44:
- dropped mode_fixup and timings support in dw-hdmi as it is no longer needed in this V5 (by [email protected])
- dropped "drm/ingenic: add some jz4780 specific features" (stimulated by [email protected])
- fixed typo in commit subject: "synopsis" -> "synopsys" (by [email protected])
- swapped clocks in jz4780.dtsi to match synopsys,dw-hdmi.yaml (by [email protected])
- improved, simplified, fixed, dtbschecked ingenic-jz4780-hdmi.yaml and made dependent of bridge/synopsys,dw-hdmi.yaml (based on suggestions by [email protected])
- fixed binding vs. driver&DTS use of hdmi-5v regulator (suggested by [email protected])
- dropped "drm/bridge: synopsis: Fix to properly handle HPD" - was a no longer needed workaround for a previous version
(suggested by [email protected])
PATCH V4 2021-09-27 18:44:38:
- fix setting output_port = 1 (issue found by [email protected])
- ci20.dts: convert to use hdmi-connector (by [email protected])
- add a hdmi-regulator to control +5V power (by [email protected])
- added a fix to dw-hdmi to call drm_kms_helper_hotplug_event on plugin event detection (by [email protected])
- always allocate extended descriptor but initialize only for jz4780 (by [email protected])
- updated to work on top of "[PATCH v3 0/6] drm/ingenic: Various improvements v3" (by [email protected])
- rebased to v5.13-rc3
PATCH V3 2021-08-08 07:10:50:
This series adds HDMI support for JZ4780 and CI20 board (and fixes one IPU related issue in registration error path)
- [patch 1/8] switched from mode_fixup to atomic_check (suggested by [email protected])
- the call to the dw-hdmi specialization is still called mode_fixup
- [patch 3/8] diverse fixes for ingenic-drm-drv (suggested by [email protected])
- factor out some non-HDMI features of the jz4780 into a separate patch
- multiple fixes around max height
- do not change regmap config but a copy on stack
- define some constants
- factor out fixing of drm_init error path for IPU into separate patch
- use FIELD_PREP()
- [patch 8/8] conversion to component framework dropped (suggested by [email protected] and [email protected])
PATCH V2 2021-08-05 16:08:05:
- code and commit messages revisited for checkpatch warnings
- rebased on v5.14-rc4
- include (failed, hence RFC 8/8) attempt to convert to component framework (was suggested by Paul Cercueil <[email protected]> a while ago)
This series adds HDMI support for JZ4780 and CI20 board
H. Nikolaus Schaller (1):
drm/bridge: display-connector: add ddc-en gpio support
Paul Boddie (1):
drm/ingenic: Add dw-hdmi driver specialization for jz4780
Paul Cercueil (1):
drm/ingenic: Implement proper .atomic_get_input_bus_fmts
drivers/gpu/drm/bridge/display-connector.c | 15 +++ drivers/gpu/drm/ingenic/Kconfig | 9 ++ drivers/gpu/drm/ingenic/Makefile | 1 + drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 28 +++++- drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c | 103 +++++++++++++++++++++ 5 files changed, 155 insertions(+), 1 deletion(-) create mode 100644 drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c
Signature · 1 line
2.33.0
PATCH v18 1/3 drm/bridge: display-connector: add ddc-en gpio support
"hdmi-connector.yaml" bindings defines an optional property "ddc-en-gpios" for a single gpio to enable DDC operation.
Usually this controls +5V power on the HDMI connector. This +5V may also be needed for HPD.
This was not reflected in code but is needed to make the CI20 board work.
Now, the driver activates the ddc gpio after probe and deactivates after remove so it is "almost on".
But only if this driver is loaded (and not e.g. blacklisted as module).
--- drivers/gpu/drm/bridge/display-connector.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+)
drivers/gpu/drm/bridge/display-connector.c +15 -0
@@ -24,6 +24,7 @@ struct display_connector {int hpd_irq;struct regulator *dp_pwr; +struct gpio_desc *ddc_en; }; static inline struct display_connector *
@@ -345,6 +346,17 @@ static int display_connector_probe(struct platform_device *pdev) } } +/* enable DDC */ +if (type ==DRM_MODE_CONNECTOR_HDMIA ) { + conn->ddc_en =devm_gpiod_get_optional (&pdev->dev,"ddc-en" , +GPIOD_OUT_HIGH ); + +if (IS_ERR (conn->ddc_en)) { +dev_err (&pdev->dev,"Couldn't get ddc-en gpio\n" ); +return PTR_ERR (conn->ddc_en); + } + } + conn->bridge.funcs = &display_connector_bridge_funcs; conn->bridge.of_node = pdev->dev.of_node;
@@ -373,6 +385,9 @@ static int display_connector_remove(struct platform_device *pdev) {struct display_connector *conn =platform_get_drvdata (pdev); +if (conn->ddc_en) +gpiod_set_value (conn->ddc_en,0 ); +if (conn->dp_pwr)regulator_disable (conn->dp_pwr);
Signature · 1 line
2.33.0
Re: [PATCH v18 1/3] drm/bridge: display-connector: add ddc-en gpio support
Quoted 52 lines · H. Nikolaus Schaller
"
hdmi-connector.yaml" bindings defines an optional property "ddc-en-gpios" for a single gpio to enable DDC operation.Usually this controls +5V power on the HDMI connector. This +5V may also be needed for HPD.
This was not reflected in code but is needed to make the CI20 board work.
Now, the driver activates the ddc gpio after probe and deactivates after remove so it is "almost on".
But only if this driver is loaded (and not e.g. blacklisted as module).
Signed-off-by H. Nikolaus Schaller <[email protected]>Reviewed-by Laurent Pinchart <[email protected]>--- drivers/gpu/drm/bridge/display-connector.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+)drivers/gpu/drm/bridge/display-connector.c +12
-0@@ -24,6 +24,7 @@ struct display_connector {int hpd_irq;struct regulator *dp_pwr; +struct gpio_desc *ddc_en; }; static inline struct display_connector *@@ -345,6 +346,17 @@ static int display_connector_probe(struct platform_device *pdev) } } +/* enable DDC */ +if (type ==DRM_MODE_CONNECTOR_HDMIA ) { + conn->ddc_en =devm_gpiod_get_optional (&pdev->dev,"ddc-en" , +GPIOD_OUT_HIGH ); + +if (IS_ERR (conn->ddc_en)) { +dev_err (&pdev->dev,"Couldn't get ddc-en gpio\n" ); +return PTR_ERR (conn->ddc_en); + } + } + conn->bridge.funcs = &display_connector_bridge_funcs; conn->bridge.of_node = pdev->dev.of_node;
drivers/gpu/drm/bridge/display-connector.c static int display_connector_remove(struct platform_device *pdev) · line 393
Quoted 9 lines
drivers/gpu/drm/bridge/display-connector.c +3
-0@@ -373,6 +385,9 @@ static int display_connector_remove(struct platform_device *pdev) {struct display_connector *conn =platform_get_drvdata (pdev); +if (conn->ddc_en) +gpiod_set_value (conn->ddc_en,0 ); +if (conn->dp_pwr)regulator_disable (conn->dp_pwr);
PATCH v18 2/3 drm/ingenic: Implement proper .atomic_get_input_bus_fmts
In reply to H. Nikolaus SchallerFrom: Paul Cercueil <[email protected]>
The .atomic_get_input_bus_fmts() callback of our top bridge should return the possible input formats for a given output format. If the requested output format is not supported, then NULL should be returned, otherwise the bus format negociation will end with a bus format that the encoder does not support.
--- drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 28 ++++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-)
drivers/gpu/drm/ingenic/ingenic-drm-drv.c +27 -1
@@ -833,6 +833,32 @@ static int ingenic_drm_bridge_atomic_check(struct drm_bridge *bridge, } } +static u32 * +ingenic_drm_bridge_atomic_get_input_bus_fmts (struct drm_bridge *bridge, +struct drm_bridge_state *bridge_state, +struct drm_crtc_state *crtc_state, +struct drm_connector_state *conn_state, +u32 output_fmt, +unsigned int *num_input_fmts) +{ +switch (output_fmt) { +case MEDIA_BUS_FMT_RGB888_1X24 : +case MEDIA_BUS_FMT_RGB666_1X18 : +case MEDIA_BUS_FMT_RGB565_1X16 : +case MEDIA_BUS_FMT_RGB888_3X8 : +case MEDIA_BUS_FMT_RGB888_3X8_DELTA : +break ; +default : + *num_input_fmts =0 ; +return NULL ; + } + +return drm_atomic_helper_bridge_propagate_bus_fmt (bridge, bridge_state, + crtc_state, conn_state, + output_fmt, + num_input_fmts); +} +static irqreturn_t ingenic_drm_irq_handler (int irq,void *arg) {struct ingenic_drm *priv =drm_device_get_priv (arg);
@@ -984,7 +1010,7 @@ static const struct drm_bridge_funcs ingenic_drm_bridge_funcs = { .atomic_reset = drm_atomic_helper_bridge_reset, .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, - .atomic_get_input_bus_fmts = drm_atomic_helper_bridge_propagate_bus_fmt, + .atomic_get_input_bus_fmts = ingenic_drm_bridge_atomic_get_input_bus_fmts, };static const struct drm_mode_config_funcs ingenic_drm_mode_config_funcs = {
Signature · 1 line
2.33.0
Re: [PATCH v18 2/3] drm/ingenic: Implement proper .atomic_get_input_bus_fmts
Quoted 52 lines · H. Nikolaus Schaller
From: Paul Cercueil <[email protected]>
The .atomic_get_input_bus_fmts() callback of our top bridge should return the possible input formats for a given output format. If the requested output format is not supported, then NULL should be returned, otherwise the bus format negociation will end with a bus format that the encoder does not support.
Signed-off-by Paul Cercueil <[email protected]>Signed-off-by H. Nikolaus Schaller <[email protected]>--- drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 28 ++++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-)drivers/gpu/drm/ingenic/ingenic-drm-drv.c +26
-0@@ -833,6 +833,32 @@ static int ingenic_drm_bridge_atomic_check(struct drm_bridge *bridge, } } +static u32 * +ingenic_drm_bridge_atomic_get_input_bus_fmts (struct drm_bridge *bridge, +struct drm_bridge_state *bridge_state, +struct drm_crtc_state *crtc_state, +struct drm_connector_state *conn_state, +u32 output_fmt, +unsigned int *num_input_fmts) +{ +switch (output_fmt) { +case MEDIA_BUS_FMT_RGB888_1X24 : +case MEDIA_BUS_FMT_RGB666_1X18 : +case MEDIA_BUS_FMT_RGB565_1X16 : +case MEDIA_BUS_FMT_RGB888_3X8 : +case MEDIA_BUS_FMT_RGB888_3X8_DELTA : +break ; +default : + *num_input_fmts =0 ; +return NULL ; + } + +return drm_atomic_helper_bridge_propagate_bus_fmt (bridge, bridge_state, + crtc_state, conn_state, + output_fmt, + num_input_fmts); +} +static irqreturn_t ingenic_drm_irq_handler (int irq,void *arg) {struct ingenic_drm *priv =drm_device_get_priv (arg);
drivers/gpu/drm/ingenic/ingenic-drm-drv.c static const struct drm_bridge_funcs ingenic_drm_bridge_funcs = { · line 1016
Quoted 8 lines
drivers/gpu/drm/ingenic/ingenic-drm-drv.c +1
-1@@ -984,7 +1010,7 @@ static const struct drm_bridge_funcs ingenic_drm_bridge_funcs = { .atomic_reset = drm_atomic_helper_bridge_reset, .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, - .atomic_get_input_bus_fmts = drm_atomic_helper_bridge_propagate_bus_fmt, + .atomic_get_input_bus_fmts = ingenic_drm_bridge_atomic_get_input_bus_fmts, };static const struct drm_mode_config_funcs ingenic_drm_mode_config_funcs = {
PATCH v18 3/3 drm/ingenic: Add dw-hdmi driver specialization for jz4780
In reply to H. Nikolaus SchallerFrom: Paul Boddie <[email protected]>
A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications.
Here we add Kconfig DRM_INGENIC_DW_HDMI, Makefile and driver code.
Note that there is no hpd-gpio installed on the CI20 board HDMI connector. Hence there is no hpd detection by the connector driver and we have to enable polling in the dw-hdmi core driver.
For that we need to set .poll_enabled but that struct component can only be accessed by core code. Hence we use the public setter function drm_kms_helper_hotplug_event() introduced before.
Also note that we disable Color Space Conversion since it is not working on jz4780.
--- drivers/gpu/drm/ingenic/Kconfig | 9 ++ drivers/gpu/drm/ingenic/Makefile | 1 + drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c | 103 ++++++++++++++++++++++ 3 files changed, 113 insertions(+) create mode 100644 drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c
drivers/gpu/drm/ingenic/Kconfig +9 -0
@@ -24,4 +24,13 @@ config DRM_INGENIC_IPU The Image Processing Unit (IPU) will appear as a second primary plane. +config DRM_INGENIC_DW_HDMI + tristate "Ingenic specific support for Synopsys DW HDMI" + depends on MACH_JZ4780 + select DRM_DW_HDMI + help + Choose this option to enable Synopsys DesignWare HDMI based driver. + If you want to enable HDMI on Ingenic JZ4780 based SoC, you should + select this option. + endif
drivers/gpu/drm/ingenic/Makefile +1 -0
@@ -1,3 +1,4 @@ obj-$(CONFIG_DRM_INGENIC) += ingenic-drm.o ingenic-drm-y = ingenic-drm-drv.o ingenic-drm-$(CONFIG_DRM_INGENIC_IPU) += ingenic-ipu.o +obj-$(CONFIG_DRM_INGENIC_DW_HDMI) += ingenic-dw-hdmi.o
drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +103 -0
103 lines not on this page · raw message
Signature · 1 line
2.33.0
Paul Cercueil
maintainerRe: [PATCH v18 3/3] drm/ingenic: Add dw-hdmi driver specialization for jz4780
Hi,
Quoted 16 lines · H. Nikolaus Schaller
From: Paul Boddie <[email protected]>
A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications.
Here we add
KconfigDRM_INGENIC_DW_HDMI,Makefileand driver code.Note that there is no hpd-gpio installed on the CI20 board HDMI connector. Hence there is no hpd detection by the connector driver and we have to enable polling in the dw-hdmi core driver.
For that we need to set .poll_enabled but that struct component can only be accessed by core code. Hence we use the public setter function
drm_kms_helper_hotplug_event()introduced before.
Quoted 6 lines
Also note that we disable Color Space Conversion since it is not working on jz4780.
Signed-off-by Paul Boddie <[email protected]>Signed-off-by Ezequiel Garcia <[email protected]>Signed-off-by H. Nikolaus Schaller <[email protected]>
The patch looks good, if I can get an ack/review (Neil?), I can merge the whole series.
Cheers,
-Paul
Quoted 156 lines
--- drivers/gpu/drm/ingenic/Kconfig | 9 ++ drivers/gpu/drm/ingenic/Makefile | 1 + drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c | 103 ++++++++++++++++++++++ 3 files changed, 113 insertions(+) create mode 100644 drivers/gpu/drm/ingenic/ingenic-dw-hdmi.cdrivers/gpu/drm/ingenic/Kconfig +0
-0@@ -24,4 +24,13 @@ config DRM_INGENIC_IPU The Image Processing Unit (IPU) will appear as a second primaryplane.
drivers/gpu/drm/ingenic/Kconfig +6
-0+config DRM_INGENIC_DW_HDMI + tristate "Ingenic specific support for Synopsys DW HDMI" + depends on MACH_JZ4780 + select DRM_DW_HDMI + help + Choose this option to enable Synopsys DesignWare HDMI baseddriver.
drivers/gpu/drm/ingenic/Kconfig +3
-0+ If you want to enable HDMI on Ingenic JZ4780 based SoC, you should + select this option. + endifdrivers/gpu/drm/ingenic/Makefile +1
-0@@ -1,3 +1,4 @@ obj-$(CONFIG_DRM_INGENIC) += ingenic-drm.o ingenic-drm-y = ingenic-drm-drv.o ingenic-drm-$(CONFIG_DRM_INGENIC_IPU) += ingenic-ipu.o +obj-$(CONFIG_DRM_INGENIC_DW_HDMI) += ingenic-dw-hdmi.odrivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +17
-0@@ -0,0 +1,103 @@ +// SPDX-License-Identifier: GPL-2.0 +/* Copyright (C) 2011-2013 Freescale Semiconductor, Inc. +* Copyright (C) 2019, 2020 Paul Boddie <[email protected]> +* +* Derived from dw_hdmi-imx.c with i.MX portions removed. +*/ + +#include <linux/module.h> +#include <linux/of_platform.h> +#include <linux/platform_device.h> + +#include <drm/bridge/dw_hdmi.h> +#include <drm/drm_of.h> +#include <drm/drm_print.h> + +static const struct dw_hdmi_mpll_config ingenic_mpll_cfg[] = { + {45250000 , { {0x01e0 ,0x0000 }, {0x21e1 ,0x0000 }, {0x41e2 ,
0x0000} } },drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +1
-0+ { 92500000, { { 0x0140, 0x0005 }, { 0x2141, 0x0005 }, { 0x4142,
0x0005} } },drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +1
-0+ { 148500000, { { 0x00a0, 0x000a }, { 0x20a1, 0x000a }, { 0x40a2,
0x000a} } },drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +1
-0+ { 216000000, { { 0x00a0, 0x000a }, { 0x2001, 0x000f }, { 0x4002,
0x000f} } },drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +1
-0+ { ~0UL, { { 0x0000, 0x0000 }, { 0x0000, 0x0000 }, { 0x0000,
0x0000} } }drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +32
-0+}; + +static const struct dw_hdmi_curr_ctrl ingenic_cur_ctr[] = { +/*pixelclk bpp8 bpp10 bpp12 */ + {54000000 , {0x091c ,0x091c ,0x06dc } }, + {58400000 , {0x091c ,0x06dc ,0x06dc } }, + {72000000 , {0x06dc ,0x06dc ,0x091c } }, + {74250000 , {0x06dc ,0x0b5c ,0x091c } }, + {118800000 , {0x091c ,0x091c ,0x06dc } }, + {216000000 , {0x06dc ,0x0b5c ,0x091c } }, + { ~0UL , {0x0000 ,0x0000 ,0x0000 } }, +}; + +/* +* Resistance term 133Ohm Cfg +* PREEMP config 0.00 +* TX/CK level 10 +*/ +static const struct dw_hdmi_phy_config ingenic_phy_config[] = { +/*pixelclk symbol term vlev */ + {216000000 ,0x800d ,0x0005 ,0x01ad }, + { ~0UL ,0x0000 ,0x0000 ,0x0000 } +}; + +static enum drm_mode_status +ingenic_dw_hdmi_mode_valid (struct dw_hdmi *hdmi,void *data, +const struct drm_display_info *info, +const struct drm_display_mode *mode) +{ +if (mode->clock <13500 ) +return MODE_CLOCK_LOW ; +/* FIXME: Hardware is capable of 270MHz, but setup data is missing. */
drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +36
-0+if (mode->clock >216000 ) +return MODE_CLOCK_HIGH ; + +return MODE_OK ; +} + +static struct dw_hdmi_plat_data ingenic_dw_hdmi_plat_data = { + .mpll_cfg = ingenic_mpll_cfg, + .cur_ctr = ingenic_cur_ctr, + .phy_config = ingenic_phy_config, + .mode_valid = ingenic_dw_hdmi_mode_valid, + .output_port =1 , +}; + +static const struct of_device_id ingenic_dw_hdmi_dt_ids[] = { + { .compatible ="ingenic,jz4780-dw-hdmi" }, + {/* Sentinel */ }, +}; +MODULE_DEVICE_TABLE (of, ingenic_dw_hdmi_dt_ids); + +static void ingenic_dw_hdmi_cleanup (void *data) +{ +struct dw_hdmi *hdmi = (struct dw_hdmi *)data; + +dw_hdmi_remove (hdmi); +} + +static int ingenic_dw_hdmi_probe (struct platform_device *pdev) +{ +struct dw_hdmi *hdmi; + + hdmi =dw_hdmi_probe (pdev, &ingenic_dw_hdmi_plat_data); +if (IS_ERR (hdmi)) +return PTR_ERR (hdmi); + + return devm_add_action_or_reset(&pdev->dev,
ingenic_dw_hdmi_cleanup, hdmi);drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +14
-1+} + +static struct platform_driver ingenic_dw_hdmi_driver = { + .probe = ingenic_dw_hdmi_probe, + .driver = { + .name ="dw-hdmi-ingenic" , + .of_match_table = ingenic_dw_hdmi_dt_ids, + }, +}; +module_platform_driver (ingenic_dw_hdmi_driver); + +MODULE_DESCRIPTION ("JZ4780 Specific DW-HDMI Driver Extension" ); +MODULE_LICENSE ("GPL v2" ); +MODULE_ALIAS ("platform:dw-hdmi-ingenic" ); --2.33.0
Re: [PATCH v18 3/3] drm/ingenic: Add dw-hdmi driver specialization for jz4780
In reply to H. Nikolaus SchallerQuoted 155 lines · H. Nikolaus Schaller
From: Paul Boddie <[email protected]>
A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications.
Here we add
KconfigDRM_INGENIC_DW_HDMI,Makefileand driver code.Note that there is no hpd-gpio installed on the CI20 board HDMI connector. Hence there is no hpd detection by the connector driver and we have to enable polling in the dw-hdmi core driver.
For that we need to set .poll_enabled but that struct component can only be accessed by core code. Hence we use the public setter function
drm_kms_helper_hotplug_event()introduced before.Also note that we disable Color Space Conversion since it is not working on jz4780.
Signed-off-by Paul Boddie <[email protected]>Signed-off-by Ezequiel Garcia <[email protected]>Signed-off-by H. Nikolaus Schaller <[email protected]>--- drivers/gpu/drm/ingenic/Kconfig | 9 ++ drivers/gpu/drm/ingenic/Makefile | 1 + drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c | 103 ++++++++++++++++++++++ 3 files changed, 113 insertions(+)create mode 100644
drivers/gpu/drm/ingenic/ingenic-dw-hdmi.cdrivers/gpu/drm/ingenic/Kconfig +9
-0@@ -24,4 +24,13 @@ config DRM_INGENIC_IPU The Image Processing Unit (IPU) will appear as a second primary plane. +config DRM_INGENIC_DW_HDMI + tristate "Ingenic specific support for Synopsys DW HDMI" + depends on MACH_JZ4780 + select DRM_DW_HDMI + help + Choose this option to enable Synopsys DesignWare HDMI based driver. + If you want to enable HDMI on Ingenic JZ4780 based SoC, you should + select this option. + endifdrivers/gpu/drm/ingenic/Makefile +1
-0@@ -1,3 +1,4 @@ obj-$(CONFIG_DRM_INGENIC) += ingenic-drm.o ingenic-drm-y = ingenic-drm-drv.o ingenic-drm-$(CONFIG_DRM_INGENIC_IPU) += ingenic-ipu.o +obj-$(CONFIG_DRM_INGENIC_DW_HDMI) += ingenic-dw-hdmi.odrivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +93
-0@@ -0,0 +1,103 @@ +// SPDX-License-Identifier: GPL-2.0 +/* Copyright (C) 2011-2013 Freescale Semiconductor, Inc. +* Copyright (C) 2019, 2020 Paul Boddie <[email protected]> +* +* Derived from dw_hdmi-imx.c with i.MX portions removed. +*/ + +#include <linux/module.h> +#include <linux/of_platform.h> +#include <linux/platform_device.h> + +#include <drm/bridge/dw_hdmi.h> +#include <drm/drm_of.h> +#include <drm/drm_print.h> + +static const struct dw_hdmi_mpll_config ingenic_mpll_cfg[] = { + {45250000 , { {0x01e0 ,0x0000 }, {0x21e1 ,0x0000 }, {0x41e2 ,0x0000 } } }, + {92500000 , { {0x0140 ,0x0005 }, {0x2141 ,0x0005 }, {0x4142 ,0x0005 } } }, + {148500000 , { {0x00a0 ,0x000a }, {0x20a1 ,0x000a }, {0x40a2 ,0x000a } } }, + {216000000 , { {0x00a0 ,0x000a }, {0x2001 ,0x000f }, {0x4002 ,0x000f } } }, + { ~0UL , { {0x0000 ,0x0000 }, {0x0000 ,0x0000 }, {0x0000 ,0x0000 } } } +}; + +static const struct dw_hdmi_curr_ctrl ingenic_cur_ctr[] = { +/*pixelclk bpp8 bpp10 bpp12 */ + {54000000 , {0x091c ,0x091c ,0x06dc } }, + {58400000 , {0x091c ,0x06dc ,0x06dc } }, + {72000000 , {0x06dc ,0x06dc ,0x091c } }, + {74250000 , {0x06dc ,0x0b5c ,0x091c } }, + {118800000 , {0x091c ,0x091c ,0x06dc } }, + {216000000 , {0x06dc ,0x0b5c ,0x091c } }, + { ~0UL , {0x0000 ,0x0000 ,0x0000 } }, +}; + +/* +* Resistance term 133Ohm Cfg +* PREEMP config 0.00 +* TX/CK level 10 +*/ +static const struct dw_hdmi_phy_config ingenic_phy_config[] = { +/*pixelclk symbol term vlev */ + {216000000 ,0x800d ,0x0005 ,0x01ad }, + { ~0UL ,0x0000 ,0x0000 ,0x0000 } +}; + +static enum drm_mode_status +ingenic_dw_hdmi_mode_valid (struct dw_hdmi *hdmi,void *data, +const struct drm_display_info *info, +const struct drm_display_mode *mode) +{ +if (mode->clock <13500 ) +return MODE_CLOCK_LOW ; +/* FIXME: Hardware is capable of 270MHz, but setup data is missing. */ +if (mode->clock >216000 ) +return MODE_CLOCK_HIGH ; + +return MODE_OK ; +} + +static struct dw_hdmi_plat_data ingenic_dw_hdmi_plat_data = { + .mpll_cfg = ingenic_mpll_cfg, + .cur_ctr = ingenic_cur_ctr, + .phy_config = ingenic_phy_config, + .mode_valid = ingenic_dw_hdmi_mode_valid, + .output_port =1 , +}; + +static const struct of_device_id ingenic_dw_hdmi_dt_ids[] = { + { .compatible ="ingenic,jz4780-dw-hdmi" }, + {/* Sentinel */ }, +}; +MODULE_DEVICE_TABLE (of, ingenic_dw_hdmi_dt_ids); + +static void ingenic_dw_hdmi_cleanup (void *data) +{ +struct dw_hdmi *hdmi = (struct dw_hdmi *)data; + +dw_hdmi_remove (hdmi); +} + +static int ingenic_dw_hdmi_probe (struct platform_device *pdev) +{ +struct dw_hdmi *hdmi; + + hdmi =dw_hdmi_probe (pdev, &ingenic_dw_hdmi_plat_data); +if (IS_ERR (hdmi)) +return PTR_ERR (hdmi); + +return devm_add_action_or_reset (&pdev->dev, ingenic_dw_hdmi_cleanup, hdmi); +} + +static struct platform_driver ingenic_dw_hdmi_driver = { + .probe = ingenic_dw_hdmi_probe,
drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c line 103
Quoted 10 lines
drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c new file +10
-0+ .driver = { + .name = "dw-hdmi-ingenic", + .of_match_table = ingenic_dw_hdmi_dt_ids, + }, +}; +module_platform_driver (ingenic_dw_hdmi_driver); + +MODULE_DESCRIPTION ("JZ4780 Specific DW-HDMI Driver Extension" ); +MODULE_LICENSE ("GPL v2" ); +MODULE_ALIAS ("platform:dw-hdmi-ingenic" );
Looks fine, LGTM now we figured out all the details !
Paul Cercueil
maintainerHi,
Series applied to drm-misc-next.
Good job to Nikolaus and everyone involved!
Cheers,
-Paul
Quoted 207 lines · H. Nikolaus Schaller
PATCH V18 2022-04-07 13:16:10:
* removed CSC filter patches (3-5/6) by Neil Armstrong <[email protected]> as they no longer seem to be required
* added "Reviewed-by: Laurent Pinchart <[email protected]>" to ddc-en patch (6/6)PATCH V17 2022-04-06 18:26:08:
* removed our local attempts to enable HPD because "drm/bridge_connector: enable HPD by default if supported"
became available and solves the issue globally (by
[email protected])
* add new mechanism to disable Color Space Conversion (by [email protected])
* rebased to v5.18-rc1 based drm-misc/drm-misc-nextPATCH V16 2022-02-26 18:13:02:
* fixed and renamed dw-hdmi bus negotiation patch (by [email protected])
* reordered and merged HPD fix (suggested by [email protected])
* fixedMODULE_ALIASfor dw-hdmi-ingenic (reported by [email protected])
* dropped some already merged commits from the seriesPATCH V15 2022-02-12 16:50:54:
* remove already (elsewhere) merged commits (suggested by [email protected])
* clarify commit message for (now) 1/7 ((suggested by [email protected]))PATCH V14 2022-02-12 15:19:25:
* make compatible toc03d0b52ff71d5("drm/connector: Fix typo in output format")
* move "dw-hdmi/ingenic-dw-hdmi: repair interworking with hdmi-connector" before drm/ingenic: Add dw-hdmi driver specialization for jz4780 (by
[email protected])
* split introduction ofdw_hdmi_enable_poll()into separate patch
* explicitly mark plane f0 as not working in jz4780 (suggested by [email protected])
* drop 1/9 since it is now in drm-misc/drm-misc-nextPATCH V13 2022-02-02 17:31:22:
* 7/9: remove call togpiod_set_value()because ofGPIOD_OUT_HIGH(by [email protected])
* 4/9: replace ".." by "." (by [email protected])
* 3/9: remove old hdmi-5v-power in the example (by [email protected])
* 2/9: disable handling of plane f0 only for jz4780 (by [email protected])PATCH V12 2022-01-31 13:26:54:
This version reworks how hdmi ddc power is controlled by connector and not by ddc/hdmi bridge driver.Also some patches of the previous version of this series have been removed since they are already applied to mips-next/linux/next/v5.17-rc1.
Fixes and changes:
- repair interworking of dw-hdmi with connector-hdmi (by [email protected])
- fixJZ_REG_LCD_OSDCsetup for jz4780 (by [email protected] and [email protected])
- adjustments forci20.dtsto use connector gpio for +5v (suggested by several)
- to add control of "ddc-en-gpios" to hdmi-connector driver (by [email protected])
- regulator code removed because we now use the "ddc-en-gpios" of the connector
driver (suggested by [email protected])
- bindings: addition of "ddc-i2c-bus" and "hdmi-5v-supply" removed (suggested by [email protected])
- rebase on v5.17-rc2PATCH V11 2021-12-02 19:39:52:
- patch 4/8: changedevm_regulator_get_optionaltodevm_regulator_getand
remove NULL check (requested by [email protected])
- patch 3/8: make hdmi-5v-supply required (requested by [email protected])PATCH V10 2021-11-30 22:26:41:
- patch 3/8: fix $id and $ref paths (found by [email protected])PATCH V9 2021-11-24 22:29:14:
- patch 6/8: remove optional <0> for assigned-clocks and unintentionally included "unwedge" setup (found by [email protected])
- patch 4/8: some cosmetics
make regulator enable/disable only if not NULL (found by
[email protected])
simplify/fix error handling and driver cleanup on remove
(proposed by [email protected])
- patch 3/8: fix #include path in example (found by [email protected])
fix missing "i" in unevaluatedProperties (found by
[email protected])
fix 4 spaces indentation for required: property (found
by [email protected])PATCH V8 2021-11-23 19:14:00:
- fix a bad editing result from patch 2/8 (found by [email protected])PATCH V7 2021-11-23 18:46:23:
- changed gpio polarity ofhdmi_powerto 0 (suggested by [email protected])
- fixed LCD1 irq number (bug found by [email protected])
- removed "- 4" for calculatingmax_register(suggested by [email protected])
- use unevaluatedPropertes instead of additionalProperties (suggested by [email protected])
- moved and renamed ingenic,jz4780-hdmi.yaml (suggested by [email protected])
- adjusted assigned-clocks changes to upstream which added some for SSI (by [email protected])
- rebased and tested with v5.16-rc2 + patch set drm/ingenic by [email protected] (by [email protected])PATCH V6 2021-11-10 20:43:33:
- changedCONFIG_DRM_INGENIC_DW_HDMIto "m" (by [email protected])
- made ingenic-dw-hdmi an independent platform driver which can be compiled as module
and removed error patch fixes for IPU (suggested by
[email protected])
- moved assigned-clocks fromjz4780.dtsitoci20.dts(suggested by [email protected])
- fixed reg property injz4780.dtsito cover all registers incl. gamma and vee (by [email protected])
- added a base patch to calculate regmap size from DTS reg property (requested by [email protected])
- restored resetting all bits except one in LCDOSDC (requested by [email protected])
- clarified setting of cpos (suggested by [email protected])
- moved bindings definition for ddc-i2c-bus (suggested by [email protected])
- simplified mask definitions forJZ_LCD_DESSIZE(requested by [email protected])
- removed setting alpha premultiplication (suggested by [email protected])
- removed some comments (suggested by [email protected])PATCH V5 2021-10-05 14:28:44:
- droppedmode_fixupand timings support in dw-hdmi as it is no
longer needed in this V5 (by [email protected])
- dropped "drm/ingenic: add some jz4780 specific features"
(stimulated by [email protected])
- fixed typo in commit subject: "synopsis" -> "synopsys" (by
[email protected])
- swapped clocks injz4780.dtsito match synopsys,dw-hdmi.yaml (by [email protected])
- improved, simplified, fixed, dtbscheckedingenic-jz4780-hdmi.yaml
and made dependent ofbridge/synopsys,dw-hdmi.yaml(based on suggestions by [email protected])
- fixed binding vs. driver&DTS use of hdmi-5v regulator (suggested by [email protected])
- dropped "drm/bridge: synopsis: Fix to properly handle HPD" - was a
no longer needed workaround for a previous version
(suggested by [email protected])PATCH V4 2021-09-27 18:44:38:
- fix settingoutput_port= 1 (issue found by [email protected])
-ci20.dts: convert to use hdmi-connector (by [email protected])
- add a hdmi-regulator to control +5V power (by [email protected])
- added a fix to dw-hdmi to calldrm_kms_helper_hotplug_eventon plugin event detection (by [email protected])
- always allocate extended descriptor but initialize only for jz4780 (by [email protected])
- updated to work on top of "[PATCH v3 0/6] drm/ingenic: Various improvements v3" (by [email protected])
- rebased to v5.13-rc3PATCH V3 2021-08-08 07:10:50:
This series adds HDMI support for JZ4780 and CI20 board (and fixes one IPU related issue in registration error path)
- [patch 1/8] switched frommode_fixuptoatomic_check(suggested by
[email protected])
- the call to the dw-hdmi specialization is still calledmode_fixup
- [patch 3/8] diverse fixes for ingenic-drm-drv (suggested by
[email protected])
- factor out some non-HDMI features of the jz4780 into a separate
patch
- multiple fixes around max height
- do not change regmap config but a copy on stack
- define some constants
- factor out fixing ofdrm_initerror path for IPU into separate
patch
- useFIELD_PREP()
- [patch 8/8] conversion to component framework dropped (suggested by [email protected] and [email protected])PATCH V2 2021-08-05 16:08:05:
- code and commit messages revisited for checkpatch warnings
- rebased on v5.14-rc4
- include (failed, hence RFC 8/8) attempt to convert to component framework
(was suggested by Paul Cercueil <[email protected]> a while ago)This series adds HDMI support for JZ4780 and CI20 board
H. Nikolaus Schaller (1):
drm/bridge: display-connector: add ddc-en gpio supportPaul Boddie (1):
drm/ingenic: Add dw-hdmi driver specialization for jz4780Paul Cercueil (1):
drm/ingenic: Implement proper .atomic_get_input_bus_fmtsdrivers/gpu/drm/bridge/display-connector.c | 15 +++ drivers/gpu/drm/ingenic/Kconfig | 9 ++ drivers/gpu/drm/ingenic/Makefile | 1 + drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 28 +++++- drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c | 103 +++++++++++++++++++++ 5 files changed, 155 insertions(+), 1 deletion(-) create mode 100644 drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c--
2.33.0