Change in coreboot[master]: drivers/i2c/at24rf08c/lenovo_serials.c: Remove comparison to NULL - coreboot-gerrit

15 Feb 2022


      Attention is currently required from: Elyes Haouas.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61967 )
Change subject: drivers/i2c/at24rf08c/lenovo_serials.c: Remove comparison to NULL
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/61967/comment/e1a35f27_4c751649 
PS1, Line 9: This makes the code more readable and makes util/lint/checkpatch.pl happy.
IMO this is a style choice that should be left to the author. What
is the message you see from checkpatch? I checked this out and then
added a `dev == NULL` back, tried to commit and checkpatch didn't
complain.
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/61967
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ice2d91e472cea5e8a1774aacb301468c6870f909
Gerrit-Change-Number: 61967
Gerrit-PatchSet: 1
Gerrit-Owner: Elyes Haouas ehaouas@noos.fr
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Nico Huber nico.h@gmx.de
Gerrit-Attention: Elyes Haouas ehaouas@noos.fr
Gerrit-Comment-Date: Tue, 15 Feb 2022 12:37:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment