Martin Roth has posted comments on this change. ( https://review.coreboot.org/19258 )
Change subject: sb/intel/*: Use common SMBus functions ......................................................................
Patch Set 15:
(6 comments)
https://review.coreboot.org/#/c/19258/15/src/southbridge/intel/common/smbus.... File src/southbridge/intel/common/smbus.c:
PS15, Line 6: * Copyright (C) 2013 Vladimir Serbinenko I was going to comment on the copyright removal, but it looks like this is just an issue with git choosing which of the multiple deleted versions is the rename.
PS15, Line 96: { Braces aren't needed for a single line if. Same for all the following cases.
PS15, Line 176: Setup May as well fix it while we're here, i guess. /* Set up transaction */
https://review.coreboot.org/#/c/19258/15/src/southbridge/intel/common/smbus.... File src/southbridge/intel/common/smbus.h:
Line 3: * can we make sure this retains the correct copyrights if possible?
PS15, Line 14: SMBUS_H INTEL_COMMON_SMBUS_H maybe?
It won't actually conflict with anything, but we have a lot of smbus.h files in the tree, and it'd be a pain if they all just used SMBUS_H for the guards.
PS15, Line 32: Remove some extra lines?