>-----Original Message-----
>From: Tony Lindgren [mailto:[email protected]] 
>Sent: Monday, June 28, 2010 7:12 PM
>To: Kalliguddi, Hema
>Cc: [email protected]; [email protected]; Felipe Balbi
>Subject: Re: [PATCH 4/4]usb : musb:USB driver using 
>omap_device_build for device registration.
>
>* Kalliguddi, Hema <[email protected]> [100628 16:26]:
>>  
>> 
>> >-----Original Message-----
>> >From: Tony Lindgren [mailto:[email protected]] 
>> >Sent: Monday, June 28, 2010 6:59 PM
>> >To: Kalliguddi, Hema
>> >Cc: [email protected]; [email protected]; 
>Felipe Balbi
>> >Subject: Re: [PATCH 4/4]usb : musb:USB driver using 
>> >omap_device_build for device registration.
>> >
>> >* Hema HK <[email protected]> [100628 15:12]:
>> >> -
>> >> - /*
>> >> -  * REVISIT: This line can be removed once all the 
>> >platforms using
>> >> -  * musb_core.c have been converted to use use clkdev.
>> >> -  */
>> >> - musb_plat.clock = "ick";
>> >
>> >This comment is still valid, please don't remove it.
>> This is still there. It is moved below under else.
>
>Not in the patch you posted?
>

-       musb_plat.clock = "ick";
-       musb_plat.board_data = board_data;
-       musb_plat.power = board_data->power >> 1;
-       musb_plat.mode = board_data->mode;
-       musb_plat.extvbus = board_data->extvbus;
-
-       if (platform_device_register(&musb_device) < 0)
-               printk(KERN_ERR "Unable to register HS-USB (MUSB) device\n");
+       char oh_name[MAX_OMAP_MUSB_HWMOD_NAME_LEN];
+       struct omap_hwmod *oh;
+       struct omap_device *od;
+       struct platform_device *pdev;
+       struct device   *dev;
+       int l, bus_id = -1;
+       struct musb_hdrc_platform_data *pdata;
+
+       l = snprintf(oh_name, MAX_OMAP_MUSB_HWMOD_NAME_LEN,
+                                               "usb_otg_hs");
+       WARN(l >= MAX_OMAP_MUSB_HWMOD_NAME_LEN,
+       "String buffer overflow in MUSB device setup\n");
+       oh = omap_hwmod_lookup(oh_name);
+       if (!oh) {
+               pr_err("Could not look up %s\n", oh_name);
+       } else {
+               musb_plat.clock = "ick";
+               musb_plat.board_data = board_data;
+               musb_plat.power = board_data->power >> 1;
+               musb_plat.mode = board_data->mode;
+               pdata = &musb_plat;

It is there in the patch I posted.only the comment is being removed. You are 
refering the comment also? 

Regards,
Hema
>Tony
>--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to