On Wed, Sep 24, 2025 at 02:49:53PM +0800, Chu Guangqing wrote:
> Sorry, Thomas. The changes have been made this time.
>
> >Hi
> >
> >Am 10.09.25 um 04:23 schrieb Chu Guangqing:
> >> + select DRM_VRAM_HELPER
> >> + select DRM_TTM_HELPER
> >
> >I told you twice that VRAM helpers are obsolete and what to use instead.
> >
> It has been modified.
> >> +
> >> +struct yhgch_vdac {
> >> + struct drm_connector base;
> >> + struct i2c_adapter adapter;
> >> + struct i2c_algo_bit_data bit_data;
> >> +};
> >
> >I've asked you several times whether to adopt the coding style in
> >ast/mgag200 for the DDC.
> It has been modified.
> >
> >No only that you not respond to to any of my reviews, you keep on submitting
> >the same code again and again. Linux kernel development does not work like
> >that.
> >
> >NAK on this driver until you get your act together.
> >
> >Best regards
> >Thomas
> >
>
> to Dmitry:
> 1. use plane_state->dst instead of crtc_h/w/x/y.
> 2. delete duplicate framebuffer's atomic_check.
> 3. use FIELD_PREP() directly.
> 4. use dev->mode_config.
> 5. delete unnecessary drm_atomic_helper_shutdown call
> 6. add AUTHOR
> 7. using .enable instead
Please read Documentation/process/submitting-patches.rst.
>
> Chu Guangqing (1):
> [DRIVER] gpu: drm: add support for Yhgc ZX1000 soc chipset
>
> MAINTAINERS | 5 +
> drivers/gpu/drm/Kconfig | 2 +
> drivers/gpu/drm/Makefile | 1 +
> drivers/gpu/drm/yhgch/yhgch-drm/Kconfig | 11 +
> drivers/gpu/drm/yhgch/yhgch-drm/Makefile | 4 +
> .../gpu/drm/yhgch/yhgch-drm/yhgch_drm_de.c | 423 ++++++++++++++++++
> .../gpu/drm/yhgch/yhgch-drm/yhgch_drm_drv.c | 310 +++++++++++++
> .../gpu/drm/yhgch/yhgch-drm/yhgch_drm_drv.h | 51 +++
> .../gpu/drm/yhgch/yhgch-drm/yhgch_drm_i2c.c | 114 +++++
> .../gpu/drm/yhgch/yhgch-drm/yhgch_drm_regs.h | 208 +++++++++
> .../gpu/drm/yhgch/yhgch-drm/yhgch_drm_vdac.c | 123 +++++
> 11 files changed, 1252 insertions(+)
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/Kconfig
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/Makefile
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/yhgch_drm_de.c
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/yhgch_drm_drv.c
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/yhgch_drm_drv.h
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/yhgch_drm_i2c.c
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/yhgch_drm_regs.h
> create mode 100644 drivers/gpu/drm/yhgch/yhgch-drm/yhgch_drm_vdac.c
>
> --
> 2.43.5
>
--
With best wishes
Dmitry