Thread (20 messages) 20 messages, 4 authors, 2020-10-30

Re: [PATCH v9 01/14] ASoC: sun4i-i2s: Change set_chan_cfg() params

From: Pierre-Louis Bossart <hidden>
Date: 2020-10-27 18:59:31
Also in: alsa-devel, linux-devicetree, lkml

quoted hunk ↗ jump to hunk
@@ -452,11 +454,11 @@ static int sun8i_i2s_set_chan_cfg(const struct sun4i_i2s *i2s,
  	case SND_SOC_DAIFMT_DSP_B:
  	case SND_SOC_DAIFMT_LEFT_J:
  	case SND_SOC_DAIFMT_RIGHT_J:
-		lrck_period = params_physical_width(params) * slots;
+		lrck_period = slot_width * slots;
  		break;
  
  	case SND_SOC_DAIFMT_I2S:
-		lrck_period = params_physical_width(params);
+		lrck_period = slot_width;
  		break;
Aren't I2S, LEFT_J and RIGHT_J pretty much the same in terms of lrclk 
rate/period? the only thing that can change is the polarity, no?

Not sure why it's handled differently here?

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help