28 Sep
2020
28 Sep
'20
10:55 a.m.
On Mon, Sep 21, 2020 at 07:15:13PM +0200, Clément Péron wrote:
Hi Maxime,
On Mon, 21 Sep 2020 at 14:29, Maxime Ripard maxime@cerno.tech wrote:
On Mon, Sep 21, 2020 at 12:27:11PM +0200, Clément Péron wrote:
As slots and slot_width can be overwritter in case set_tdm() is called. Avoid to have this logic in set_chan_cfg().
Instead pass the required values as params to set_chan_cfg().
It's not really clear here what the issue is, and how passing the slots and slot_width as arguments addresses it
This also fix a bug when i2s->slot_width is set for TDM but not properly used in set_chan_cfg().
Which bug?
Do you mean my commit log is too short or is it a real question to understand ?
Both, actually :)
Maxime