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>




[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux