From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wang Huan-B18965 <B18965@freescale.com>
Date: Wed, 22 May 2013 02:59:15 +0000
Subject: [U-Boot] [PATCH v3 1/6] arm: mvf600: Add Vybrid MVF600 CPU
	support
In-Reply-To: <519B7B1D.6020101@freescale.com>
References: <1369126981-13970-1-git-send-email-b18965@freescale.com>
	<1369126981-13970-2-git-send-email-b18965@freescale.com>
	<519B7B1D.6020101@freescale.com>
Message-ID: <81BA6E5E0BC2344391CABCEE22D1B6D83359EB@039-SN1MPN1-003.039d.mgd.msft.net>
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, Fabio,

> -----Original Message-----
> From: Estevam Fabio-R49496
> Sent: Tuesday, May 21, 2013 9:48 PM
> To: Wang Huan-B18965
> Cc: sbabic at denx.de; u-boot at lists.denx.de; Liu Hui-R64343; Estevam Fabio-
> R49496
> Subject: Re: [PATCH v3 1/6] arm: mvf600: Add Vybrid MVF600 CPU support
> 
> On 05/21/2013 06:02 AM, Alison Wang wrote:
> 
> > +#ifdef CONFIG_MXC_OCOTP
> > +void enable_ocotp_clk(unsigned char enable) {
> > +	struct ccm_reg *ccm = (struct ccm_reg *)CCM_BASE_ADDR;
> > +	u32 reg;
> > +
> > +	reg = readl(&ccm->ccgr6);
> > +	if (enable)
> > +		reg |= CCM_CCGR6_OCOTP_CTRL_MASK;
> > +	else
> > +		reg |= ~CCM_CCGR6_OCOTP_CTRL_MASK;
> 
> Don't you mean: reg &= ~CCM_CCGR6_OCOTP_CTRL_MASK; ?
[Alison Wang] Yes. I will change. Thanks.


Best Regards,
Alison Wang