On Tue, 2008-09-02 at 21:31 +0100, Mark Brown wrote: > On Tue, Sep 02, 2008 at 09:08:09PM +0100, Liam Girdwood wrote: > > On Tue, 2008-09-02 at 19:54 +0100, Mark Brown wrote: > > > > OK, a comment to that effect somewhere (perhaps in set_sysclk() which > > > only accepts 26MHz) probably wouldn't go amiss. > > > The codec set_sysclk() should also accept 19.2MHz and 38.4MHz otherwise > > return -EINVAL. It should also have a nice comment too :) > > Ideally, yes but it looked like that'd need additional code to set the > codec up to use them. Obviously the ideal thing would be if the driver > supports everything the chip does. Then this should also be marked as a TODO. The likely users of the SYSCLK value are the codecs DSP core (for audio filtering) and codec DAI mastering. Liam -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
