Re: [PATCH] drm/exynos/vidi: Remove redundant error handling in vidi_get_modes()

Markus Elfring
2025-03-07T14:49:16Z

…
> drm_edid_alloc() fails, the function will immediately return 0,
…

failed?

…
> the event of failure in these two functions, it is still necessary
> to call the subsequent drm_edid_connector_update() function with
…

You may occasionally put more than 66 characters into text lines
of such a change description.

> … To maintain the integrity of
> the operation, redundant error handling needs to be removed.

How do you think about to add any tags (like “Fixes” and “Cc”) accordingly?
https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.14-rc5#n145

Regards,
Markus