All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Rosin <peda@axentia.se>
To: Fabio Estevam <festevam@gmail.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>, Takashi Iwai <tiwai@suse.com>,
	Nicolas Ferre <nicolas.ferre@microchip.com>,
	Russell King <linux@armlinux.org.uk>,
	Rob Herring <robh+dt@kernel.org>,
	Alexandre Belloni <alexandre.belloni@free-electrons.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 1/2] dt-bindings: trivial: add tfa9879 device
Date: Sun, 3 Dec 2017 07:59:04 +0100	[thread overview]
Message-ID: <8e295c61-089e-170d-bca2-42ded2021c99@axentia.se> (raw)
In-Reply-To: <CAOMZO5BA2ZbBnPVORALkEGV21j7H9GJW_2uVvtEEU=8qM599-w@mail.gmail.com>

On 2017-12-02 22:17, Fabio Estevam wrote:
> On Fri, Dec 1, 2017 at 8:44 PM, Peter Rosin <peda@axentia.se> wrote:
>> Add record for NXP TFA9879 Mono BTL Class D audio amplifier.
>>
>> Signed-off-by: Peter Rosin <peda@axentia.se>
>> ---
>>  Documentation/devicetree/bindings/trivial-devices.txt | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/trivial-devices.txt b/Documentation/devicetree/bindings/trivial-devices.txt
>> index af284fbd4d23..4cf30ce4e71f 100644
>> --- a/Documentation/devicetree/bindings/trivial-devices.txt
>> +++ b/Documentation/devicetree/bindings/trivial-devices.txt
>> @@ -157,6 +157,7 @@ nxp,pcf2127         Real-time clock
>>  nxp,pcf2129            Real-time clock
>>  nxp,pcf8563            Real-time clock/calendar
>>  nxp,pcf85063           Tiny Real-Time Clock
>> +nxp,tfa9879            Mono BTL Class D audio amplifier
> 
> This is already described at Documentation/devicetree/bindings/sound/tfa9879.txt

Right. However, the patch adding that should have been sent to me, the
maintainer of the driver. That is carefully recorded in MAINTAINERS. So,
forgive me for assuming that nothing had changed in the driver behind my
back.

Had that patch been sent my way as it should have been, I would have
insisted that maintenance of the bindings had been kept together with
the maintenance of the driver.

Cheers,
Peter

WARNING: multiple messages have this Message-ID (diff)
From: Peter Rosin <peda@axentia.se>
To: Fabio Estevam <festevam@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	Takashi Iwai <tiwai@suse.com>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>,
	Alexandre Belloni <alexandre.belloni@free-electrons.com>,
	Russell King <linux@armlinux.org.uk>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	Nicolas Ferre <nicolas.ferre@microchip.com>
Subject: Re: [PATCH 1/2] dt-bindings: trivial: add tfa9879 device
Date: Sun, 3 Dec 2017 07:59:04 +0100	[thread overview]
Message-ID: <8e295c61-089e-170d-bca2-42ded2021c99@axentia.se> (raw)
In-Reply-To: <CAOMZO5BA2ZbBnPVORALkEGV21j7H9GJW_2uVvtEEU=8qM599-w@mail.gmail.com>

On 2017-12-02 22:17, Fabio Estevam wrote:
> On Fri, Dec 1, 2017 at 8:44 PM, Peter Rosin <peda@axentia.se> wrote:
>> Add record for NXP TFA9879 Mono BTL Class D audio amplifier.
>>
>> Signed-off-by: Peter Rosin <peda@axentia.se>
>> ---
>>  Documentation/devicetree/bindings/trivial-devices.txt | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/trivial-devices.txt b/Documentation/devicetree/bindings/trivial-devices.txt
>> index af284fbd4d23..4cf30ce4e71f 100644
>> --- a/Documentation/devicetree/bindings/trivial-devices.txt
>> +++ b/Documentation/devicetree/bindings/trivial-devices.txt
>> @@ -157,6 +157,7 @@ nxp,pcf2127         Real-time clock
>>  nxp,pcf2129            Real-time clock
>>  nxp,pcf8563            Real-time clock/calendar
>>  nxp,pcf85063           Tiny Real-Time Clock
>> +nxp,tfa9879            Mono BTL Class D audio amplifier
> 
> This is already described at Documentation/devicetree/bindings/sound/tfa9879.txt

Right. However, the patch adding that should have been sent to me, the
maintainer of the driver. That is carefully recorded in MAINTAINERS. So,
forgive me for assuming that nothing had changed in the driver behind my
back.

