Use the correct types and sizes. v2: use u8 shorthand for unsigned char. Signed-off-by: Grigori Goronzy <greg@xxxxxxxxxxxx> --- drivers/usb/serial/ch341.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/usb/serial/ch341.c b/drivers/usb/serial/ch341.c index db4b561..95c8a40 100644 --- a/drivers/usb/serial/ch341.c +++ b/drivers/usb/serial/ch341.c @@ -115,7 +115,7 @@ static int ch341_control_out(struct usb_device *dev, u8 request, static int ch341_control_in(struct usb_device *dev, u8 request, u16 value, u16 index, - char *buf, unsigned bufsize) + u8 *buf, unsigned bufsize) { int r; @@ -168,9 +168,9 @@ static int ch341_set_handshake(struct usb_device *dev, u8 control) static int ch341_get_status(struct usb_device *dev, struct ch341_private *priv) { - char *buffer; + unsigned char *buffer; int r; - const unsigned size = 8; + const unsigned size = 2; unsigned long flags; buffer = kmalloc(size, GFP_KERNEL); @@ -198,9 +198,9 @@ out: kfree(buffer); static int ch341_configure(struct usb_device *dev, struct ch341_private *priv) { - char *buffer; + unsigned char *buffer; int r; - const unsigned size = 8; + const unsigned size = 2; buffer = kmalloc(size, GFP_KERNEL); if (!buffer) -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html