From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefano Babic <sbabic@denx.de>
Date: Wed, 22 Oct 2014 16:23:16 +0200
Subject: [U-Boot] [PATCH 1/2] arm: mx6: add support for TBS2910 Matrix
	ARM miniPC
In-Reply-To: <5447A01D.5020906@web.de>
References: <1413630834-3509-1-git-send-email-smoch@web.de>
	<5446462C.3090901@denx.de> <54469DF0.3030703@web.de>
	<54476C83.5090904@denx.de> <5447A01D.5020906@web.de>
Message-ID: <5447BDD4.40306@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

Hi Soeren,

On 22/10/2014 14:16, Soeren Moch wrote:

> U-Boot without video support is useless for me. So I will try to
> setup the video clock correctly in a new version of this patch.

ok, fine with me.

> You are right here, there is no problem with setup_display() in
> board_init().
> 
> So I wonder why all imx6q boards have setup_display() in
> board_early_init(),
> you even committed such code for mx6qsabreauto yesterday.

Well, after each review a code reveals new aspects.

Anyway, this is not a big issue, but setting the display is not one
thing that must be done very early.

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================