public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: linux-arm-kernel@lists.infradead.org
Cc: Mark Rutland <mark.rutland@arm.com>,
	Ankit Jindal <ankit.jindal@linaro.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Varka Bhadram <varkabhadram@gmail.com>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Guenter Roeck <linux@roeck-us.net>,
	"Hans J. Koch" <hjk@hansjkoch.de>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"patches@apm.com" <patches@apm.com>,
	Rob Herring <robh+dt@kernel.org>,
	Tushar Jagad <tushar.jagad@linaro.org>
Subject: Re: [PATCH v3 5/6] Documentation: dt-bindings: Add binding info for X-Gene QMTM UIO driver
Date: Tue, 21 Oct 2014 12:05:42 +0200	[thread overview]
Message-ID: <3725049.9xPbukhCva@wuerfel> (raw)
In-Reply-To: <20141021091412.GB15293@leverpostej>

On Tuesday 21 October 2014 10:14:12 Mark Rutland wrote:
> 
> > +The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
> > +and Traffic manager). It is a device for managing hardware queues.
> > +It also implements QoS among hardware queues hence term "traffic"
> > +manager is present in its name. QMTM UIO nodes are defined for user
> > +space access to this device using UIO framework.
> 
> The binding should describe the hardware, not the software. Please drop
> mention of UIO, userspace, etc.

I have a bad feeling about the entire idea of doing the UIO driver first,
there are too many unknowns here and we should not break the binding
when the proper driver gets added.

The X-Gene is meant for server workloads, so the UIO approach is not
a good longterm solution anyway. My impression is that it would be
better to first get the kernel driver for this hardware merged and
the binding fixed, and then a UIO stub could get added to that driver,
taking over the hardware by user space. This would also solve the
arbitration between the two driver implementations. It's also possible
that by that time, we will have a functional VFIO framework for
platform devices.

	Arnd

  reply	other threads:[~2014-10-21 10:05 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-21  5:56 [PATCH v3 0/6] UIO driver for APM X-Gene QMTM Ankit Jindal
2014-10-21  5:56 ` [PATCH v3 1/6] uio: code style cleanup Ankit Jindal
2014-10-21  5:56 ` [PATCH v3 2/6] uio: Add new UIO_MEM_PHYS_CACHE type for mem regions Ankit Jindal
2014-10-21  6:38   ` Kumar Gala
2014-10-31  9:30     ` Ankit Jindal
2014-10-31 13:39       ` Kumar Gala
     [not found]         ` <4B098F57-E202-412C-B54D-085A46792EDC-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2014-11-05 12:55           ` Ankit Jindal
     [not found]             ` <CAPTm99UUA0Oe40uH73Rj8_hvyMYwuxcKhFvU_KVZxmKpR1jn2Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-11-05 15:09               ` Kumar Gala
2014-11-10 11:53                 ` Ankit Jindal
2014-10-31  9:35     ` Russell King - ARM Linux
2014-10-21  5:56 ` [PATCH v3 3/6] Documentation: Update documentation for UIO_MEM_PHYS_CACHE Ankit Jindal
2014-10-21  5:56 ` [PATCH v3 4/6] uio: Add X-Gene QMTM UIO driver Ankit Jindal
2014-10-21  6:04   ` Varka Bhadram
2014-10-21  6:16     ` Ankit Jindal
2014-10-21  6:17       ` Varka Bhadram
2014-10-21  6:23         ` Ankit Jindal
2014-10-21  5:56 ` [PATCH v3 5/6] Documentation: dt-bindings: Add binding info for " Ankit Jindal
2014-10-21  9:14   ` Mark Rutland
2014-10-21 10:05     ` Arnd Bergmann [this message]
2014-10-31 10:04     ` Ankit Jindal
     [not found]       ` <CAPTm99X9k+cfELMLL8_PmYN6RNn7CL7KugnV5JKcW6Jqhs8idg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-10-31 10:09         ` Mark Rutland
2014-10-31 10:15           ` Ankit Jindal
2014-10-21  5:56 ` [PATCH v3 6/6] MAINTAINERS: Add entry for APM " Ankit Jindal

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=3725049.9xPbukhCva@wuerfel \
    --to=arnd@arndb.de \
    --cc=ankit.jindal@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hjk@hansjkoch.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=linux@roeck-us.net \
    --cc=mark.rutland@arm.com \
    --cc=patches@apm.com \
    --cc=robh+dt@kernel.org \
    --cc=tushar.jagad@linaro.org \
    --cc=varkabhadram@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox