[U-Boot] [PATCH v2 10/10] mx23_olinuxino: Add ethernet support

[Marek Vasut]

Dear Otavio Salvador,

> On Wed, Feb 13, 2013 at 7:36 PM, Marek Vasut <marex at denx.de> wrote:
> > Dear Otavio Salvador,
> > 
> >> On Wed, Feb 13, 2013 at 7:19 PM, Marek Vasut <marex at denx.de> wrote:
> >> > Dear Otavio Salvador,
> >> > 
> >> >> This adds support to the LAN9512 chip included in the board and
> >> >> extend the environment to easy netboot use.
> >> >> 
> >> >> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> >> > 
> >> > You completely ignored previous discussion to make this ethernet
> >> > thingie part of the previous patch.
> >> 
> >> I made the HUB part of the previous patch; I don't agree in moving
> >> ethernet to there.
> > 
> > The ethernet is integral part of the hub chip. This approach you take
> > here breaks bisectability.
> 
> It does not; in case you do bisect you'll be able to build and run
> both revisions; it'll be a matter of one more 'bad' command call. I
> prefer to keep it split as it is much easier for someone reading the
> history to stop what has been done.

Yes, you're adding only part of the support in one patch and the other part in 
another patch. It makes no sense to me.

Best regards,
Marek Vasut