From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@deeprootsystems.com (Kevin Hilman) Date: Fri, 27 Aug 2010 11:05:03 -0700 Subject: [PATCH v4] davinci: Add MityDSP-L138/MityARM-1808 SOM support In-Reply-To: <4C77E6BD.4000303@criticallink.com> (Michael Williamson's message of "Fri, 27 Aug 2010 12:24:29 -0400") References: <4C516634.5040804@criticallink.com> <877hjd106u.fsf@deeprootsystems.com> <4C7702EB.8040905@criticallink.com> <8762ywrvpt.fsf@deeprootsystems.com> <4C77E6BD.4000303@criticallink.com> Message-ID: <87fwy0orsg.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Michael Williamson writes: > On 08/27/2010 10:11 AM, Kevin Hilman wrote: >> Michael Williamson writes: >> >> >>>> Normally, I like to see new board support broken down more. It's rather >>>> difficult to do a good review of new board file when everything is added >>>> in a single patch. >>>> >>>> Typically, a basic patch that just supports basic boot (typically to >>>> UART console) is the first patch. Then additional patches are added to >>>> add peripheral support (display, MMC, SPI, flash, regulators, ...) >>>> >>>> Breaking things up this way helps reviewers and maintainers greatly. >>>> >>>> [...] >>>> >>>> >>> OK. Most of the board file is a large cut-and-paste of the da850 >>> EVM. It seems unlikely that I'll have any measure of success here, but if >>> I get time I might try again. >>> >> OK, if it's mostly a cut-and-paste of da850, then maybe just break it up >> into two patches, 1) the cut, paste& rename patch and 2) a separate >> patch for things that are different on your board. >> >> Thanks, >> >> Kevin >> > > I think I am going to try again. I intend to basically start over, leaving > the console UART, NAND, and EMAC (hardcoded to MII interface for now). > This is the minimum set of devices I need to get one of our base boards to > boot and mount a root filesystem and run some tests here. > > I will remove all the ATAG garbage and anything that is runtime configurable > as well as the LCD, McASP, SPI, SPI-NOR, and I2C based peripherals. If the > patch is OK, then I'll add them back one by one for review. > > Would this be a reasonable approach? Yes, perfect! > And should I post it as a v5 to this > series or create a different series (given the large change of code)? A new series is fine. > Should I pull the general ARM mailing list off the distribution as > this should now be specific only to mach-davinci? Keeping linux-arm-kernel in copy is good. Kevin