kernel/git/stable/linux.git/drivers/soundwire/bus.c, branch linux-5.2.y

soundwire: avoid multiple assignments

2019-05-02T15:17:51+00:00

Modify the code to avoid multiple assignments by assigning to variable after error checks in soundwire bus. CHECK: multiple assignments should be avoided Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: more alignment fixes

2019-05-02T15:17:51+00:00

Found few more issues reported checkpatch on code alignment so fix those as well in the soundwire core. Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: remove empty line before/after braces

2019-05-02T15:17:50+00:00

Linux code style doesn't expect empty lines before or after braces and gives warning: CHECK: Blank lines aren't necessary after an open brace '{' CHECK: Blank lines aren't necessary before a close brace '}' Fix these instances in soundwire core Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: add missing newlines in dynamic debug logs

2019-05-02T15:16:29+00:00

For some reason the newlines are not used everywhere. Fix as needed. Reported-by: Joe Perches Reviewed-by: Takashi Iwai Signed-off-by: Pierre-Louis Bossart Signed-off-by: Greg Kroah-Hartman

soundwire: bus: remove spurious newline

2019-05-02T15:16:27+00:00

tools complain here and the location of the newline does not improve readability. Reviewed-by: Takashi Iwai Signed-off-by: Pierre-Louis Bossart Signed-off-by: Greg Kroah-Hartman

soundwire: bus: fix boolean comparisons

2019-05-02T15:16:27+00:00

no need for an explicit test against false reported by Coccinelle Reviewed-by: Takashi Iwai Signed-off-by: Pierre-Louis Bossart Signed-off-by: Greg Kroah-Hartman

soundwire: bus: remove useless parentheses

2019-05-02T15:16:27+00:00

and make the code more readable Reviewed-by: Takashi Iwai Signed-off-by: Pierre-Louis Bossart Signed-off-by: Greg Kroah-Hartman

soundwire: bus: fix typos in comments

2019-05-02T15:16:27+00:00

spelling mistakes Reviewed-by: Takashi Iwai Signed-off-by: Pierre-Louis Bossart Signed-off-by: Greg Kroah-Hartman

soundwire: bus: fix alignment issues

2019-05-02T15:16:27+00:00

Use Linux style Reviewed-by: Takashi Iwai Signed-off-by: Pierre-Louis Bossart Signed-off-by: Greg Kroah-Hartman

soundwire: remove useless initializations

2019-04-14T10:03:26+00:00

No need for explicit initialization of page and ssp fields, they are already zeroed with a memset. Detected with cppcheck: [drivers/soundwire/bus.c:309]: (style) Variable 'msg->page' is reassigned a value before the old one has been used. Signed-off-by: Pierre-Louis Bossart Reviewed-by: Takashi Iwai Signed-off-by: Vinod Koul