… > 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