Hi Fengguang, My apologies for the inconvenience caused. During phy interface separation from otg.h, as the enum "usb_otg_state" was having multiple otg states info, I felt that should remain in otg.h. Also as removal of member 'state' of this enum type from usb_phy struct did not generate any compilation issues, I removed member state from struct usb_phy. As this is causing build break in musb code, I pushed a patch to balbi's tree for fixing this. Also added you as a reviewer in the to list of the patch. Plz verify and let me know if you still see any issues. Thanks, Venu > -----Original Message----- > From: Fengguang Wu [mailto:fengguang.wu@xxxxxxxxx] > Sent: Thursday, September 06, 2012 7:41 AM > To: Venu Byravarasu > Cc: kernel-janitors@xxxxxxxxxxxxxxx; Felipe Balbi; linux- > omap@xxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx > Subject: [balbi-usb:master 18/19] drivers/usb/musb/musb_core.c:357:21: > error: 'struct usb_phy' has no member named 'state' > > Hi Venu, > > FYI, kernel build failed on > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git master > head: 8de3474a41642da9dfb38208844007a48f87abce > commit: de4217d90fed1b1714a270ceb5d092f7314e8bda [18/19] usb: otg: > Move phy interface to separate file. > config: x86_64-allmodconfig (attached as .config) > > All related error/warning messages: > > drivers/usb/musb/musb_core.c: In function 'musb_otg_timer_func': > drivers/usb/musb/musb_core.c:357:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:361:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:366:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:369:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:372:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c: In function 'musb_hnp_stop': > drivers/usb/musb/musb_core.c:388:2: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:390:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:393:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:399:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:407:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c: In function 'musb_stage0_irq': > drivers/usb/musb/musb_core.c:446:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:451:23: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:474:16: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:479:16: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:484:5: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:489:23: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:492:16: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_core.c:513:5: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:530:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/musb_core.c:542:14: error: 'struct usb_phy' has no > member named 'state' > -- > drivers/usb/musb/musb_gadget.c: In function 'musb_gadget_wakeup': > drivers/usb/musb/musb_gadget.c:1630:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:1670:3: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c: In function 'musb_gadget_start': > drivers/usb/musb/musb_gadget.c:1952:13: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c: In function 'musb_gadget_stop': > drivers/usb/musb/musb_gadget.c:2063:13: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c: In function 'musb_g_resume': > drivers/usb/musb/musb_gadget.c:2096:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2109:3: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c: In function 'musb_g_suspend': > drivers/usb/musb/musb_gadget.c:2122:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2125:15: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2139:3: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c: In function 'musb_g_disconnect': > drivers/usb/musb/musb_gadget.c:2171:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2173:3: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2175:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2179:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2186:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c: In function 'musb_g_reset': > drivers/usb/musb/musb_gadget.c:2242:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_gadget.c:2245:14: error: 'struct usb_phy' has no > member named 'state' > -- > drivers/usb/musb/musb_virthub.c: In function 'musb_port_suspend': > drivers/usb/musb/musb_virthub.c:81:22: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:83:15: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:93:15: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:99:4: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c: In function 'musb_port_reset': > drivers/usb/musb/musb_virthub.c:120:17: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c: In function 'musb_root_disconnect': > drivers/usb/musb/musb_virthub.c:191:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:195:15: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:201:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:205:14: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c:208:3: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_virthub.c: In function 'musb_hub_control': > drivers/usb/musb/musb_virthub.c:345:15: error: 'struct usb_phy' has no > member named 'state' > -- > drivers/usb/musb/musb_host.c: In function 'musb_bus_suspend': > drivers/usb/musb/musb_host.c:2290:21: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_host.c:2300:15: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/musb/musb_host.c:2307:3: error: 'struct usb_phy' has no > member named 'state' > -- > drivers/usb/musb/tusb6010.c: In function 'musb_do_idle': > drivers/usb/musb/tusb6010.c:419:21: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:424:4: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c: In function 'tusb_musb_try_idle': > drivers/usb/musb/tusb6010.c:479:19: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:480:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:497:2: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c: In function 'tusb_musb_set_vbus': > drivers/usb/musb/tusb6010.c:529:14: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:542:23: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:545:16: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:548:16: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:551:16: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:559:15: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:573:2: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c: In function 'tusb_otg_ints': > drivers/usb/musb/tusb6010.c:686:20: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:688:17: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:693:4: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:699:4: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:702:23: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:748:3: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:751:22: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/musb/tusb6010.c:763:16: error: 'struct usb_phy' has no member > named 'state' > -- > drivers/usb/otg/gpio_vbus.c: In function 'gpio_vbus_work': > drivers/usb/otg/gpio_vbus.c:119:17: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/otg/gpio_vbus.c:141:17: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/otg/gpio_vbus.c: In function 'gpio_vbus_set_peripheral': > drivers/usb/otg/gpio_vbus.c:194:11: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/otg/gpio_vbus.c: In function 'gpio_vbus_set_power': > drivers/usb/otg/gpio_vbus.c:216:9: error: 'struct usb_phy' has no member > named 'state' > drivers/usb/otg/gpio_vbus.c: In function 'gpio_vbus_probe': > drivers/usb/otg/gpio_vbus.c:266:16: error: 'struct usb_phy' has no member > named 'state' > -- > drivers/usb/otg/nop-usb-xceiv.c: In function 'nop_set_peripheral': > drivers/usb/otg/nop-usb-xceiv.c:78:10: error: 'struct usb_phy' has no > member named 'state' > drivers/usb/otg/nop-usb-xceiv.c: In function 'nop_usb_xceiv_probe': > drivers/usb/otg/nop-usb-xceiv.c:120:10: error: 'struct usb_phy' has no > member named 'state' > > --- > 0-DAY kernel build testing backend Open Source Technology Centre > Fengguang Wu <wfg@xxxxxxxxxxxxxxx> Intel Corporation ----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. ----------------------------------------------------------------------------------- -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html