Hi Kevin, On 1/18/2011 1:07 PM, Kevin Hilman wrote:
> Hi Michael, > > Michael Williamson <[email protected]> writes: > >> The following patch series is an attempt to clean up unused and platform >> specific >> pinmux arrays in the da850 CPU files. This series was developed as a result >> of >> the following email thread: >> >> http://www.mail-archive.com/[email protected]/msg19921.html > > Thanks for taking this on. > > I noticed you only move changes to the EVM's board file. Won't this > break the other da850-based boards (your board and the hawk board?) > I don't think it breaks the other boards. I am building all three with the da8xx_omapl_defconfig. I can only test with mine, though... Neither my board nor the hawkboard use the arrays that are moved (or deleted) in this patch series. The hawkboard defines it's own MMC and McASP pin set in the most recent patch series that is queued. I haven't submitted MMC and McASP patches yet for the MityDSP SoMs. [FYI, there is a v1 of this series posted now, Sekhar pointed out a problem with the original series] -Mike > Kevin > >> This patch is against commit b411b51a71cd9c926712b33ab21d001ed7e57838 of >> linux-davinci. >> >> This series should not be applied until someone with a da850 EVM can please >> test >> and confirm the mcasp and mmc interfaces continue to work properly with >> these patches. >> >> I noticed that none of the boards initialize the pinmux settings for the >> UARTs. >> Apparently, they are all relying on the bootloader to do this. >> >> Michael Williamson (5): >> davinci: da850: remove unused cpgmac pinmux array >> davinci: da850: remove unused emif pinmux array >> davinci: da850: move da850_evm specific mcasp pins to board file. >> davinci: da850: move da850_evm specific mmcsd pinmux array to board >> file. >> davinci: da850: remove unused uart pinmux arrays. >> >> arch/arm/mach-davinci/board-da850-evm.c | 18 ++++++++- >> arch/arm/mach-davinci/da850.c | 56 >> ---------------------------- >> arch/arm/mach-davinci/include/mach/da8xx.h | 7 --- >> 3 files changed, 16 insertions(+), 65 deletions(-) >> >> _______________________________________________ >> Davinci-linux-open-source mailing list >> [email protected] >> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
