linux-stable.git/drivers/soundwire, branch v5.2.17

soundwire: cadence_master: fix definitions for INTSTAT0/1

2019-09-06T08:23:16+00:00

[ Upstream commit 664b16589f882202b8fa8149d0074f3159bade76 ]

Two off-by-one errors: INTSTAT0 missed BIT(31) and INTSTAT1 is only
defined on first 16 bits.

Signed-off-by: Pierre-Louis Bossart 
Link: https://lore.kernel.org/r/20190725234032.21152-15-pierre-louis.bossart@linux.intel.com
Signed-off-by: Vinod Koul 
Signed-off-by: Sasha Levin

soundwire: cadence_master: fix register definition for SLAVE_STATE

2019-09-06T08:23:16+00:00

[ Upstream commit b07dd9b400981f487940a4d84292d3a0e7cd9362 ]

wrong prefix and wrong macro.

Signed-off-by: Pierre-Louis Bossart 
Link: https://lore.kernel.org/r/20190725234032.21152-14-pierre-louis.bossart@linux.intel.com
Signed-off-by: Vinod Koul 
Signed-off-by: Sasha Levin

Merge tag 'soundwire-5.2-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/soundwire into char-misc-linus

2019-06-10T16:07:39+00:00

Vinod writes:

soundwire fixes for v5.2-rc4

Srinivas Kandagatla fixed by bunch of issues, two in core for locking
and out of bound access and one in intel driver copy-paste

* tag 'soundwire-5.2-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/soundwire:
  soundwire: intel: set dai min and max channels correctly
  soundwire: stream: fix bad unlock balance
  soundwire: stream: fix out of boundary access on port properties

soundwire: intel: set dai min and max channels correctly

2019-06-06T17:18:41+00:00

Looks like there is a copy paste error.
This patch fixes it!

Signed-off-by: Srinivas Kandagatla 
Signed-off-by: Vinod Koul

soundwire: stream: fix bad unlock balance

2019-06-06T17:16:04+00:00

the msg lock is taken for multi-link cases only but released
unconditionally, leading to an unlock balance warning for single-link usages
This patch fixes this.

 =====================================
 WARNING: bad unlock balance detected!
 5.1.0-16506-gc1c383a6f0a2-dirty #1523 Tainted: G        W
 -------------------------------------
 aplay/2954 is trying to release lock (&bus->msg_lock) at:
 do_bank_switch+0x21c/0x480
 but there are no more locks to release!

Signed-off-by: Srinivas Kandagatla 
Acked-by: Pierre-Louis Bossart 
Acked-by: Sanyog Kale 
[vkoul: edited the change log as suggested by Pierre]
Signed-off-by: Vinod Koul

soundwire: stream: fix out of boundary access on port properties

2019-05-27T05:04:12+00:00

Assigning local iterator to array element and using it again for
indexing would cross the array boundary.
Fix this by directly referring array element without using the local
variable.

Signed-off-by: Srinivas Kandagatla 
Acked-by: Pierre-Louis Bossart 
Signed-off-by: Vinod Koul

treewide: Add SPDX license identifier - Makefile/Kconfig

2019-05-21T08:50:46+00:00

Add SPDX license identifiers to all Make/Kconfig files which:

 - Have no license information of any form

These files fall under the project license, GPL v2 only. The resulting SPDX
license identifier is:

  GPL-2.0-only

Signed-off-by: Thomas Gleixner 
Signed-off-by: Greg Kroah-Hartman

soundwire: add a blank line between functions

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

For improving code readability it helps to have a blank line between
function so add when missing.

Checkpatch complains:
CHECK: Please use a blank line after function/struct/union/enum
declarations

Signed-off-by: Vinod Koul 
Signed-off-by: Greg Kroah-Hartman

soundwire: fix more typos

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

Found few more typos in the code, fix them

CHECK: 'and and' may be misspelled - perhaps 'and'?
CHECK: 'smaple' may be misspelled - perhaps 'sample'?

Signed-off-by: Vinod Koul 
Signed-off-by: Greg Kroah-Hartman

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