Re: [PATCH 1/4] Remove redundant return value check

Uwe Kleine-König
2023-11-02T15:47:19Z

[-- Attachment #1: Type: text/plain, Size: 960 bytes --]

Hello Alexandra,

it would be great if you could improve the Subject line for the next
patch submission round by using an appropriate prefix. Currently at
least three of the four patches in this series have "Remove redundant
return value check" as Subject/commit short log. (I didn't get patch
3/4.) Something like

media: atomisp: i2c: Remove redundant return value check

seems to be appropriate for this patch.

You make life easier for the recipients of your patch because it becomes
easier to see if you're responsible for this patch or not. So you're
wasting less time for people like me who have no further interest in
this patch set and the maintainers of the affected drivers immediately
see what was touched. That's a win for everybody; including you.

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]