On 24/09/2012 17:11, Eric Nelson wrote:
> Sorry Stefano,
>
>>
>> Between V3 and V4 you changed also ipu_common.c, and that change does
>> not apply.
>>
>>> /* address of boot parameters */
>>> diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c
>>> index cc8f881..5f0a1a8 100644
>>> --- a/drivers/video/ipu_common.c
>>> +++ b/drivers/video/ipu_common.c
>>> @@ -218,7 +218,7 @@ static struct clk ipu_clk = {
>>> #else
>>> .enable_reg = (u32 *)(CCM_BASE_ADDR +
>>> offsetof(struct mxc_ccm_reg, CCGR3)),
>>> - .enable_shift = MXC_CCM_CCGR3_IPU1_IPU_DI0_OFFSET,
>>> + .enable_shift = MXC_CCM_CCGR3_IPU1_IPU_MASK,
>>
>> But in ipu_common.c:
>>
>> .enable_reg = (u32 *)(CCM_BASE_ADDR +
>> offsetof(struct mxc_ccm_reg, CCGR5)),
>> .enable_shift = MXC_CCM_CCGR5_CG5_OFFSET,
>>
>
> CCGR5/CG5 is the IPU clock on i.MX5x, not i.MX6x.
Right, but I see no #ifdef..
>
>> I do not see another patch changing that. Am I missing something ?
>>
>
> This patch requires the previous change to name the CCGRx bitfields:
>
> http://patchwork.ozlabs.org/patch/185856/
Ah, ok, I see ! Next iteration, thanks.
Best regards,
Stefano
--
=====================================================================
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: sba...@denx.de
=====================================================================
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
