Re: [RESEND PATCH v2] char: xillybus: Add driver for XillyUSB (Xillybus variant for USB)

From: Greg KH
Date: Mon Feb 01 2021 - 11:07:19 EST


On Mon, Feb 01, 2021 at 05:56:19PM +0200, eli.billauer@xxxxxxxxx wrote:
> From: Eli Billauer <eli.billauer@xxxxxxxxx>
>
> The XillyUSB driver is the USB variant for the Xillybus FPGA IP core.
> Even though it presents a nearly identical API on the FPGA and host,
> it's almost a complete rewrite of the driver: The framework for exchanging
> data on a USB bus is fundamentally different from doing the same with a
> PCIe interface, which leaves very little in common between the existing
> driver and the new one for XillyUSB.
>
> Signed-off-by: Eli Billauer <eli.billauer@xxxxxxxxx>
> ---
>
> Notes:
> Changelog:
>
> v2:
> - Add comment in Kconfig file, saying XILLYUSB really doesn't depend
> on XILLYBUS (following comment by Randy Dunlap)
>
> - Use SEEK_* predefined constants instead of numbers

As brought up on the other list, this should either:
- use the same device nodes as the existing xillybus drivers do,
as you have the same api
- just do it all in userspace using libusb.

thanks,

greg k-h