Re: [PATCH] usb: renesas_usbhs: fix signed-unsigned return

2016-04-14 Thread Sudip Mukherjee
On Thursday 14 April 2016 04:25 PM, Felipe Balbi wrote: Hi, Yoshihiro Shimoda writes: From: Sudip Mukherjee Sent: Saturday, April 09, 2016 12:05 AM The return type of usbhsp_setup_pipecfg() was u16 but it was returning a negative value (-EINVAL). Instead

Re: [PATCH] usb: renesas_usbhs: fix signed-unsigned return

2016-04-14 Thread Sudip Mukherjee
On Thursday 14 April 2016 04:25 PM, Felipe Balbi wrote: Hi, Yoshihiro Shimoda writes: From: Sudip Mukherjee Sent: Saturday, April 09, 2016 12:05 AM The return type of usbhsp_setup_pipecfg() was u16 but it was returning a negative value (-EINVAL). Instead lets return a pointer to u16 which

RE: [PATCH] usb: renesas_usbhs: fix signed-unsigned return

2016-04-14 Thread Felipe Balbi
Hi, Yoshihiro Shimoda writes: >> From: Sudip Mukherjee >> Sent: Saturday, April 09, 2016 12:05 AM >> >> The return type of usbhsp_setup_pipecfg() was u16 but it was returning >> a negative value (-EINVAL). Instead lets return a pointer to u16 which >> will

RE: [PATCH] usb: renesas_usbhs: fix signed-unsigned return

2016-04-14 Thread Felipe Balbi
Hi, Yoshihiro Shimoda writes: >> From: Sudip Mukherjee >> Sent: Saturday, April 09, 2016 12:05 AM >> >> The return type of usbhsp_setup_pipecfg() was u16 but it was returning >> a negative value (-EINVAL). Instead lets return a pointer to u16 which >> will hold the value to be returned or in

RE: [PATCH] usb: renesas_usbhs: fix signed-unsigned return

2016-04-14 Thread Yoshihiro Shimoda
Hi, > From: Sudip Mukherjee > Sent: Saturday, April 09, 2016 12:05 AM > > The return type of usbhsp_setup_pipecfg() was u16 but it was returning > a negative value (-EINVAL). Instead lets return a pointer to u16 which > will hold the value to be returned or in case of error, return the > error

RE: [PATCH] usb: renesas_usbhs: fix signed-unsigned return

2016-04-14 Thread Yoshihiro Shimoda
Hi, > From: Sudip Mukherjee > Sent: Saturday, April 09, 2016 12:05 AM > > The return type of usbhsp_setup_pipecfg() was u16 but it was returning > a negative value (-EINVAL). Instead lets return a pointer to u16 which > will hold the value to be returned or in case of error, return the > error