RE: [PATCH] staging: pl2303: adding TD620 Device ID
Thanks for the feedback. I'll review and resubmit. -Original Message- From: Johan Hovold [mailto:jhov...@gmail.com] On Behalf Of Johan Hovold Sent: Wednesday, September 20, 2017 2:47 AM To: Schoon, Michael <michael.sch...@hp.com> Cc: jo...@kernel.org; linux-usb@vger.kernel.org; Greg Kroah-Hartman <gre...@linuxfoundation.org> Subject: Re: [PATCH] staging: pl2303: adding TD620 Device ID On Wed, Sep 20, 2017 at 08:15:38AM +0200, Greg Kroah-Hartman wrote: > On Tue, Sep 19, 2017 at 09:41:48PM +, Schoon, Michael wrote: > > Signed-off-by: Michael Schoon <michael.sch...@hp.com> > > No changelog at all? I know I can't take patches like that, hopefully > Johan has the same rule :) Indeed, I do. Please also fix up the Subject; this driver does not live in staging so that should be "USB: serial: pl2303:" instead. Also try to keep any new id-defines you add sorted by PID. Thanks, Johan -- 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] staging: pl2303: adding TD620 Device ID
On Wed, Sep 20, 2017 at 08:15:38AM +0200, Greg Kroah-Hartman wrote: > On Tue, Sep 19, 2017 at 09:41:48PM +, Schoon, Michael wrote: > > Signed-off-by: Michael Schoon> > No changelog at all? I know I can't take patches like that, hopefully > Johan has the same rule :) Indeed, I do. Please also fix up the Subject; this driver does not live in staging so that should be "USB: serial: pl2303:" instead. Also try to keep any new id-defines you add sorted by PID. Thanks, Johan -- 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] staging: pl2303: adding TD620 Device ID
On Tue, Sep 19, 2017 at 09:41:48PM +, Schoon, Michael wrote: > Signed-off-by: Michael SchoonNo changelog at all? I know I can't take patches like that, hopefully Johan has the same rule :) > --- > drivers/usb/serial/pl2303.c | 1 + > drivers/usb/serial/pl2303.h | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/drivers/usb/serial/pl2303.c b/drivers/usb/serial/pl2303.c > index a585b47..eab0e55 100644 > --- a/drivers/usb/serial/pl2303.c > +++ b/drivers/usb/serial/pl2303.c > @@ -94,6 +94,7 @@ static const struct usb_device_id id_table[] = { > { USB_DEVICE(HP_VENDOR_ID, HP_LD960_PRODUCT_ID) }, > { USB_DEVICE(HP_VENDOR_ID, HP_LCM220_PRODUCT_ID) }, > { USB_DEVICE(HP_VENDOR_ID, HP_LCM960_PRODUCT_ID) }, > + { USB_DEVICE(HP_VENDOR_ID, HP_TD620_PRODUCT_ID) }, > { USB_DEVICE(CRESSI_VENDOR_ID, CRESSI_EDY_PRODUCT_ID) }, > { USB_DEVICE(ZEAGLE_VENDOR_ID, ZEAGLE_N2ITION3_PRODUCT_ID) }, > { USB_DEVICE(SONY_VENDOR_ID, SONY_QN3USB_PRODUCT_ID) }, Your patch has had the tabs turned into spaces, making this impossible to apply. Try testing it out first by emailing it to yourself. thanks, greg k-h -- 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] staging: pl2303: adding TD620 Device ID
Signed-off-by: Michael Schoon--- drivers/usb/serial/pl2303.c | 1 + drivers/usb/serial/pl2303.h | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/usb/serial/pl2303.c b/drivers/usb/serial/pl2303.c index a585b47..eab0e55 100644 --- a/drivers/usb/serial/pl2303.c +++ b/drivers/usb/serial/pl2303.c @@ -94,6 +94,7 @@ static const struct usb_device_id id_table[] = { { USB_DEVICE(HP_VENDOR_ID, HP_LD960_PRODUCT_ID) }, { USB_DEVICE(HP_VENDOR_ID, HP_LCM220_PRODUCT_ID) }, { USB_DEVICE(HP_VENDOR_ID, HP_LCM960_PRODUCT_ID) }, + { USB_DEVICE(HP_VENDOR_ID, HP_TD620_PRODUCT_ID) }, { USB_DEVICE(CRESSI_VENDOR_ID, CRESSI_EDY_PRODUCT_ID) }, { USB_DEVICE(ZEAGLE_VENDOR_ID, ZEAGLE_N2ITION3_PRODUCT_ID) }, { USB_DEVICE(SONY_VENDOR_ID, SONY_QN3USB_PRODUCT_ID) }, diff --git a/drivers/usb/serial/pl2303.h b/drivers/usb/serial/pl2303.h index 3b5a15d..ca0bfba 100644 --- a/drivers/usb/serial/pl2303.h +++ b/drivers/usb/serial/pl2303.h @@ -126,6 +126,7 @@ #define HP_LCM220_PRODUCT_ID 0x3139 #define HP_LCM960_PRODUCT_ID 0x3239 #define HP_LD220_PRODUCT_ID 0x3524 +#define HP_TD620_PRODUCT_ID 0x0956 /* Cressi Edy (diving computer) PC interface */ #define CRESSI_VENDOR_ID 0x04b8 -- 2.7.4 -- 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