Had that patch been sent my way as it should have been, I would have
insisted that maintenance of the bindings had been kept together with
the maintenance of the driver.

Cheers,
Peter

WARNING: multiple messages have this Message-ID (diff)
From: peda@axentia.se (Peter Rosin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] dt-bindings: trivial: add tfa9879 device
Date: Sun, 3 Dec 2017 07:59:04 +0100	[thread overview]
Message-ID: <8e295c61-089e-170d-bca2-42ded2021c99@axentia.se> (raw)
In-Reply-To: <CAOMZO5BA2ZbBnPVORALkEGV21j7H9GJW_2uVvtEEU=8qM599-w@mail.gmail.com>

On 2017-12-02 22:17, Fabio Estevam wrote:
> On Fri, Dec 1, 2017 at 8:44 PM, Peter Rosin <peda@axentia.se> wrote:
>> Add record for NXP TFA9879 Mono BTL Class D audio amplifier.
>>
>> Signed-off-by: Peter Rosin <peda@axentia.se>
>> ---
>>  Documentation/devicetree/bindings/trivial-devices.txt | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/trivial-devices.txt b/Documentation/devicetree/bindings/trivial-devices.txt
>> index af284fbd4d23..4cf30ce4e71f 100644
>> --- a/Documentation/devicetree/bindings/trivial-devices.txt
>> +++ b/Documentation/devicetree/bindings/trivial-devices.txt
>> @@ -157,6 +157,7 @@ nxp,pcf2127         Real-time clock
>>  nxp,pcf2129            Real-time clock
>>  nxp,pcf8563            Real-time clock/calendar
>>  nxp,pcf85063           Tiny Real-Time Clock
>> +nxp,tfa9879            Mono BTL Class D audio amplifier
> 
> This is already described at Documentation/devicetree/bindings/sound/tfa9879.txt

Right. However, the patch adding that should have been sent to me, the
maintainer of the driver. That is carefully recorded in MAINTAINERS. So,
forgive me for assuming that nothing had changed in the driver behind my
back.

Had that patch been sent my way as it should have been, I would have
insisted that maintenance of the bindings had been kept together with
the maintenance of the driver.

Cheers,
Peter

  reply	other threads:[~2017-12-03  6:59 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-01 22:44 [PATCH 0/2] Add devicetree for Axentia Nattis Peter Rosin
2017-12-01 22:44 ` Peter Rosin
2017-12-01 22:44 ` Peter Rosin
2017-12-01 22:44 ` [PATCH 1/2] dt-bindings: trivial: add tfa9879 device Peter Rosin
2017-12-01 22:44   ` Peter Rosin
2017-12-01 22:44   ` Peter Rosin
2017-12-02 21:17   ` [alsa-devel] " Fabio Estevam
2017-12-02 21:17     ` Fabio Estevam
2017-12-02 21:17     ` Fabio Estevam
2017-12-03  6:59     ` Peter Rosin [this message]
2017-12-03  6:59       ` Peter Rosin
2017-12-03  6:59       ` Peter Rosin
2017-12-03 11:12       ` Fabio Estevam
2017-12-03 11:12         ` Fabio Estevam
2017-12-03 11:12         ` Fabio Estevam
2017-12-03 20:53         ` Peter Rosin
2017-12-03 20:53           ` Peter Rosin
2017-12-03 20:53           ` Peter Rosin
2017-12-03 21:01           ` Fabio Estevam
2017-12-03 21:01             ` Fabio Estevam
2017-12-03 21:01             ` Fabio Estevam
2017-12-01 22:44 ` [PATCH 2/2] ARM: dts: at91: add devicetree for the Axentia Nattis with Natte power Peter Rosin
2017-12-01 22:44   ` Peter Rosin
2017-12-01 22:44   ` Peter Rosin
2017-12-02  1:05 ` [PATCH 0/2] Add devicetree for Axentia Nattis Javier Martinez Canillas
2017-12-02  1:05   ` Javier Martinez Canillas
2017-12-02  1:05   ` Javier Martinez Canillas
2017-12-02  7:58   ` Peter Rosin
2017-12-02  7:58     ` Peter Rosin
2017-12-02  7:58     ` Peter Rosin

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=8e295c61-089e-170d-bca2-42ded2021c99@axentia.se \
    --to=peda@axentia.se \
    --cc=alexandre.belloni@free-electrons.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=nicolas.ferre@microchip.com \
    --cc=robh+dt@kernel.org \
    --cc=tiwai@suse.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.