mbox series

[v3,0/5] soundwire: intel/cadence: better initialization

Message ID 20191022235448.17586-1-pierre-louis.bossart@linux.intel.com (mailing list archive)
Headers show
Series soundwire: intel/cadence: better initialization | expand

Message

Pierre-Louis Bossart Oct. 22, 2019, 11:54 p.m. UTC 
Changes since v2: addressed feedback from Vinod Koul on patch 2&4
Add kernel taint when using debugfs hw_reset (similar to regmap)
Remove useless goto label

Changes since v1: addressed feedback from Vinod Koul
clarified init changes impact Intel and Cadence sides
remove unnecessary intermediate variable
disable interrupts when exit_reset fails, updated error handling
returned -EINVAL on debugfs invalid parameter

Pierre-Louis Bossart (5):
  soundwire: intel/cadence: fix startup sequence
  soundwire: cadence_master: add hw_reset capability in debugfs
  soundwire: intel: add helper for initialization
  soundwire: intel/cadence: add flag for interrupt enable
  soundwire: cadence_master: make clock stop exit configurable on init

 drivers/soundwire/cadence_master.c | 134 +++++++++++++++++++++--------
 drivers/soundwire/cadence_master.h |   5 +-
 drivers/soundwire/intel.c          |  39 ++++++---
 3 files changed, 129 insertions(+), 49 deletions(-)

Comments

Vinod Koul Oct. 24, 2019, 11:52 a.m. UTC | #1 
On 22-10-19, 18:54, Pierre-Louis Bossart wrote:
> Changes since v2: addressed feedback from Vinod Koul on patch 2&4
> Add kernel taint when using debugfs hw_reset (similar to regmap)
> Remove useless goto label
> 
> Changes since v1: addressed feedback from Vinod Koul
> clarified init changes impact Intel and Cadence sides
> remove unnecessary intermediate variable
> disable interrupts when exit_reset fails, updated error handling
> returned -EINVAL on debugfs invalid parameter

Applied, thanks

> 
> Pierre-Louis Bossart (5):
>   soundwire: intel/cadence: fix startup sequence
>   soundwire: cadence_master: add hw_reset capability in debugfs
>   soundwire: intel: add helper for initialization
>   soundwire: intel/cadence: add flag for interrupt enable
>   soundwire: cadence_master: make clock stop exit configurable on init
> 
>  drivers/soundwire/cadence_master.c | 134 +++++++++++++++++++++--------
>  drivers/soundwire/cadence_master.h |   5 +-
>  drivers/soundwire/intel.c          |  39 ++++++---
>  3 files changed, 129 insertions(+), 49 deletions(-)
> 
> -- 
> 2.20.1