devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Emil Medve <Emilian.Medve@Freescale.com>
To: Mark Rutland <mark.rutland@arm.com>
Cc: "scottwood@freescale.com" <scottwood@Freescale.com>,
	"galak@kernel.crashing.org" <galak@kernel.crashing.org>,
	"corbet@lwn.net" <corbet@lwn.net>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
	"galak@codeaurora.org" <galak@codeaurora.org>,
	Pawel Moll <Pawel.Moll@arm.com>,
	"Geoff.Thorpe@freescale.com" <Geoff.Thorpe@Freescale.com>,
	"linuxppc-dev@ozlabs.org" <linuxppc-dev@ozlabs.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>
Subject: Re: [PATCH 2/4] dt/bindings: Introduce the FSL QorIQ DPAA BMan portal(s)
Date: Fri, 24 Oct 2014 04:26:37 -0500	[thread overview]
Message-ID: <544A1B4D.3050904@Freescale.com> (raw)
In-Reply-To: <20141023111622.GB13690@leverpostej>

Hello Mark,


On 10/23/2014 06:16 AM, Mark Rutland wrote:
>>>> +NOTE:	The bindings described in this document are preliminary and subject to
>>>> +	change
>>>
>>> While we've tried that elsewhere, unstable DT bindings have been shown
>>> to be a major source of pain.
>>
>> Agreed
>>
>>> I'd feel rather uncomfortable accepting a
>>> binding that we already believe to be insufficient to describe the
>>> hardware.
>>>
>>> What do you expect to change?
>>
>> Related bindings seem incomplete. As such, the PAMU binding (pamu.txt)
>> covers incompletely a dynamic LIODN assignment/programming model. The
>> current driver uses a static assignment scheme that the binding needs to
>> include. I also suspect that once the driver starts supporting the
>> dynamic LIODN assignment/programming we might find some wrinkles
> 
> Ok. Given that the driver doesn't seem to be in mainline (per a quick
> grep), just put "DO NOT MERGE" or something like that in the commit
> message. Once this is merged you're going to need to support it.

We have a driver and once the bindings get accepted we'll (update the
driver as needed and) post the code


Cheers,

  parent reply	other threads:[~2014-10-24  9:26 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-22 14:09 [PATCH 1/4] dt/bindings: Introduce the FSL QorIQ DPAA BMan Emil Medve
2014-10-22 14:09 ` [PATCH 2/4] dt/bindings: Introduce the FSL QorIQ DPAA BMan portal(s) Emil Medve
2014-10-22 14:29   ` Mark Rutland
2014-10-22 20:04     ` Emil Medve
2014-10-23 11:16       ` Mark Rutland
2014-10-23 13:28         ` Geoff Thorpe
2014-10-24  9:26         ` Emil Medve [this message]
2014-10-28 18:09       ` Scott Wood
2014-10-22 14:09 ` [PATCH 3/4] dt/bindings: Introduce the FSL QorIQ DPAA QMan Emil Medve
2014-10-22 14:37   ` Mark Rutland
2014-10-22 20:05     ` Emil Medve
2014-10-23 11:26       ` Mark Rutland
2014-10-23 13:51         ` Geoff Thorpe
2014-10-24  9:53         ` Emil Medve
2014-10-22 14:09 ` [PATCH 4/4] dt/bindings: Introduce the FSL QorIQ DPAA QMan portal(s) Emil Medve
2014-10-28 18:27   ` Scott Wood
     [not found] ` <1413986972-621-1-git-send-email-Emilian.Medve-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org>
2014-10-28 14:36   ` [PATCH 1/4] dt/bindings: Introduce the FSL QorIQ DPAA BMan Kumar Gala
     [not found]     ` <EA33D307-CCDA-4A30-9386-46FA832FE0CA-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2014-10-28 18:08       ` Scott Wood
     [not found]     ` <1414519738.23458.84.camel__4795.38602890006$1414521743$gmane$org@snotra.buserror.net>
2014-10-29 21:40       ` Emil Medve
2014-10-29 22:16         ` Scott Wood
     [not found]         ` <1414620996.23458.141.camel__29590.7804662876$1414621051$gmane$org@snotra.buserror.net>
2014-10-30  4:32           ` Emil Medve
2014-10-30 14:51             ` Scott Wood
     [not found]             ` <1414680683.23458.148.camel__4514.07629666409$1414680744$gmane$org@snotra.buserror.net>
2014-10-30 16:19               ` Emil Medve
2014-10-30 16:29                 ` Scott Wood
     [not found]                 ` <1414686590.23458.151.camel__44619.4786033176$1414686664$gmane$org@snotra.buserror.net>
2014-10-30 16:45                   ` Emil Medve
2014-10-30 21:26                     ` Scott Wood
2014-10-30 21:30                       ` Emil Medve
2014-10-30 15:10         ` Varun Sethi
2014-10-28 18:19   ` Scott Wood

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=544A1B4D.3050904@Freescale.com \
    --to=emilian.medve@freescale.com \
    --cc=Geoff.Thorpe@Freescale.com \
    --cc=Pawel.Moll@arm.com \
    --cc=corbet@lwn.net \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=galak@kernel.crashing.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=linux-doc@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).