Re: [PATCH] fbdev: udlfb: Use usb_control_msg_send()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 19, 2023 at 11:16:25PM +0200, Helge Deller wrote:
> Use the newly introduced usb_control_msg_send() instead of usb_control_msg()
> when selecting the channel.
> 
> Signed-off-by: Helge Deller <deller@xxxxxx>
> ---
>  drivers/video/fbdev/udlfb.c | 14 +++-----------
>  1 file changed, 3 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/video/fbdev/udlfb.c b/drivers/video/fbdev/udlfb.c
> index 256d9b61f4ea..dabc30a09f96 100644
> --- a/drivers/video/fbdev/udlfb.c
> +++ b/drivers/video/fbdev/udlfb.c
> @@ -1543,24 +1543,16 @@ static const struct device_attribute fb_device_attrs[] = {
>  static int dlfb_select_std_channel(struct dlfb_data *dlfb)
>  {
>  	int ret;
> -	void *buf;
>  	static const u8 set_def_chn[] = {
>  				0x57, 0xCD, 0xDC, 0xA7,
>  				0x1C, 0x88, 0x5E, 0x15,
>  				0x60, 0xFE, 0xC6, 0x97,
>  				0x16, 0x3D, 0x47, 0xF2  };
> 
> -	buf = kmemdup(set_def_chn, sizeof(set_def_chn), GFP_KERNEL);
> -
> -	if (!buf)
> -		return -ENOMEM;
> -
> -	ret = usb_control_msg(dlfb->udev, usb_sndctrlpipe(dlfb->udev, 0),
> -			NR_USB_REQUEST_CHANNEL,
> +	ret = usb_control_msg_send(dlfb->udev, 0, NR_USB_REQUEST_CHANNEL,
>  			(USB_DIR_OUT | USB_TYPE_VENDOR), 0, 0,
> -			buf, sizeof(set_def_chn), USB_CTRL_SET_TIMEOUT);
> -
> -	kfree(buf);
> +			&set_def_chn, sizeof(set_def_chn), USB_CTRL_SET_TIMEOUT,
> +			GFP_KERNEL);
> 
>  	return ret;
>  }

Reviewed-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux