On Sat, Jul 29, 2017 at 10:17 PM, <codekipper@xxxxxxxxx> wrote: > From: Marcus Cooper <codekipper@xxxxxxxxx> > > On newer SoCs the bit fields for the blck and lrclk polarity are in > a different locations. Use regmap fields to set the polarity bits > as intended. > > Signed-off-by: Marcus Cooper <codekipper@xxxxxxxxx> > --- > sound/soc/sunxi/sun4i-i2s.c | 45 ++++++++++++++++++++++++++++++++------------- > 1 file changed, 32 insertions(+), 13 deletions(-) > > diff --git a/sound/soc/sunxi/sun4i-i2s.c b/sound/soc/sunxi/sun4i-i2s.c > index 171df99a267e..90daa974bd27 100644 > --- a/sound/soc/sunxi/sun4i-i2s.c > +++ b/sound/soc/sunxi/sun4i-i2s.c > @@ -50,6 +50,8 @@ > #define SUN4I_I2S_FMT0_FMT_RIGHT_J (2 << 0) > #define SUN4I_I2S_FMT0_FMT_LEFT_J (1 << 0) > #define SUN4I_I2S_FMT0_FMT_I2S (0 << 0) > +#define SUN4I_I2S_FMT0_POLARITY_INVERTED (1) > +#define SUN4I_I2S_FMT0_POLARITY_NORMAL (0) > > #define SUN4I_I2S_FMT1_REG 0x08 > #define SUN4I_I2S_FIFO_TX_REG 0x0c > @@ -101,6 +103,8 @@ > * @fmt_offset: Value by which wss and sr needs to be adjusted. > * @field_fmt_set_wss: regmap field to set word select size. > * @field_fmt_set_sr: regmap field to set sample resolution. > + * @field_fmt_set_bclk_polarity: regmap field to set clk polarity. > + * @field_fmt_set_lrclk_polarity: regmap field to set frame polarity. You can cut these down to "field_fmt_{b,lr}clk_pol". It still makes sense in this context, and you can make the lines fit under 80 characters in the last hunk. ChenYu _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel