From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1UlJ5k-0005kE-OI for barebox@lists.infradead.org; Sat, 08 Jun 2013 13:22:01 +0000 Date: Sat, 8 Jun 2013 15:21:38 +0200 From: Sascha Hauer Message-ID: <20130608132138.GI32299@pengutronix.de> References: <1370425962.3824.1.camel@lws-weitzel> <1370522836-11258-1-git-send-email-j.weitzel@phytec.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH v2] omap4: set voltage according to mpu freq To: "menon.nishanth@gmail.com" Cc: "U-Boot Version 2 (barebox)" On Thu, Jun 06, 2013 at 08:40:46AM -0500, menon.nishanth@gmail.com wrote: > On Thu, Jun 6, 2013 at 7:47 AM, Jan Weitzel wrote: > > +noinline int omap4460_scale_vcores(unsigned vsel0_pin, unsigned volt_mv) > > +{ > > + void __iomem *base; > > + u32 val = 0; > > + > > + /* For VC bypass only VCOREx_CGF_FORCE is necessary and > > + * VCOREx_CFG_VOLTAGE changes can be discarded > > + */ > > + writel(0, OMAP44XX_PRM_VC_CFG_I2C_MODE); > > + writel(0x6026, OMAP44XX_PRM_VC_CFG_I2C_CLK); > > + > > + /* TPS - supplies vdd_mpu on 4460 > > + * Setup SET1 and SET0 with right values so that kernel > > + * can use either of them based on its needs. > > + */ > > + > > + omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET0, volt_mv); > Just a nitpick - the general rule of TPS+OMAP4460 integration is *NOT* > to program SET0 register. this is intended to be at boot voltage > required to be used when reboot due to s/w controlled or h/w watchdog. So this line should simply be removed? Ok, this line is only moved and not introduced in this patch, so it should be fine to apply this patch and leave this for a separate patch. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox