All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoffer Dall <christoffer.dall@linaro.org>
To: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
Cc: Scott Wood <scottwood@freescale.com>,
	Yoder Stuart-B08248 <B08248@freescale.com>,
	Alex Williamson <alex.williamson@redhat.com>,
	Kim Phillips <kim.phillips@linaro.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"a.motakis@virtualopensystems.com"
	<a.motakis@virtualopensystems.com>,
	"agraf@suse.de" <agraf@suse.de>,
	Wood Scott-B07421 <B07421@freescale.com>,
	Sethi Varun-B16395 <B16395@freescale.com>,
	Bhushan Bharat-R65777 <R65777@freescale.com>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"santosh.shukla@linaro.org" <santosh.shukla@linaro.org>,
	"kvm@vger.kernel.org" <kvm@vger.kernel.org>
Subject: Re: RFC: (re-)binding the VFIO platform driver to a platform device
Date: Wed, 2 Oct 2013 21:44:06 +0100	[thread overview]
Message-ID: <20131002204406.GE3029@lvm> (raw)
In-Reply-To: <20131002203943.GB10871@kroah.com>

On Wed, Oct 02, 2013 at 01:39:43PM -0700, gregkh@linuxfoundation.org wrote:
> On Wed, Oct 02, 2013 at 09:27:38PM +0100, Christoffer Dall wrote:
> > > > What you're suggesting would let users specify that
> > > > a serial driver should handle a NIC hardware, no?  That sounds much much
> > > > worse to me.
> > > 
> > > The flag (and wildcard match, if applicable) would be set by the driver,
> > > not by the user.  Whereas PCI's "new_id" and the "new_compatible" being
> > > suggested in this thread would allow exactly that, assuming the driver
> > > doesn't reject the device in the probe function.
> > > 
> > ok, fair enough, but still, I don't see the _generic_ need for having a
> > kernel feature that allows some random driver to bind to a device, but
> > then again, I'm not an authority in this area.
> 
> Again, this is already there, and has been for years with no problems,
> so I really don't understand the issue you have with it.
> 
As I said, I'm not an authority, just sounded to me like we were trying
to build something very generic to solve a very specific case, but I
certainly don't want to invent problems that don't exist.

