[PATCH 01/74] ST SPEAr: Padmux code Updated

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Tue Sep 7 00:07:48 EDT 2010


On 09:21 Tue 07 Sep     , viresh kumar wrote:
> On 9/7/2010 4:19 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> >>  
> >> > -void spear300_pmx_init(void)
> >> > -{
> >> > -	spear_pmx_init(&pmx_driver, SPEAR300_SOC_CONFIG_BASE,
> >> > +	/* pmx initialization */
> >> > +	pmx_driver.base = ioremap(SPEAR300_SOC_CONFIG_BASE,
> >> >  			SPEAR300_SOC_CONFIG_SIZE);
> >> > +	if (pmx_driver.base) {
> >> > +		ret = pmx_register(&pmx_driver);
> >> > +		if (ret)
> >> > +			printk(KERN_ERR "padmux: registeration failed. err no"
> >> > +					": %d\n", ret);
> >> > +		iounmap(pmx_driver.base);
> > why unmap it?
> 
> pmx_register will actually enable all selected devices and configure padmux.
> After this we will never change device configuration again. So don't need
> this mapping.
so put a comment in the code

Best Regards,
J.



More information about the linux-arm-kernel mailing list