public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Eli Billauer <eli.billauer@gmail.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: arnd@arndb.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] char: xillybus: Add driver for XillyUSB (Xillybus variant for USB)
Date: Fri, 08 Jan 2021 12:59:14 +0200	[thread overview]
Message-ID: <5FF83B02.5000000@gmail.com> (raw)
In-Reply-To: <X/bzBobubF1C5x3Q@kroah.com>

Hello, Greg.

On 07/01/21 13:39, Greg KH wrote:
> My point is, do NOT have different file names.  Userspace should not
> care about the backing transport layer of a device.
>    
Regarding sound cards and such -- we agree perfectly. For a driver like 
XillyUSB, it's not necessarily clear what is correct formally. Either 
way, for XillyUSB the choice on this matter was practical.

First thing, this is what users expect. Xillybus is an FPGA project, and 
its users are hardware oriented. They've spent time designing the FPGA 
logic, and the connection with the host has been part of the effort. As 
the connection via PCIe or USB is part of the design, they definitely 
see these as different things, and don't expect them to appear to be the 
same.

Also, the application software is virtually always written specially for 
the project. Even if a specific hardware application is designed with 
both connection options possible (USB or PCIe), it will be very much 
desired to distinguish between them. For example, because odds are that 
the PCIe option will allow a much higher bandwidth capacity.

Another thing is that it's quite possible that some users will connect 
an FPGA board to a host through USB and PCIe simultaneously. For 
example, when developing an FPGA to work with the USB variant, but at 
the same time using the PCIe variant for passing debug data. In this 
case, using the same namespace for both variants will cause confusion.

So yes, calling the new XillyUSB device files xillyusb_* instead of 
xillybus_* may appear awkward. But I'm quite sure that no single user 
will ever thank me or anyone else for merging them. Maybe the formality 
is still important enough to do this anyhow...?

Thanks and regards,
    Eli

      reply	other threads:[~2021-01-08 11:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-13 17:05 [PATCH] char: xillybus: Add driver for XillyUSB (Xillybus variant for USB) eli.billauer
2020-12-13 17:22 ` Randy Dunlap
2020-12-13 17:30   ` Eli Billauer
2021-01-05 13:47 ` Greg KH
2021-01-06 14:03   ` Eli Billauer
2021-01-06 14:23     ` Greg KH
2021-01-07 11:18       ` Eli Billauer
2021-01-07 11:39         ` Greg KH
2021-01-08 10:59           ` Eli Billauer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5FF83B02.5000000@gmail.com \
    --to=eli.billauer@gmail.com \
    --cc=arnd@arndb.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox