On 12/21/15 08:41, Shawn Guo wrote: > On Wed, Dec 16, 2015 at 04:09:02PM +0200, Igor Grinberg wrote: >> Hi Shawn, >> >> On 12/14/15 04:47, Shawn Guo wrote: >>> On Mon, Dec 07, 2015 at 09:14:31AM +0200, Ilya Ledvich wrote: >>>> Ilya Ledvich (8): >>>> ARM: dts: imx7d: cl-som-imx7: add basic module support >>>> ARM: dts: imx7d: cl-som-imx7: add usb otg support >>>> ARM: dts: imx7d: cl-som-imx7: add eMMC support >>>> ARM: dts: imx7d: cl-som-imx7: add fec1 support >>>> ARM: dts: imx7d: cl-som-imx7: add eeprom support >>>> ARM: dts: imx7d: cl-som-imx7: add gpio extender support >>>> ARM: dts: imx7d: cl-som-imx7: add fec2 support >>> >>> For the initial board support submission, it doesn't really need such >>> splitting. Please merge above patches into one. >> >> Yeah, I know there is no need for such splitting, yet I think >> if the splitting is done right - it is much simpler to review, > > Not really for a initial board support. > >> it provides a better bisectability (in case it will be needed in >> the future), > > This is an initial board support submission and all the devices added > there should be functional. The bisectability shouldn't needed in this > case. > >> and also if for some reason one (or more) patches are >> needing some time consuming rework, others (earlier ones in the series) >> can be accepted meanwhile. > > It's not really the case for this series. I'm glad you think so :-) > >> >> So, I really like to keep it this way, unless you insist. > > Please merge them. Ok. -- Regards, Igor. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html