[U-Boot] [PATCH 05/17] drivers: usb: musb: adopt musb backend driver to driver model

Mugunthan V N mugunthanvnm at ti.com
Mon Feb 29 14:26:46 CET 2016


On Monday 29 February 2016 05:34 PM, Marek Vasut wrote:
> On 02/29/2016 04:44 AM, Mugunthan V N wrote:
>> Currently all backend driver ops uses hard coded physical
>> address, so to adopt the driver to DM, add device pointer to ops
>> call backs so that drivers that drivers can get physical
>> addresses from the usb driver priv/plat data.
>>
>> Signed-off-by: Mugunthan V N <mugunthanvnm at ti.com>
>> ---
> 
> [...]
> 
>> @@ -704,8 +719,13 @@ static int dsps_resume(struct device *dev)
>>  	struct omap_musb_board_data *data = plat->board_data;
>>  
>>  	/* Start the on-chip PHY and its PLL. */
>> +#ifndef CONFIG_DM_USB
>>  	if (data->set_phy_power)
>>  		data->set_phy_power(1);
>> +#else
>> +	if (data->set_phy_power)
>> +		data->set_phy_power(data->dev, 1);
> 
> Would it be possible to avoid adding this sea of ifdefs into the driver?
> 

May be introducing a void pointer and pass it back in ops callback can
avoid ifdefs. For non DM mode, it will be holding NULL pointer and for
DM mode it will be holding a dev pointer. If its ok, I can fix in v2.

Regards
Mugunthan V N



More information about the U-Boot mailing list