Hello Greg, I've run the checkpatch.pl script on audio_code.c:
./scripts/checkpatch.pl --file --terse drivers/staging/greybus/audio_codec.c
and got the following warning:
drivers/staging/greybus/audio_codec.c:1032: WARNING: void function return statements are not generally useful
As Brian wrote, we can either remove the return or leave it as is, so my "changes" can be ignored. This is not a coding style issue and I should have been more clear on the commit message. I apologize for the mistake.
Thank you and @Brian for taking the time to reply. Kind regards, Senthu Am 28.10.2021 18:42, schrieb Greg KH:
On Thu, Oct 28, 2021 at 05:38:08PM +0200, Senthu Sivarajah wrote:From: Senthuran Sivarajah <ssivaraj@xxxxxxxxxxxxxxx> Fixed a coding style issue.What coding style exactly?Signed-off-by: Senthuran Sivarajah <ssivaraj@xxxxxxxxxxxxxxx> --- drivers/staging/greybus/audio_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.cindex b589cf6b1d03..9f99862791dc 100644 --- a/drivers/staging/greybus/audio_codec.c +++ b/drivers/staging/greybus/audio_codec.c@@ -1028,7 +1028,7 @@ static int gbcodec_probe(struct snd_soc_component *comp)static void gbcodec_remove(struct snd_soc_component *comp) { /* Empty function for now */ - return; + return 0;Did you build this change? Exactly what tool reported a problem with this code and what was the exact error? thanks, greg k-h
_______________________________________________ greybus-dev mailing list greybus-dev@xxxxxxxxxxxxxxxx https://lists.linaro.org/mailman/listinfo/greybus-dev
|