-Christoffer

  reply	other threads:[~2013-10-02 20:44 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-01 18:38 RFC: (re-)binding the VFIO platform driver to a platform device Kim Phillips
2013-10-01 19:15 ` Scott Wood
2013-10-01 19:17   ` Scott Wood
2013-10-01 22:01     ` Kim Phillips
2013-10-01 21:59   ` Kim Phillips
2013-10-01 22:44     ` Scott Wood
2013-10-01 20:00 ` Greg Kroah-Hartman
2013-10-01 22:02   ` Kim Phillips
2013-10-02  1:53     ` Christoffer Dall
2013-10-02  2:35       ` Alex Williamson
2013-10-02 15:14         ` Christoffer Dall
2013-10-02 15:29           ` Alex Williamson
2013-10-02 18:25           ` Yoder Stuart-B08248
2013-10-02 18:32             ` Scott Wood
2013-10-02 18:43               ` Christoffer Dall
2013-10-02 20:04                 ` Kim Phillips
2013-10-02 20:13                   ` Christoffer Dall
2013-10-02 20:19                     ` Scott Wood
2013-10-02 20:14                 ` Scott Wood
2013-10-02 20:27                   ` Christoffer Dall
2013-10-02 20:39                     ` gregkh
2013-10-02 20:44                       ` Christoffer Dall [this message]
2013-10-02 20:37                 ` gregkh
2013-10-02 20:42                   ` Christoffer Dall
2013-10-02 21:08                   ` Scott Wood
2013-10-02 21:16                     ` gregkh
2013-10-02 21:35                       ` Scott Wood
2013-10-02 23:40                         ` gregkh
2013-10-03 18:33                           ` Scott Wood
2013-10-03 18:54                             ` gregkh
2013-10-03 19:11                               ` Scott Wood
2013-10-03 20:32                                 ` gregkh
2013-10-09 19:02                                   ` Yoder Stuart-B08248
2013-10-09 19:16                                     ` gregkh
2013-10-09 19:49                                       ` Scott Wood
2013-10-09 19:21                                     ` Scott Wood
2013-10-09 19:44                                       ` Yoder Stuart-B08248
2013-10-09 19:44                                         ` Yoder Stuart-B08248
2013-10-09 20:03                                         ` Scott Wood
2013-10-10  3:05                                           ` Kim Phillips
2013-10-10  8:01                                             ` Bhushan Bharat-R65777
2013-10-10 15:27                                               ` Scott Wood
2013-10-11  6:27                                                 ` [PATCH 1/4] driver core: Add new device_driver flag to allow binding via sysfs only Kim Phillips
2013-10-11  6:27                                                   ` [PATCH 2/4] driver core: platform: allow platform drivers to bind to any device Kim Phillips
2013-10-11  6:27                                                   ` [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only Kim Phillips
2013-10-11 20:43                                                     ` Scott Wood
2013-10-11 23:17                                                       ` Kim Phillips
2013-10-14 13:01                                                         ` Yoder Stuart-B08248
2013-10-14 17:13                                                           ` Scott Wood
2013-10-24 11:32                                                         ` Bhushan Bharat-R65777
2013-10-28 17:47                                                     ` Alex Williamson
2013-10-28 18:00                                                       ` Scott Wood
2013-10-28 18:09                                                         ` Scott Wood
2013-10-29  3:38                                                           ` Bhushan Bharat-R65777
2013-10-29  3:38                                                             ` Bhushan Bharat-R65777
2013-10-29  3:40                                                             ` Scott Wood
2013-10-29  3:52                                                               ` Bhushan Bharat-R65777
2013-10-29  3:52                                                                 ` Bhushan Bharat-R65777
2013-10-29  4:29                                                                 ` Scott Wood
2013-10-29  4:31                                                                   ` Bhushan Bharat-R65777
2013-10-29  4:31                                                                     ` Bhushan Bharat-R65777
2013-10-29  4:35                                                                     ` Scott Wood
2013-10-29  4:45                                                                       ` Bhushan Bharat-R65777
2013-10-29  4:45                                                                         ` Bhushan Bharat-R65777
2013-10-29  4:54                                                                         ` Scott Wood
2013-10-29  6:39                                                                           ` Bhushan Bharat-R65777
2013-10-29  6:39                                                                             ` Bhushan Bharat-R65777
2013-10-11  6:27                                                   ` [PATCH] VFIO: platform: allow the driver to bind to any device explicitly via sysfs Kim Phillips
2013-10-10  7:45                                           ` RFC: (re-)binding the VFIO platform driver to a platform device Bhushan Bharat-R65777
2013-10-10  7:45                                             ` Bhushan Bharat-R65777
2013-10-10 13:43                                             ` Yoder Stuart-B08248
2013-10-10 13:43                                               ` Yoder Stuart-B08248
2013-10-10 15:23                                             ` Scott Wood
2013-10-10 15:25                                               ` Bhushan Bharat-R65777
2013-10-10 15:25                                                 ` Bhushan Bharat-R65777

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=20131002204406.GE3029@lvm \
    --to=christoffer.dall@linaro.org \
    --cc=B07421@freescale.com \
    --cc=B08248@freescale.com \
    --cc=B16395@freescale.com \
    --cc=R65777@freescale.com \
    --cc=a.motakis@virtualopensystems.com \
    --cc=agraf@suse.de \
    --cc=alex.williamson@redhat.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=kim.phillips@linaro.org \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peter.maydell@linaro.org \
    --cc=santosh.shukla@linaro.org \
    --cc=scottwood@freescale.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.