From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Mon, 4 Aug 2014 10:27:38 +0200
Subject: [U-Boot] [PATCH 18/18] arm: mx6: cm_fx6: add sata support
In-Reply-To: <53DF34E0.7000908@compulab.co.il>
References: <1407051288-17324-1-git-send-email-nikita@compulab.co.il>
	<201408031610.34341.marex@denx.de>
	<53DF34E0.7000908@compulab.co.il>
Message-ID: <201408041027.38513.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Monday, August 04, 2014 at 09:23:12 AM, Igor Grinberg wrote:
> Hi Marek,
> 
> On 08/03/14 17:10, Marek Vasut wrote:
> > On Sunday, August 03, 2014 at 09:34:48 AM, Nikita Kiryanov wrote:
> >> Add support for SATA.
> >> 
> >> Cc: Igor Grinberg <grinberg@compulab.co.il>
> >> Cc: Stefano Babic <sbabic@denx.de>
> >> Cc: Tom Rini <trini@ti.com>
> >> Signed-off-by: Nikita Kiryanov <nikita@compulab.co.il>
> > 
> > I'd just squash all the cf_mx6 patches into the "add cf_mx6 board" patch.
> 
> Well, I tend to disagree on this.
> I find extremely hard to review the "long long squashed into one patch"
> patches. It is much easier to review small, functionality oriented,
> patches.

You do have a valid point when it comes to review, all right. There's no point 
in keeping them separate when they're applied though. Tough decisions this is. 
Let's see what others think then .

Best regards,
Marek Vasut