kernel/linux.git/drivers/soundwire, branch v4.17.1

soundwire: Fix a signedness bug

2018-01-22T15:45:26+00:00

"ret" is an int and "buf" is a u8. sdw_read() returns negative error codes which are truncated to the u8, 0-255 range before being stored as an int. It means that "ret" can't be less than zero. Fixes: b0a9c37b0178 ("soundwire: Add slave status handling") Signed-off-by: Dan Carpenter Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: select REGMAP_SOUNDWIRE

2018-01-09T16:01:16+00:00

SoundWire bus needs to select the regmap support. So, add it Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: Fix incorrect return value check

2018-01-09T16:01:16+00:00

pm_runtime_get_sync will return negative values for error and 0 or 1 for success. Update the error check accordingly. Signed-off-by: Shreyas NC Reported-by: Dan Carpenter Fixes: 9d715fa005eb: ("soundwire: Add IO transfer") Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: Fix typo in return value check of sdw_read()

2018-01-08T16:53:21+00:00

Fix the typo, 'status' should be instead of 'status2'. Fixes: b0a9c37b0178 ("soundwire: Add slave status handling") Signed-off-by: Wei Yongjun Acked-by: Pierre-Louis Bossart Acked-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: intel: fix missing assignment to ret

2018-01-08T16:53:21+00:00

Currently the return status ret is being checked but it has not been updated since the previous check on ret. It appears that assignment of ret from return status of the call to sdw_cdns_enable_interrupt was accidentally ommited. Fix this. Detected by CoverityScan, CID#1463148 ("Logically dead code") Fixes: 71bb8a1b059e ("soundwire: intel: Add Intel Master driver") Signed-off-by: Colin Ian King Acked-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: fix sign extension when shifting buf[2] 24 places

2018-01-08T16:53:20+00:00

The buf[2] left shift by 24 bits is promoted to int (32 bit signed) and then signed-extended to unsigned long long. Hence if the upper bit to buf[2] is set then all the upper bits of addr end up as 1. Fix this by casting it to u64 before shifting it. Also replace the unsigned long long casts to u64 casts to match the same type of addr. Detected by CoverityScan, CID#1463147 ("Unintended sign extension") Fixes: d52d7a1be02c ("soundwire: Add Slave status handling helpers") Signed-off-by: Colin Ian King Acked-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: intel: Add Intel init module

2017-12-19T10:32:35+00:00

The SoundWire Master is implemented as part of Audio controller in Intel platforms. Add a init module which creates SoundWire Master platform devices based on the links supported in the hardware. Signed-off-by: Sanyog Kale Reviewed-by: Philippe Ombredanne Acked-By: Pierre-Louis Bossart Reviewed-by: Takashi Iwai Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: intel: Add Intel Master driver

2017-12-19T10:32:02+00:00

Some Intel platforms have SoundWire Master, so add Intel SoundWire Master driver which uses Cadence module. This patch adds probe and initialization routines for Intel Master driver. Signed-off-by: Hardik T Shah Signed-off-by: Sanyog Kale Reviewed-by: Philippe Ombredanne Acked-By: Pierre-Louis Bossart Reviewed-by: Takashi Iwai Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: cdns: Add sdw_master_ops and IO transfer support

2017-12-19T10:31:33+00:00

Implement sdw_master_ops with support for xfer_msg, xfer_msg_defer and reset_page_addr. Since Cadence module doesn't know the systems it will be used, set the read_prop to the bus helper. Signed-off-by: Hardik T Shah Signed-off-by: Sanyog Kale Reviewed-by: Philippe Ombredanne Acked-By: Pierre-Louis Bossart Reviewed-by: Takashi Iwai Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman

soundwire: cdns: Add cadence library

2017-12-19T10:31:00+00:00

Cadence IP implements SoundWire Master. Add base cadence library initialization and interrupt handling Signed-off-by: Hardik T Shah Signed-off-by: Sanyog Kale Reviewed-by: Philippe Ombredanne Acked-By: Pierre-Louis Bossart Reviewed-by: Takashi Iwai Signed-off-by: Vinod Koul Signed-off-by: Greg Kroah-Hartman