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

viresh kumar viresh.kumar at st.com
Tue Sep 7 00:10:01 EDT 2010


On 9/7/2010 9:37 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> 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
> 

Will do that.



More information about the linux-arm-kernel mailing list