[PATCH v8 0/8] MIPS: JZ4780 and CI20 HDMI

Tue Nov 23 20:44:07 UTC 2021

Hi Paul,

> Am 23.11.2021 um 21:12 schrieb Paul Cercueil <paul at crapouillou.net>:
> 
> Hi Nikolaus,
> 
> I think if you can fix the last few things I commented on, and I get an ACK from Rob for the Device Tree related patches, then it will be ready to merge.

Fine! Especially for finding the NULL regulator risk.

Will do in the next days.
For the unwedge pinmux I have to check if we need it at all.

BR and thanks,
Nikolaus

> 
> Cheers,
> -Paul
> 
> 
> Le mar., nov. 23 2021 at 19:13:53 +0100, H. Nikolaus Schaller <hns at goldelico.com> a écrit :
>> PATCH V8 2021-11-23 19:14:00:
>> - fix a bad editing result from patch 2/8 (found by paul at crapouillou.net)
>> PATCH V7 2021-11-23 18:46:23:
>> - changed gpio polarity of hdmi_power to 0 (suggested by paul at crapouillou.net)
>> - fixed LCD1 irq number (bug found by paul at crapouillou.net)
>> - removed "- 4" for calculating max_register (suggested by paul at crapouillou.net)
>> - use unevaluatedPropertes instead of additionalProperties (suggested by robh at kernel.org)
>> - moved and renamed ingenic,jz4780-hdmi.yaml (suggested by robh at kernel.org)
>> - adjusted assigned-clocks changes to upstream which added some for SSI (by hns at goldelico.com)
>> - rebased and tested with v5.16-rc2 + patch set drm/ingenic by paul at crapouillou.net (by hns at goldelico.com)
>> PATCH V6 2021-11-10 20:43:33:
>> - changed CONFIG_DRM_INGENIC_DW_HDMI to "m" (by hns at goldelico.com)
>> - made ingenic-dw-hdmi an independent platform driver which can be compiled as module
>>  and removed error patch fixes for IPU (suggested by paul at crapouillou.net)
>> - moved assigned-clocks from jz4780.dtsi to ci20.dts (suggested by paul at crapouillou.net)
>> - fixed reg property in jz4780.dtsi to cover all registers incl. gamma and vee (by hns at goldelico.com)
>> - added a base patch to calculate regmap size from DTS reg property (requested by paul at crapouillou.net)
>> - restored resetting all bits except one in LCDOSDC (requested by paul at crapouillou.net)
>> - clarified setting of cpos (suggested by paul at crapouillou.net)
>> - moved bindings definition for ddc-i2c-bus (suggested by paul at crapouillou.net)
>> - simplified mask definitions for JZ_LCD_DESSIZE (requested by paul at crapouillou.net)
>> - removed setting alpha premultiplication (suggested by paul at crapouillou.net)
>> - removed some comments (suggested by paul at crapouillou.net)
>> 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 hns at goldelico.com)
>> - dropped "drm/ingenic: add some jz4780 specific features" (stimulated by paul at crapouillou.net)
>> - fixed typo in commit subject: "synopsis" -> "synopsys" (by hns at goldelico.com)
>> - swapped clocks in jz4780.dtsi to match synopsys,dw-hdmi.yaml (by hns at goldelico.com)
>> - improved, simplified, fixed, dtbschecked ingenic-jz4780-hdmi.yaml and made dependent of bridge/synopsys,dw-hdmi.yaml (based on suggestions by maxime at cerno.tech)
>> - fixed binding vs. driver&DTS use of hdmi-5v regulator (suggested by maxime at cerno.tech)
>> - dropped "drm/bridge: synopsis: Fix to properly handle HPD" - was a no longer needed workaround for a previous version
>>  (suggested by maxime at cerno.tech)
>> PATCH V4 2021-09-27 18:44:38:
>> - fix setting output_port = 1 (issue found by paul at crapouillou.net)
>> - ci20.dts: convert to use hdmi-connector (by hns at goldelico.com)
>> - add a hdmi-regulator to control +5V power (by hns at goldelico.com)
>> - added a fix to dw-hdmi to call drm_kms_helper_hotplug_event on plugin event detection (by hns at goldelico.com)
>> - always allocate extended descriptor but initialize only for jz4780 (by hns at goldelico.com)
>> - updated to work on top of "[PATCH v3 0/6] drm/ingenic: Various improvements v3" (by paul at crapouillou.net)
>> - 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 robert.foss at linaro.org)
>>  - the call to the dw-hdmi specialization is still called mode_fixup
>> - [patch 3/8] diverse fixes for ingenic-drm-drv (suggested by paul at crapouillou.net)
>>  - 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 Laurent.pinchart at ideasonboard.com and paul at crapouillou.net)
>> 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 <paul at crapouillou.net> a while ago)
>> This series adds HDMI support for JZ4780 and CI20 board
>> H. Nikolaus Schaller (3):
>>  drm/ingenic: prepare ingenic drm for later addition of JZ4780
>>  MIPS: defconfig: CI20: configure for DRM_DW_HDMI_JZ4780
>>  [RFC] MIPS: DTS: Ingenic: adjust register size to available registers
>> Paul Boddie (4):
>>  drm/ingenic: Add support for JZ4780 and HDMI output
>>  drm/ingenic: Add dw-hdmi driver for jz4780
>>  MIPS: DTS: jz4780: Account for Synopsys HDMI driver and LCD
>>    controllers
>>  MIPS: DTS: CI20: Add DT nodes for HDMI setup
>> Sam Ravnborg (1):
>>  dt-bindings: display: Add ingenic,jz4780-dw-hdmi DT Schema
>> .../display/bridge/ingenic,jz4780-hdmi.yaml   |  76 +++++++++++
>> .../display/bridge/synopsys,dw-hdmi.yaml      |   3 +
>> arch/mips/boot/dts/ingenic/ci20.dts           |  83 ++++++++++-
>> arch/mips/boot/dts/ingenic/jz4725b.dtsi       |   2 +-
>> arch/mips/boot/dts/ingenic/jz4740.dtsi        |   2 +-
>> arch/mips/boot/dts/ingenic/jz4770.dtsi        |   2 +-
>> arch/mips/boot/dts/ingenic/jz4780.dtsi        |  40 ++++++
>> arch/mips/configs/ci20_defconfig              |   6 +
>> drivers/gpu/drm/ingenic/Kconfig               |   9 ++
>> drivers/gpu/drm/ingenic/Makefile              |   1 +
>> drivers/gpu/drm/ingenic/ingenic-drm-drv.c     |  62 ++++++++-
>> drivers/gpu/drm/ingenic/ingenic-drm.h         |  38 ++++++
>> drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c     | 129 ++++++++++++++++++
>> 13 files changed, 444 insertions(+), 9 deletions(-)
>> create mode 100644 Documentation/devicetree/bindings/display/bridge/ingenic,jz4780-hdmi.yaml
>> create mode 100644 drivers/gpu/drm/ingenic/ingenic-dw-hdmi.c
>> --
>> 2.33.0
> 
> 