Re: [PATCH 0/2] staging: greybus: fix warnings reported by checkpatch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On 4/11/22 8:50 PM, Jaehee Park wrote:
The first patch corrects a typo in a comment. The second patch fixes
'void function return statements are not generally useful' warning.

Jaehee Park (2):
   staging: greybus: correct typo in comment 'Atleast' to 'At least'
   staging: greybus: remove return in an empty void function

  drivers/staging/greybus/arche-apb-ctrl.c | 2 +-
  drivers/staging/greybus/audio_codec.c    | 1 -
  2 files changed, 1 insertion(+), 2 deletions(-)

These are really trivial patches.  Whether the second
one is even worth changing is questionable (it represents
"code churn" without really adding value).

That said, you've properly submitted the patches and I
have no problem with either one.  So someone else can
object, but if they do not:

Reviewed-by: Alex Elder <elder@xxxxxxxxxx>
greybus-dev mailing list -- greybus-dev@xxxxxxxxxxxxxxxx
To unsubscribe send an email to greybus-dev-leave@xxxxxxxxxxxxxxxx

[Index of Archives]     [Asterisk App Development]     [PJ SIP]     [Gnu Gatekeeper]     [IETF Sipping]     [Info Cyrus]     [ALSA User]     [Fedora Linux Users]     [Linux SCTP]     [DCCP]     [Gimp]     [Yosemite News]     [Deep Creek Hot Springs]     [Yosemite Campsites]     [ISDN Cause Codes]     [Asterisk Books]

  Powered by Linux