From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jeroen Hofstee <jeroen@myspectrum.nl>
Date: Wed, 15 Oct 2014 09:42:41 +0200
Subject: [U-Boot] [PATCH 2/2] video: ipu_disp: remove pixclk fixup
In-Reply-To: <1413311835-22715-2-git-send-email-jeroen@myspectrum.nl>
References: <1413311835-22715-1-git-send-email-jeroen@myspectrum.nl>
	<1413311835-22715-2-git-send-email-jeroen@myspectrum.nl>
Message-ID: <543E2571.1030009@myspectrum.nl>
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

Hello Stefano,

On 14-10-14 20:37, Jeroen Hofstee wrote:
> The ipu display insists on having a lower_margin smaller
> then 2. If this is not the case it will attempt to force
> it and adjust the pixclk accordingly. This multiplies pixclk
> in Hz with the width and height, since this is typically
> a * 10^7 * b * 10^2 * c * 10^2 this will overflow the
> uint_32 and make things even worse. Since this is a
> bootloader and the adjustment is neglectible, just force
> it to two and warn about it.
>
> Cc: Stefano Babic <sbabic@denx.de>
> Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl

The lower_margin should be larger then 1 / _not_
smalller then 2 actually.

Regards,
Jeroen