From mboxrd@z Thu Jan 1 00:00:00 1970 From: Igor Grinberg Date: Tue, 10 Jan 2012 09:12:53 +0200 Subject: [U-Boot] [PATCH v4 5/5] ARM:AM33XX: Add SPL support for AM335X EVM In-Reply-To: <1326177539-19643-6-git-send-email-chandan.nath@ti.com> References: <1326177539-19643-1-git-send-email-chandan.nath@ti.com> <1326177539-19643-6-git-send-email-chandan.nath@ti.com> Message-ID: <4F0BE4F5.8000307@compulab.co.il> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Chandan, Tom, On 01/10/12 08:38, Chandan Nath wrote: > This patch is added to support SPL feature on AM335X > platform. In this patch, MMC1 is configured as boot > device for SPL and support for other devices will be > added in the next patch series. > > Signed-off-by: Chandan Nath > Signed-off-by: Tom Rini This version looks much better! The only thing got into my mind while looking at it is that I would split it to some more patches instead of one big patch, as it seems that you are mixing the generalization/cleanup with the actual addition of the SPL, otherwise Acked-by: Igor Grinberg -- Regards, Igor.