lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 18 Apr 2016 15:08:42 +0800 From: Peter Chen <hzpeterchen@...il.com> To: Roger Quadros <rogerq@...com> Cc: stern@...land.harvard.edu, balbi@...nel.org, gregkh@...uxfoundation.org, peter.chen@...escale.com, dan.j.williams@...el.com, jun.li@...escale.com, mathias.nyman@...ux.intel.com, tony@...mide.com, Joao.Pinto@...opsys.com, abrestic@...omium.org, r.baldyga@...sung.com, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org Subject: Re: [PATCH v6 11/12] usb: core: hub: Notify OTG fsm when A device sets b_hnp_enable On Tue, Apr 05, 2016 at 05:05:16PM +0300, Roger Quadros wrote: > This is the a_set_b_hnp_enable flag in the OTG state machine > diagram and must be set when the A-Host has successfully set > the b_hnp_enable feature of the OTG-B-Peripheral attached to it. > > When this bit changes we kick our OTG FSM to make note of the > change and act accordingly. Since we have still not added fsm.a_set_b_hnp_en in OTG FSM, and this patch set is mainly for DRD, would you please move out this patch from this set? Peter > > Signed-off-by: Roger Quadros <rogerq@...com> > --- > drivers/usb/core/hub.c | 17 +++++++++++++++-- > 1 file changed, 15 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c > index 38cc4ba..27e3b4c 100644 > --- a/drivers/usb/core/hub.c > +++ b/drivers/usb/core/hub.c > @@ -2273,6 +2273,7 @@ static int usb_enumerate_device_otg(struct usb_device *udev) > && udev->parent == udev->bus->root_hub) { > struct usb_otg_descriptor *desc = NULL; > struct usb_bus *bus = udev->bus; > + struct usb_hcd *hcd = bus_to_hcd(bus); > unsigned port1 = udev->portnum; > > /* descriptor may appear anywhere in config */ > @@ -2302,6 +2303,9 @@ static int usb_enumerate_device_otg(struct usb_device *udev) > dev_err(&udev->dev, "can't set HNP mode: %d\n", > err); > bus->b_hnp_enable = 0; > + } else { > + /* notify OTG fsm about a_set_b_hnp_enable */ > + usb_otg_kick_fsm(hcd->otg_dev); > } > } else if (desc->bLength == sizeof > (struct usb_otg_descriptor)) { > @@ -2312,10 +2316,14 @@ static int usb_enumerate_device_otg(struct usb_device *udev) > USB_DEVICE_A_ALT_HNP_SUPPORT, > 0, NULL, 0, > USB_CTRL_SET_TIMEOUT); > - if (err < 0) > + if (err < 0) { > dev_err(&udev->dev, > "set a_alt_hnp_support failed: %d\n", > err); > + } else { > + /* notify OTG fsm about a_set_b_hnp_enable */ > + usb_otg_kick_fsm(hcd->otg_dev); > + } > } > } > #endif > @@ -4355,8 +4363,13 @@ hub_port_init(struct usb_hub *hub, struct usb_device *udev, int port1, > */ > if (!hdev->parent) { > delay = HUB_ROOT_RESET_TIME; > - if (port1 == hdev->bus->otg_port) > + if (port1 == hdev->bus->otg_port) { > hdev->bus->b_hnp_enable = 0; > +#ifdef CONFIG_USB_OTG > + /* notify OTG fsm about a_set_b_hnp_enable change */ > + usb_otg_kick_fsm(hcd->otg_dev); > +#endif > + } > } > > /* Some low speed devices have problems with the quick delay, so */ > -- > 2.5.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards, Peter Chen
Powered by blists - more mailing lists