Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752331AbbBKJdh (ORCPT ); Wed, 11 Feb 2015 04:33:37 -0500 Received: from cantor2.suse.de ([195.135.220.15]:59553 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752269AbbBKJde (ORCPT ); Wed, 11 Feb 2015 04:33:34 -0500 Date: Wed, 11 Feb 2015 10:33:32 +0100 Message-ID: From: Takashi Iwai To: Chris Rorvick Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Stefan Hajnoczi Subject: Re: [PATCH 1/6] ALSA: line6: Improve line6_read/write_data() interfaces In-Reply-To: <1423630997-25464-2-git-send-email-chris@rorvick.com> References: <1423630997-25464-1-git-send-email-chris@rorvick.com> <1423630997-25464-2-git-send-email-chris@rorvick.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.4 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3459 Lines: 94 At Tue, 10 Feb 2015 23:03:12 -0600, Chris Rorvick wrote: > > Use explicit types to reflect the range of valid values. Well, this isn't necessarily to be changed at these places. If we want to be safer, there should be proper range checks instead. Also, readers may wonder when they see the different types between read and write without explanations. Though, I agree that size_t is superfluous, especially for 64bit, too. thanks, Takashi > > Signed-off-by: Chris Rorvick > --- > sound/usb/line6/driver.c | 10 +++++----- > sound/usb/line6/driver.h | 8 ++++---- > 2 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/sound/usb/line6/driver.c b/sound/usb/line6/driver.c > index 1e58e92..aac1e35 100644 > --- a/sound/usb/line6/driver.c > +++ b/sound/usb/line6/driver.c > @@ -299,8 +299,8 @@ static void line6_data_received(struct urb *urb) > /* > Read data from device. > */ > -int line6_read_data(struct usb_line6 *line6, int address, void *data, > - size_t datalen) > +int line6_read_data(struct usb_line6 *line6, u16 address, void *data, > + u8 datalen) > { > struct usb_device *usbdev = line6->usbdev; > int ret; > @@ -309,7 +309,7 @@ int line6_read_data(struct usb_line6 *line6, int address, void *data, > /* query the serial number: */ > ret = usb_control_msg(usbdev, usb_sndctrlpipe(usbdev, 0), 0x67, > USB_TYPE_VENDOR | USB_RECIP_DEVICE | USB_DIR_OUT, > - (datalen << 8) | 0x21, address, > + ((u16) datalen << 8) | 0x21, address, > NULL, 0, LINE6_TIMEOUT * HZ); > > if (ret < 0) { > @@ -357,8 +357,8 @@ EXPORT_SYMBOL_GPL(line6_read_data); > /* > Write data to device. > */ > -int line6_write_data(struct usb_line6 *line6, int address, void *data, > - size_t datalen) > +int line6_write_data(struct usb_line6 *line6, u16 address, void *data, > + u16 datalen) > { > struct usb_device *usbdev = line6->usbdev; > int ret; > diff --git a/sound/usb/line6/driver.h b/sound/usb/line6/driver.h > index 8247a6b..603bdc4 100644 > --- a/sound/usb/line6/driver.h > +++ b/sound/usb/line6/driver.h > @@ -147,8 +147,8 @@ struct usb_line6 { > > extern char *line6_alloc_sysex_buffer(struct usb_line6 *line6, int code1, > int code2, int size); > -extern int line6_read_data(struct usb_line6 *line6, int address, void *data, > - size_t datalen); > +extern int line6_read_data(struct usb_line6 *line6, u16 address, void *data, > + u8 datalen); > extern int line6_read_serial_number(struct usb_line6 *line6, > int *serial_number); > extern int line6_send_raw_message_async(struct usb_line6 *line6, > @@ -161,8 +161,8 @@ extern void line6_start_timer(struct timer_list *timer, unsigned int msecs, > void (*function)(unsigned long), > unsigned long data); > extern int line6_version_request_async(struct usb_line6 *line6); > -extern int line6_write_data(struct usb_line6 *line6, int address, void *data, > - size_t datalen); > +extern int line6_write_data(struct usb_line6 *line6, u16 address, void *data, > + u16 datalen); > > int line6_probe(struct usb_interface *interface, > const struct usb_device_id *id, > -- > 2.1.0 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/