[PATCHv2 1/8] ALSA: Add SAI SoC Digital Audio Interface driver.
Li Xiubo
Li.Xiubo at freescale.com
Tue Nov 19 23:16:09 EST 2013
> > > The udelay just doesn't make sense to what you are talking about.
> > >
> > > Does SAI really need 10us delay between two register-updating?
> > >
> >
> > No, this is not must be.
>
> Then you should explain in your comments why you really put it here or
> just drop it if it's just a mistake.
>
The udelay will be removed then.
> > > ISSUE 2: There are two cases listed in 'Synchronous mode'.
> > > However, your driver doesn't take care of them.
> > > The SAI's synchronous mode looks like more flexible
> > > than SSI's. The driver needs to be more sophisticated
> > > so that it can handle multiple cases when TX/RX clocks
> > > are controlled by either TX or RX, and surely, the
> > > asynchronous mode as well.
> > >
> >
> > Because in Vybrid the transmitter bit clock and frame sync are to be
> > used by both the transmitter and receiver, and only this case can be
> > used here, so now I only handle this case.
>
> It's fairly okay if adding explicit comments to indicate that currently
> the driver only supports its Synchronous mode with clocks controlled by
> TX only.
>
Just think, on other platforms maybe only the Rx's clock is available.
Thus I think there should be one DT property to control this, and then the SAI driver can be more flexible.
Or could you give me some more practical ideas ?
--
Best Regards,
Xiubo
More information about the linux-arm-kernel
mailing list