All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: Jason Cooper <jason@lakedaemon.net>
Cc: Eduardo Valentin <edubezval@gmail.com>,
	Ezequiel Garcia <ezequiel.garcia@free-electrons.com>,
	Zhang Rui <rui.zhang@intel.com>,
	Gregory Clement <gregory.clement@free-electrons.com>,
	linux-pm@vger.kernel.org, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/2] thermal: armada: Remove support for A375-Z1 SoC
Date: Fri, 21 Nov 2014 22:51:22 +0100	[thread overview]
Message-ID: <20141121225122.1165b466@free-electrons.com> (raw)
In-Reply-To: <20141121201858.GB22670@titan.lakedaemon.net>

Dear Jason Cooper,

On Fri, 21 Nov 2014 15:18:58 -0500, Jason Cooper wrote:

> > First of, I would like to mention that best thing to avoid such
> > situation is to be careful when documenting dt entries that represent hw
> > that no one has access to (except internal people).
> 
> Agreed, mainline support for an SoC so early in it's lifetime was new
> for all of us.  Lesson learned.

So the suggestion would be to not document the DT bindings at all,
until we reach a "stable" hardware that is distributed externally?

Note that the Z1 stepping is not completely internal: a small
selection of early customers have access to it. But it normally never
ends up in final products, it's aimed at allowing those early customers
to start their development soon.

I don't mind adjusting how DT bindings are documented for such early
SoCs stepping. But I really believe it's important to have a way to
handle this situation nicely: we've been asking for years SoC vendors
to start upstreaming their code early. Now that they start to do it, we
shouldn't complain and instead adapt to this situation :-)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

WARNING: multiple messages have this Message-ID (diff)
From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] thermal: armada: Remove support for A375-Z1 SoC
Date: Fri, 21 Nov 2014 22:51:22 +0100	[thread overview]
Message-ID: <20141121225122.1165b466@free-electrons.com> (raw)
In-Reply-To: <20141121201858.GB22670@titan.lakedaemon.net>

Dear Jason Cooper,

On Fri, 21 Nov 2014 15:18:58 -0500, Jason Cooper wrote:

> > First of, I would like to mention that best thing to avoid such
> > situation is to be careful when documenting dt entries that represent hw
> > that no one has access to (except internal people).
> 
> Agreed, mainline support for an SoC so early in it's lifetime was new
> for all of us.  Lesson learned.

So the suggestion would be to not document the DT bindings at all,
until we reach a "stable" hardware that is distributed externally?

Note that the Z1 stepping is not completely internal: a small
selection of early customers have access to it. But it normally never
ends up in final products, it's aimed at allowing those early customers
to start their development soon.

I don't mind adjusting how DT bindings are documented for such early
SoCs stepping. But I really believe it's important to have a way to
handle this situation nicely: we've been asking for years SoC vendors
to start upstreaming their code early. Now that they start to do it, we
shouldn't complain and instead adapt to this situation :-)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2014-11-21 21:51 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-04 16:00 [PATCH 0/2] Farewell Armada 375 Z1 support Ezequiel Garcia
2014-11-04 16:00 ` Ezequiel Garcia
2014-11-04 16:00 ` [PATCH 1/2] thermal: armada: Remove support for A375-Z1 SoC Ezequiel Garcia
2014-11-04 16:00   ` Ezequiel Garcia
2014-11-07  3:26   ` Jason Cooper
2014-11-07  3:26     ` Jason Cooper
2014-11-07 12:41     ` Ezequiel Garcia
2014-11-07 12:41       ` Ezequiel Garcia
2014-11-07 12:59       ` Jason Cooper
2014-11-07 12:59         ` Jason Cooper
2014-11-07 22:27       ` Thomas Petazzoni
2014-11-07 22:27         ` Thomas Petazzoni
2014-11-09  3:16   ` Jason Cooper
2014-11-09  3:16     ` Jason Cooper
2014-11-20 19:38   ` Eduardo Valentin
2014-11-20 19:38     ` Eduardo Valentin
2014-11-21 20:18     ` Jason Cooper
2014-11-21 20:18       ` Jason Cooper
2014-11-21 21:51       ` Thomas Petazzoni [this message]
2014-11-21 21:51         ` Thomas Petazzoni
2014-11-21 22:05         ` Jason Cooper
2014-11-21 22:05           ` Jason Cooper
2014-11-21 22:31           ` Thomas Petazzoni
2014-11-21 22:31             ` Thomas Petazzoni
2014-11-04 16:00 ` [PATCH 2/2] ARM: mvebu: Remove thermal quirk for A375 Z1 revision Ezequiel Garcia
2014-11-04 16:00   ` Ezequiel Garcia
2014-11-09  3:38   ` Jason Cooper
2014-11-09  3:38     ` Jason Cooper

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=20141121225122.1165b466@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=andrew@lunn.ch \
    --cc=edubezval@gmail.com \
    --cc=ezequiel.garcia@free-electrons.com \
    --cc=gregory.clement@free-electrons.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=rui.zhang@intel.com \
    --cc=sebastian.hesselbarth@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 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.