NothingOSS/android_kernel_msm-6.1_nothing_sm7635
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

usb: misc: lvstest: allow specifying port for compliance mode

Browse source 
In some cases when connecting to a test fixture the host port
might not have triggered a connect change. As a result, dev->portnum
does not get updated and an attempt to enable the port for
compliance mode will pass 0 as the port number to the HCD, which is
invalid. Instead allow the user to pass an explicit port number
from the command line to specify the port to act on. Otherwise, if
omitted, the behavior is the same as before.

Change-Id: I2e323c9016915b2d535ac227fd6f3d5846aa4026
Signed-off-by: Jack Pham <jackp@codeaurora.org>
This commit is contained in:
Jack Pham 2017-10-02 14:13:47 -07:00 • committed by Wesley Cheng
parent 84a14b5670
commit 6c815cebc3
1 changed files with 34 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

40
drivers/usb/misc/lvstest.c
View file

@ -105,8 +105,14 @@ static ssize_t u3_entry_store(struct device *dev,
struct usb_device *hdev = interface_to_usbdev(intf);
struct lvs_rh *lvs = usb_get_intfdata(intf);
struct usb_device *udev;
int port;
int ret;
if (!kstrtoint(buf, 0, &port) || port >= 1 || port <= 255) {
lvs->portnum = port;
lvs->present = true;
}
udev = create_lvs_device(intf);
if (!udev) {
dev_err(dev, "failed to create lvs device\n");
@ -134,8 +140,14 @@ static ssize_t u3_exit_store(struct device *dev,
struct usb_device *hdev = interface_to_usbdev(intf);
struct lvs_rh *lvs = usb_get_intfdata(intf);
struct usb_device *udev;
int port;
int ret;
if (!kstrtoint(buf, 0, &port) || port >= 1 || port <= 255) {
lvs->portnum = port;
lvs->present = true;
}
udev = create_lvs_device(intf);
if (!udev) {
dev_err(dev, "failed to create lvs device\n");
@ -162,9 +174,13 @@ static ssize_t hot_reset_store(struct device *dev,
struct usb_interface *intf = to_usb_interface(dev);
struct usb_device *hdev = interface_to_usbdev(intf);
struct lvs_rh *lvs = usb_get_intfdata(intf);
int port;
int ret;
ret = lvs_rh_set_port_feature(hdev, lvs->portnum,
if (kstrtoint(buf, 0, &port) || port < 1 || port > 255)
port = lvs->portnum;
ret = lvs_rh_set_port_feature(hdev, port,
USB_PORT_FEAT_RESET);
if (ret < 0) {
dev_err(dev, "can't issue hot reset %d\n", ret);
@ -181,10 +197,13 @@ static ssize_t warm_reset_store(struct device *dev,
struct usb_interface *intf = to_usb_interface(dev);
struct usb_device *hdev = interface_to_usbdev(intf);
struct lvs_rh *lvs = usb_get_intfdata(intf);
int port;
int ret;
ret = lvs_rh_set_port_feature(hdev, lvs->portnum,
USB_PORT_FEAT_BH_PORT_RESET);
if (kstrtoint(buf, 0, &port) || port < 1 || port > 255)
port = lvs->portnum;
ret = lvs_rh_set_port_feature(hdev, port, USB_PORT_FEAT_BH_PORT_RESET);
if (ret < 0) {
dev_err(dev, "can't issue warm reset %d\n", ret);
return ret;
@ -256,9 +275,15 @@ static ssize_t get_dev_desc_store(struct device *dev,
struct device_attribute *attr, const char *buf, size_t count)
{
struct usb_interface *intf = to_usb_interface(dev);
struct lvs_rh *lvs = usb_get_intfdata(intf);
struct usb_device *udev;
struct usb_device_descriptor *descriptor;
int ret;
int ret, port;
if (!kstrtoint(buf, 0, &port) || port >= 1 || port <= 255) {
lvs->portnum = port;
lvs->present = true;
}
descriptor = kmalloc(sizeof(*descriptor), GFP_KERNEL);
if (!descriptor)
@ -277,7 +302,6 @@ static ssize_t get_dev_desc_store(struct device *dev,
USB_CTRL_GET_TIMEOUT);
if (ret < 0)
dev_err(dev, "can't read device descriptor %d\n", ret);
destroy_lvs_device(udev);
free_desc:
@ -296,10 +320,14 @@ static ssize_t enable_compliance_store(struct device *dev,
struct usb_interface *intf = to_usb_interface(dev);
struct usb_device *hdev = interface_to_usbdev(intf);
struct lvs_rh *lvs = usb_get_intfdata(intf);
int port;
int ret;
if (kstrtoint(buf, 0, &port) || port < 1 || port > 255)
port = lvs->portnum;
ret = lvs_rh_set_port_feature(hdev,
lvs->portnum | USB_SS_PORT_LS_COMP_MOD << 3,
port | (USB_SS_PORT_LS_COMP_MOD << 3),
USB_PORT_FEAT_LINK_STATE);
if (ret < 0) {
dev_err(dev, "can't enable compliance mode %d\n", ret);