All of lore.kernel.org
 help / color / mirror / Atom feed
From: Emil Medve <Emilian.Medve@Freescale.com>
To: Mark Rutland <mark.rutland@arm.com>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Pawel Moll <Pawel.Moll@arm.com>,
	"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
	"Geoff.Thorpe@freescale.com" <Geoff.Thorpe@Freescale.com>,
	"corbet@lwn.net" <corbet@lwn.net>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
	"linuxppc-dev@ozlabs.org" <linuxppc-dev@ozlabs.org>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"galak@codeaurora.org" <galak@codeaurora.org>,
	"scottwood@freescale.com" <scottwood@Freescale.com>
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,

WARNING: multiple messages have this Message-ID (diff)
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:28 UTC|newest]

Thread overview: 58+ 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 ` 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:09   ` Emil Medve
2014-10-22 14:29   ` Mark Rutland
2014-10-22 14:29     ` Mark Rutland
2014-10-22 20:04     ` Emil Medve
2014-10-22 20:04       ` Emil Medve
2014-10-23 11:16       ` Mark Rutland
2014-10-23 11:16         ` Mark Rutland
2014-10-23 13:28         ` Geoff Thorpe
2014-10-23 13:28           ` Geoff Thorpe
2014-10-24  9:26         ` Emil Medve [this message]
2014-10-24  9:26           ` Emil Medve
2014-10-28 18:09       ` Scott Wood
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:09   ` Emil Medve
2014-10-22 14:37   ` Mark Rutland
2014-10-22 14:37     ` Mark Rutland
2014-10-22 20:05     ` Emil Medve
2014-10-22 20:05       ` Emil Medve
2014-10-23 11:26       ` Mark Rutland
2014-10-23 11:26         ` Mark Rutland
2014-10-23 13:51         ` Geoff Thorpe
2014-10-23 13:51           ` Geoff Thorpe
2014-10-24  9:53         ` Emil Medve
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-22 14:09   ` Emil Medve
2014-10-28 18:27   ` Scott Wood
2014-10-28 18:27     ` Scott Wood
2014-10-28 14:36 ` [PATCH 1/4] dt/bindings: Introduce the FSL QorIQ DPAA BMan Kumar Gala
2014-10-28 14:36   ` Kumar Gala
2014-10-28 18:08   ` Scott Wood
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 21:40       ` Emil Medve
2014-10-29 22:16       ` Scott Wood
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  4:32           ` Emil Medve
2014-10-30 14:51           ` Scott Wood
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:19               ` Emil Medve
2014-10-30 16:29               ` Scott Wood
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 16:45                   ` Emil Medve
2014-10-30 21:26                   ` Scott Wood
2014-10-30 21:26                     ` Scott Wood
2014-10-30 21:30                     ` Emil Medve
2014-10-30 21:30                       ` Emil Medve
2014-10-30 15:10       ` Varun Sethi
2014-10-30 15:10         ` Varun Sethi
2014-10-28 18:19 ` Scott Wood
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=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 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.