答复: [v2] drm: xlnx: zynqmp_dpsub: Add NULL checks in zynqmp_audio_init()

[Charles Han(韩春超)]

Thank you for your suggestions. I will read these documents carefully.

-----邮件原件-----
发件人: Markus Elfring <Markus.Elfring at web.de> 
发送时间: 2025年2月11日 15:56
收件人: Charles Han(韩春超) <hanchunchao at inspur.com>; dri-devel at lists.freedesktop.org; linux-arm-kernel at lists.infradead.org
抄送: LKML <linux-kernel at vger.kernel.org>; David Airlie <airlied at gmail.com>; Laurent Pinchart <laurent.pinchart at ideasonboard.com>; Maarten Lankhorst <maarten.lankhorst at linux.intel.com>; Maxime Ripard <mripard at kernel.org>; Michal Simek <michal.simek at amd.com>; Simona Vetter <simona at ffwll.ch>; Thomas Zimmermann <tzimmermann at suse.de>; Tomi Valkeinen <tomi.valkeinen at ideasonboard.com>; Vishal Sagar <vishal.sagar at amd.com>
主题: Re: [v2] drm: xlnx: zynqmp_dpsub: Add NULL checks in zynqmp_audio_init()

…
> Add NULL check in zynqmp_audio_init(), to handle kernel NULL pointer 
> dereference error.

* Can it be more desirable to prevent such issues?

* Would the message subject have been nicer with a key word like “PATCH”
  (besides other refinements)?


…
> ---
>  drivers/gpu/drm/xlnx/zynqmp_dp_audio.c | 4 ++++
…

Will you become more familiar with patch version descriptions?
https://lore.kernel.org/all/?q=%22This+looks+like+a+new+version+of+a+previously+submitted+patch%22
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.14-rc2#n310

Regards,
Markus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3830 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20250211/76bfc1e7/attachment-0001.bin>