Re: [PATCH 20/23] musb_virthub: use HUB_CHAR_*
On Tue, Jan 27, 2015 at 07:06:14PM +0300, Sergei Shtylyov wrote: > Hello. > > On 01/27/2015 06:35 PM, Felipe Balbi wrote: > > >>>Fix using the bare numbers to set the 'wHubCharacteristics' field of the > >>>Hub > >>>Descriptor while the values are #define'd in . > > >>>Signed-off-by: Sergei Shtylyov > > >>>--- > >>> drivers/usb/musb/musb_virthub.c |7 --- > >>> 1 file changed, 4 insertions(+), 3 deletions(-) > > >>>Index: usb/drivers/usb/musb/musb_virthub.c > >>>=== > >>>--- usb.orig/drivers/usb/musb/musb_virthub.c > >>>+++ usb/drivers/usb/musb/musb_virthub.c > >>>@@ -349,9 +349,10 @@ int musb_hub_control( > >>> desc->bDescriptorType = 0x29; > >>> desc->bNbrPorts = 1; > >>> desc->wHubCharacteristics = cpu_to_le16( > >>>-0x0001/* per-port power switching */ > >>>- | 0x0010/* no overcurrent reporting */ > >>>- ); > >>>+HUB_CHAR_INDV_PORT_LPSM /* per-port power switching */ > >>>+ > > >>Ugh, don't know how this empty line sneaked into the patch. Greg, could > >>you please drop it? > > >fixed > >You mean you've merged the patch to your tree? If so, be prepared for the > conflict with Greg's tree since Greg has already merged the patch to his > 'usb-next' branch... Thanks anyway! musb goes through me, right ? :-) Conflicts are easy to solve, no problem. cheers -- balbi signature.asc Description: Digital signature
Re: [PATCH 20/23] musb_virthub: use HUB_CHAR_*
Hello. On 01/27/2015 06:35 PM, Felipe Balbi wrote: Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in . Signed-off-by: Sergei Shtylyov --- drivers/usb/musb/musb_virthub.c |7 --- 1 file changed, 4 insertions(+), 3 deletions(-) Index: usb/drivers/usb/musb/musb_virthub.c === --- usb.orig/drivers/usb/musb/musb_virthub.c +++ usb/drivers/usb/musb/musb_virthub.c @@ -349,9 +349,10 @@ int musb_hub_control( desc->bDescriptorType = 0x29; desc->bNbrPorts = 1; desc->wHubCharacteristics = cpu_to_le16( - 0x0001/* per-port power switching */ - | 0x0010/* no overcurrent reporting */ - ); + HUB_CHAR_INDV_PORT_LPSM /* per-port power switching */ + Ugh, don't know how this empty line sneaked into the patch. Greg, could you please drop it? fixed You mean you've merged the patch to your tree? If so, be prepared for the conflict with Greg's tree since Greg has already merged the patch to his 'usb-next' branch... Thanks anyway! WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 20/23] musb_virthub: use HUB_CHAR_*
On Sun, Jan 25, 2015 at 07:51:49PM +0300, Sergei Shtylyov wrote: > Hello. > > On 01/19/2015 01:57 AM, Sergei Shtylyov wrote: > > >Fix using the bare numbers to set the 'wHubCharacteristics' field of the > >Hub > >Descriptor while the values are #define'd in . > > >Signed-off-by: Sergei Shtylyov > > >--- > > drivers/usb/musb/musb_virthub.c |7 --- > > 1 file changed, 4 insertions(+), 3 deletions(-) > > > >Index: usb/drivers/usb/musb/musb_virthub.c > >=== > >--- usb.orig/drivers/usb/musb/musb_virthub.c > >+++ usb/drivers/usb/musb/musb_virthub.c > >@@ -349,9 +349,10 @@ int musb_hub_control( > > desc->bDescriptorType = 0x29; > > desc->bNbrPorts = 1; > > desc->wHubCharacteristics = cpu_to_le16( > >- 0x0001/* per-port power switching */ > >-| 0x0010/* no overcurrent reporting */ > >-); > >+ HUB_CHAR_INDV_PORT_LPSM /* per-port power switching */ > >+ > >Ugh, don't know how this empty line sneaked into the patch. Greg, could > you please drop it? fixed -- balbi signature.asc Description: Digital signature
Re: [PATCH 20/23] musb_virthub: use HUB_CHAR_*
Hello. On 01/19/2015 01:57 AM, Sergei Shtylyov wrote: Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in . Signed-off-by: Sergei Shtylyov --- drivers/usb/musb/musb_virthub.c |7 --- 1 file changed, 4 insertions(+), 3 deletions(-) Index: usb/drivers/usb/musb/musb_virthub.c === --- usb.orig/drivers/usb/musb/musb_virthub.c +++ usb/drivers/usb/musb/musb_virthub.c @@ -349,9 +349,10 @@ int musb_hub_control( desc->bDescriptorType = 0x29; desc->bNbrPorts = 1; desc->wHubCharacteristics = cpu_to_le16( - 0x0001/* per-port power switching */ - | 0x0010/* no overcurrent reporting */ - ); + HUB_CHAR_INDV_PORT_LPSM /* per-port power switching */ + Ugh, don't know how this empty line sneaked into the patch. Greg, could you please drop it? + | HUB_CHAR_NO_OCPM /* no overcurrent reporting */ + ); desc->bPwrOn2PwrGood = 5;/* msec/2 */ desc->bHubContrCurrent = 0; WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 20/23] musb_virthub: use HUB_CHAR_*
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in . Signed-off-by: Sergei Shtylyov --- drivers/usb/musb/musb_virthub.c |7 --- 1 file changed, 4 insertions(+), 3 deletions(-) Index: usb/drivers/usb/musb/musb_virthub.c === --- usb.orig/drivers/usb/musb/musb_virthub.c +++ usb/drivers/usb/musb/musb_virthub.c @@ -349,9 +349,10 @@ int musb_hub_control( desc->bDescriptorType = 0x29; desc->bNbrPorts = 1; desc->wHubCharacteristics = cpu_to_le16( - 0x0001/* per-port power switching */ - | 0x0010/* no overcurrent reporting */ - ); + HUB_CHAR_INDV_PORT_LPSM /* per-port power switching */ + + | HUB_CHAR_NO_OCPM /* no overcurrent reporting */ + ); desc->bPwrOn2PwrGood = 5; /* msec/2 */ desc->bHubContrCurrent = 0; -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html