Hi Andrew, Thank you for comments.
On Thu, Aug 16, 2012 at 8:29 PM, Andrew Dyer <amd...@gmail.com> wrote: > I don't have any comment on the body of the patch, but calling this an i2s > patch is misleading. I2s is a generic standard for moving stereo audio > around, and many chips support it. > > This patch looks like i2s support for a specific codec chip and the > filenames should reflect that. > In this patch we can support different codecs without doing any changes to i2s files. The only change would be in sound.c, where you have to check for specific codec required and set values accordingly. We use I2S1 here, but it is not specific to any particular codec. > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > Regards, Rajeshwari Shinde. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
