From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Tue, 2 Feb 2016 21:26:28 +0100 Subject: [U-Boot] [PATCH 2/2] imx: mx6ul/sx: Fix issue in LCDIF clock dividers calculation In-Reply-To: <1453816918-25383-2-git-send-email-ye.li@nxp.com> References: <1453816918-25383-1-git-send-email-ye.li@nxp.com> <1453816918-25383-2-git-send-email-ye.li@nxp.com> Message-ID: <56B110F4.80901@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 26/01/2016 15:01, Ye Li wrote: > The checking with max frequency supported is not correct, because the temp > is calculated by max pre and post dividers. We can decrease any divider to > meet the max frequency limitation. Actually, the calculation below the codes > is doing this way to find best pre and post dividers. > > Signed-off-by: Ye Li > --- > arch/arm/cpu/armv7/mx6/clock.c | 4 ---- > 1 files changed, 0 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/cpu/armv7/mx6/clock.c b/arch/arm/cpu/armv7/mx6/clock.c > index 007204d..88380a6 100644 > --- a/arch/arm/cpu/armv7/mx6/clock.c > +++ b/arch/arm/cpu/armv7/mx6/clock.c > @@ -638,10 +638,6 @@ void mxs_set_lcdclk(u32 base_addr, u32 freq) > } > > temp = freq * max_pred * max_postd; > - if (temp > max) { > - puts("Please decrease freq, too large!\n"); > - return; > - } > if (temp < min) { > /* > * Register: PLL_VIDEO > Applied to u-boot-imx, thanks ! Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk 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 =====================================================================