Re: [Sound-open-firmware] [PATCH] cnl: ssp: fix DSP_B setting in ssp

On 3/6/18 10:31 PM, Rander Wang wrote:

set ssp according to DSP_B spec

Signed-off-by: Rander Wang rander.wang@linux.intel.com

src/drivers/apl-ssp.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/drivers/apl-ssp.c b/src/drivers/apl-ssp.c index a726f3c..c32bd35 100644 --- a/src/drivers/apl-ssp.c +++ b/src/drivers/apl-ssp.c @@ -333,10 +333,11 @@ static inline int ssp_set_config(struct dai *dai, sscr0 |= SSCR0_MOD | SSCR0_FRDC(config->num_slots);

/* set asserted frame length */

• frame_len = config->sample_container_bits;
  

• frame_len = 1;
  

this needs to be done for DSP_B as well.

/* handle frame polarity, DSP_A default is rising/active high */

• sspsp |= SSPSP_SFRMP(inverted_frame);
  

• sspsp |= SSPSP_SFRMP(!inverted_frame);
  

I don't think this is correct. the documentation says with this bit asserted (value=1) the frame is active high which is exactly what we want.

• sspsp |= SSPSP_FSRT;
  

why is this necessary? this disables a fix for dummy stop/padding.

break;

default: