Message ID | 20240521105817.3301-1-keith.zhao@starfivetech.com (mailing list archive) |
---|---|
Headers | show |
Series | drm/verisilicon : support DC8200 and inno hdmi | expand |
On 21/05/2024 12:58, keith wrote: > Verisilicon/DC8200 display controller IP has 2 display pipes and each > pipe support a primary plane and a cursor plane . > In addition, there are four overlay planes as two display pipes common resources. > > The first display pipe is bound to the inno HDMI encoder. > The second display pipe is bound to a simple encoder, which is used to > find dsi bridge by dts node. > > Patch 1 adds YAML schema for JH7110 display pipeline. > > Patches 2 to 3 add inno common api and match the ROCKCHIP inno hdmi driver > by calling the common api. > The collating public interface is based on ROCKCHIP inno hdmi, > and it can be resused by JH7110 inno hdmi. > Those common api are tested on rk-3128 SDK, which kernel version is 4.x. > step1, make sure the process is consistent with the latest kernel version. > step2, just remove the interface and add a common interface. > > Patches 4 to 8 add kms driver for dc8200 display controller. > > Patch 9 adds inno hdmi support for JH7110 display pipeline. > > Patch 10 adds a simple encoder. > > This patchset should be applied on next branch. > Please fix your clock, so this patchset does not appear +4h in the future and on top of other patches... Best regards, Krzysztof
Hi Alex, Am Dienstag, 21. Mai 2024, 12:58:07 CEST schrieb keith: > Verisilicon/DC8200 display controller IP has 2 display pipes and each > pipe support a primary plane and a cursor plane . > In addition, there are four overlay planes as two display pipes common resources. > > The first display pipe is bound to the inno HDMI encoder. > The second display pipe is bound to a simple encoder, which is used to > find dsi bridge by dts node. > > Patch 1 adds YAML schema for JH7110 display pipeline. > > Patches 2 to 3 add inno common api and match the ROCKCHIP inno hdmi driver > by calling the common api. > The collating public interface is based on ROCKCHIP inno hdmi, > and it can be resused by JH7110 inno hdmi. > Those common api are tested on rk-3128 SDK, which kernel version is 4.x. as you were working on the rk3128-inno-hdmi variant recently and I don't really have a rk3036 or rk3128 in working condition right now, could you give this series a try. For reference, the full series is at lore: https://lore.kernel.org/dri-devel/20240521105817.3301-1-keith.zhao@starfivetech.com/ and generalizes the inno-hdmi driver into the bridge model we have in a number of other places already. Thanks Heiko > step1, make sure the process is consistent with the latest kernel version. > step2, just remove the interface and add a common interface. > > Patches 4 to 8 add kms driver for dc8200 display controller. > > Patch 9 adds inno hdmi support for JH7110 display pipeline. > > Patch 10 adds a simple encoder. > > This patchset should be applied on next branch. > > V1: > Changes since v1: > - Further standardize the yaml file. > - Dts naming convention improved. > - Fix the problem of compiling and loading ko files. > - Use drm new api to automatically manage resources. > - Drop vs_crtc_funcs&vs_plane_funcs, subdivide the plane's help interface. > - Reduce the modifiers unused. > - Optimize the hdmi driver code > > V2: > Changes since v2: > - fix the error about checking the yaml file. > - match drm driver GEM DMA API. > - Delete the custom crtc property . > - hdmi use drmm_ new api to automatically manage resources. > - update the modifiers comments. > - enabling KASAN, fix the error during removing module > > V3: > Changes since v3: > - Delete the custom plane property. > - Delete the custom fourcc modifiers. > - Adjust the calculation mode of hdmi pixclock. > - Add match data for dc8200 driver. > - Adjust some magic values. > - Add a simple encoder for dsi output. > > V4: > Changes since v4: > - Delete the display subsystem module as all crtcs and planes are a driver. > - Delete the custom struct, directly use the drm struct data. > - Tidy up the inno hdmi public interface. > - Add a simple encoder for dsi output. > > keith (10): > dt-bindings: display: Add YAML schema for JH7110 display pipeline > drm/bridge: add common api for inno hdmi > drm/rockchip:hdmi: migrate to use inno-hdmi bridge driver > drm/vs: Add hardware funcs for vs. > drm/vs: add vs mode config init > drm/vs: add vs plane api > drm/vs: add ctrc fun > drm/vs: add vs drm master driver > drm/vs: Innosilicon HDMI support > drm/vs: add simple dsi encoder > > .../display/bridge/innosilicon,inno-hdmi.yaml | 49 + > .../display/rockchip/rockchip,inno-hdmi.yaml | 27 +- > .../starfive/starfive,dsi-encoder.yaml | 92 ++ > .../starfive/starfive,jh7110-dc8200.yaml | 169 +++ > .../starfive/starfive,jh7110-inno-hdmi.yaml | 75 ++ > .../soc/starfive/starfive,jh7110-syscon.yaml | 1 + > MAINTAINERS | 11 + > drivers/gpu/drm/Kconfig | 2 + > drivers/gpu/drm/Makefile | 1 + > drivers/gpu/drm/bridge/Kconfig | 2 + > drivers/gpu/drm/bridge/Makefile | 1 + > drivers/gpu/drm/bridge/innosilicon/Kconfig | 6 + > drivers/gpu/drm/bridge/innosilicon/Makefile | 2 + > .../gpu/drm/bridge/innosilicon/inno-hdmi.c | 587 +++++++++ > .../gpu/drm/bridge/innosilicon/inno-hdmi.h | 97 ++ > drivers/gpu/drm/rockchip/Kconfig | 1 + > drivers/gpu/drm/rockchip/Makefile | 2 +- > drivers/gpu/drm/rockchip/inno_hdmi-rockchip.c | 517 ++++++++ > .../{inno_hdmi.h => inno_hdmi-rockchip.h} | 45 - > drivers/gpu/drm/rockchip/inno_hdmi.c | 1073 ----------------- > drivers/gpu/drm/verisilicon/Kconfig | 23 + > drivers/gpu/drm/verisilicon/Makefile | 11 + > .../gpu/drm/verisilicon/inno_hdmi-starfive.c | 481 ++++++++ > .../gpu/drm/verisilicon/inno_hdmi-starfive.h | 152 +++ > drivers/gpu/drm/verisilicon/vs_crtc.c | 241 ++++ > drivers/gpu/drm/verisilicon/vs_crtc.h | 17 + > drivers/gpu/drm/verisilicon/vs_dc_hw.c | 1060 ++++++++++++++++ > drivers/gpu/drm/verisilicon/vs_dc_hw.h | 493 ++++++++ > drivers/gpu/drm/verisilicon/vs_drv.c | 721 +++++++++++ > drivers/gpu/drm/verisilicon/vs_drv.h | 98 ++ > drivers/gpu/drm/verisilicon/vs_modeset.c | 36 + > drivers/gpu/drm/verisilicon/vs_modeset.h | 10 + > drivers/gpu/drm/verisilicon/vs_plane.c | 487 ++++++++ > drivers/gpu/drm/verisilicon/vs_plane.h | 26 + > drivers/gpu/drm/verisilicon/vs_simple_enc.c | 190 +++ > drivers/gpu/drm/verisilicon/vs_simple_enc.h | 25 + > drivers/gpu/drm/verisilicon/vs_type.h | 84 ++ > include/drm/bridge/inno_hdmi.h | 69 ++ > 38 files changed, 5840 insertions(+), 1144 deletions(-) > create mode 100644 Documentation/devicetree/bindings/display/bridge/innosilicon,inno-hdmi.yaml > create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,dsi-encoder.yaml > create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,jh7110-dc8200.yaml > create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,jh7110-inno-hdmi.yaml > create mode 100644 drivers/gpu/drm/bridge/innosilicon/Kconfig > create mode 100644 drivers/gpu/drm/bridge/innosilicon/Makefile > create mode 100644 drivers/gpu/drm/bridge/innosilicon/inno-hdmi.c > create mode 100644 drivers/gpu/drm/bridge/innosilicon/inno-hdmi.h > create mode 100644 drivers/gpu/drm/rockchip/inno_hdmi-rockchip.c > rename drivers/gpu/drm/rockchip/{inno_hdmi.h => inno_hdmi-rockchip.h} (85%) > delete mode 100644 drivers/gpu/drm/rockchip/inno_hdmi.c > create mode 100644 drivers/gpu/drm/verisilicon/Kconfig > create mode 100644 drivers/gpu/drm/verisilicon/Makefile > create mode 100644 drivers/gpu/drm/verisilicon/inno_hdmi-starfive.c > create mode 100644 drivers/gpu/drm/verisilicon/inno_hdmi-starfive.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_crtc.c > create mode 100644 drivers/gpu/drm/verisilicon/vs_crtc.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_dc_hw.c > create mode 100644 drivers/gpu/drm/verisilicon/vs_dc_hw.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_drv.c > create mode 100644 drivers/gpu/drm/verisilicon/vs_drv.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_modeset.c > create mode 100644 drivers/gpu/drm/verisilicon/vs_modeset.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_plane.c > create mode 100644 drivers/gpu/drm/verisilicon/vs_plane.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_simple_enc.c > create mode 100644 drivers/gpu/drm/verisilicon/vs_simple_enc.h > create mode 100644 drivers/gpu/drm/verisilicon/vs_type.h > create mode 100644 include/drm/bridge/inno_hdmi.h > >
Hi Heiko, Am 21.05.24 um 10:03 schrieb Heiko Stübner: > Hi Alex, > > Am Dienstag, 21. Mai 2024, 12:58:07 CEST schrieb keith: >> Verisilicon/DC8200 display controller IP has 2 display pipes and each >> pipe support a primary plane and a cursor plane . >> In addition, there are four overlay planes as two display pipes common resources. >> >> The first display pipe is bound to the inno HDMI encoder. >> The second display pipe is bound to a simple encoder, which is used to >> find dsi bridge by dts node. >> >> Patch 1 adds YAML schema for JH7110 display pipeline. >> >> Patches 2 to 3 add inno common api and match the ROCKCHIP inno hdmi driver >> by calling the common api. >> The collating public interface is based on ROCKCHIP inno hdmi, >> and it can be resused by JH7110 inno hdmi. >> Those common api are tested on rk-3128 SDK, which kernel version is 4.x. > as you were working on the rk3128-inno-hdmi variant recently > and I don't really have a rk3036 or rk3128 in working condition > right now, could you give this series a try. Sure, I'm going to check this soon. Regards, Alex > For reference, the full series is at lore: > https://lore.kernel.org/dri-devel/20240521105817.3301-1-keith.zhao@starfivetech.com/ > > and generalizes the inno-hdmi driver into the bridge model we > have in a number of other places already. > > > Thanks > Heiko > > > >> step1, make sure the process is consistent with the latest kernel version. >> step2, just remove the interface and add a common interface. >> >> Patches 4 to 8 add kms driver for dc8200 display controller. >> >> Patch 9 adds inno hdmi support for JH7110 display pipeline. >> >> Patch 10 adds a simple encoder. >> >> This patchset should be applied on next branch. >> >> V1: >> Changes since v1: >> - Further standardize the yaml file. >> - Dts naming convention improved. >> - Fix the problem of compiling and loading ko files. >> - Use drm new api to automatically manage resources. >> - Drop vs_crtc_funcs&vs_plane_funcs, subdivide the plane's help interface. >> - Reduce the modifiers unused. >> - Optimize the hdmi driver code >> >> V2: >> Changes since v2: >> - fix the error about checking the yaml file. >> - match drm driver GEM DMA API. >> - Delete the custom crtc property . >> - hdmi use drmm_ new api to automatically manage resources. >> - update the modifiers comments. >> - enabling KASAN, fix the error during removing module >> >> V3: >> Changes since v3: >> - Delete the custom plane property. >> - Delete the custom fourcc modifiers. >> - Adjust the calculation mode of hdmi pixclock. >> - Add match data for dc8200 driver. >> - Adjust some magic values. >> - Add a simple encoder for dsi output. >> >> V4: >> Changes since v4: >> - Delete the display subsystem module as all crtcs and planes are a driver. >> - Delete the custom struct, directly use the drm struct data. >> - Tidy up the inno hdmi public interface. >> - Add a simple encoder for dsi output. >> >> keith (10): >> dt-bindings: display: Add YAML schema for JH7110 display pipeline >> drm/bridge: add common api for inno hdmi >> drm/rockchip:hdmi: migrate to use inno-hdmi bridge driver >> drm/vs: Add hardware funcs for vs. >> drm/vs: add vs mode config init >> drm/vs: add vs plane api >> drm/vs: add ctrc fun >> drm/vs: add vs drm master driver >> drm/vs: Innosilicon HDMI support >> drm/vs: add simple dsi encoder >> >> .../display/bridge/innosilicon,inno-hdmi.yaml | 49 + >> .../display/rockchip/rockchip,inno-hdmi.yaml | 27 +- >> .../starfive/starfive,dsi-encoder.yaml | 92 ++ >> .../starfive/starfive,jh7110-dc8200.yaml | 169 +++ >> .../starfive/starfive,jh7110-inno-hdmi.yaml | 75 ++ >> .../soc/starfive/starfive,jh7110-syscon.yaml | 1 + >> MAINTAINERS | 11 + >> drivers/gpu/drm/Kconfig | 2 + >> drivers/gpu/drm/Makefile | 1 + >> drivers/gpu/drm/bridge/Kconfig | 2 + >> drivers/gpu/drm/bridge/Makefile | 1 + >> drivers/gpu/drm/bridge/innosilicon/Kconfig | 6 + >> drivers/gpu/drm/bridge/innosilicon/Makefile | 2 + >> .../gpu/drm/bridge/innosilicon/inno-hdmi.c | 587 +++++++++ >> .../gpu/drm/bridge/innosilicon/inno-hdmi.h | 97 ++ >> drivers/gpu/drm/rockchip/Kconfig | 1 + >> drivers/gpu/drm/rockchip/Makefile | 2 +- >> drivers/gpu/drm/rockchip/inno_hdmi-rockchip.c | 517 ++++++++ >> .../{inno_hdmi.h => inno_hdmi-rockchip.h} | 45 - >> drivers/gpu/drm/rockchip/inno_hdmi.c | 1073 ----------------- >> drivers/gpu/drm/verisilicon/Kconfig | 23 + >> drivers/gpu/drm/verisilicon/Makefile | 11 + >> .../gpu/drm/verisilicon/inno_hdmi-starfive.c | 481 ++++++++ >> .../gpu/drm/verisilicon/inno_hdmi-starfive.h | 152 +++ >> drivers/gpu/drm/verisilicon/vs_crtc.c | 241 ++++ >> drivers/gpu/drm/verisilicon/vs_crtc.h | 17 + >> drivers/gpu/drm/verisilicon/vs_dc_hw.c | 1060 ++++++++++++++++ >> drivers/gpu/drm/verisilicon/vs_dc_hw.h | 493 ++++++++ >> drivers/gpu/drm/verisilicon/vs_drv.c | 721 +++++++++++ >> drivers/gpu/drm/verisilicon/vs_drv.h | 98 ++ >> drivers/gpu/drm/verisilicon/vs_modeset.c | 36 + >> drivers/gpu/drm/verisilicon/vs_modeset.h | 10 + >> drivers/gpu/drm/verisilicon/vs_plane.c | 487 ++++++++ >> drivers/gpu/drm/verisilicon/vs_plane.h | 26 + >> drivers/gpu/drm/verisilicon/vs_simple_enc.c | 190 +++ >> drivers/gpu/drm/verisilicon/vs_simple_enc.h | 25 + >> drivers/gpu/drm/verisilicon/vs_type.h | 84 ++ >> include/drm/bridge/inno_hdmi.h | 69 ++ >> 38 files changed, 5840 insertions(+), 1144 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/display/bridge/innosilicon,inno-hdmi.yaml >> create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,dsi-encoder.yaml >> create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,jh7110-dc8200.yaml >> create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,jh7110-inno-hdmi.yaml >> create mode 100644 drivers/gpu/drm/bridge/innosilicon/Kconfig >> create mode 100644 drivers/gpu/drm/bridge/innosilicon/Makefile >> create mode 100644 drivers/gpu/drm/bridge/innosilicon/inno-hdmi.c >> create mode 100644 drivers/gpu/drm/bridge/innosilicon/inno-hdmi.h >> create mode 100644 drivers/gpu/drm/rockchip/inno_hdmi-rockchip.c >> rename drivers/gpu/drm/rockchip/{inno_hdmi.h => inno_hdmi-rockchip.h} (85%) >> delete mode 100644 drivers/gpu/drm/rockchip/inno_hdmi.c >> create mode 100644 drivers/gpu/drm/verisilicon/Kconfig >> create mode 100644 drivers/gpu/drm/verisilicon/Makefile >> create mode 100644 drivers/gpu/drm/verisilicon/inno_hdmi-starfive.c >> create mode 100644 drivers/gpu/drm/verisilicon/inno_hdmi-starfive.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_crtc.c >> create mode 100644 drivers/gpu/drm/verisilicon/vs_crtc.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_dc_hw.c >> create mode 100644 drivers/gpu/drm/verisilicon/vs_dc_hw.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_drv.c >> create mode 100644 drivers/gpu/drm/verisilicon/vs_drv.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_modeset.c >> create mode 100644 drivers/gpu/drm/verisilicon/vs_modeset.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_plane.c >> create mode 100644 drivers/gpu/drm/verisilicon/vs_plane.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_simple_enc.c >> create mode 100644 drivers/gpu/drm/verisilicon/vs_simple_enc.h >> create mode 100644 drivers/gpu/drm/verisilicon/vs_type.h >> create mode 100644 include/drm/bridge/inno_hdmi.h >> >> > > >