All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@baylibre.com>
To: Lina Iyer <lina.iyer@linaro.org>
Cc: ulf.hansson@linaro.org, rjw@rjwysocki.net,
	linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	andy.gross@linaro.org, sboyd@codeaurora.org,
	linux-arm-msm@vger.kernel.org, brendan.jackman@arm.com,
	lorenzo.pieralisi@arm.com, sudeep.holla@arm.com,
	Juri.Lelli@arm.com, devicetree@vger.kernel.org,
	Rob Herring <robh@kernel.org>
Subject: Re: [PATCH 3/4] dt-bindings: Update domain-idle-state binding to use correct compatibles
Date: Tue, 25 Oct 2016 13:49:34 -0700	[thread overview]
Message-ID: <7hh980taqp.fsf@baylibre.com> (raw)
In-Reply-To: <1477409199-52182-4-git-send-email-lina.iyer@linaro.org> (Lina Iyer's message of "Tue, 25 Oct 2016 08:26:38 -0700")

Lina Iyer <lina.iyer@linaro.org> writes:

> Update domain-idle-state binding to use "domain-idle-state" compatible
> from Documentation/devicetree/bindings/arm/idle-states.txt.
>
> Cc: <devicetree@vger.kernel.org>
> Cc: Rob Herring <robh@kernel.org>
> Suggested-by: Sudeep Holla <sudeep.holla@arm.com>
> Signed-off-by: Lina Iyer <lina.iyer@linaro.org>
> ---
>  Documentation/devicetree/bindings/power/power_domain.txt | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)

With no current users for this, I don't see the point of adding a
compatible now.

IMO, this should wait and be added with the identified user we can
discuss it then.

Kevin

WARNING: multiple messages have this Message-ID (diff)
From: khilman@baylibre.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/4] dt-bindings: Update domain-idle-state binding to use correct compatibles
Date: Tue, 25 Oct 2016 13:49:34 -0700	[thread overview]
Message-ID: <7hh980taqp.fsf@baylibre.com> (raw)
In-Reply-To: <1477409199-52182-4-git-send-email-lina.iyer@linaro.org> (Lina Iyer's message of "Tue, 25 Oct 2016 08:26:38 -0700")

Lina Iyer <lina.iyer@linaro.org> writes:

> Update domain-idle-state binding to use "domain-idle-state" compatible
> from Documentation/devicetree/bindings/arm/idle-states.txt.
>
> Cc: <devicetree@vger.kernel.org>
> Cc: Rob Herring <robh@kernel.org>
> Suggested-by: Sudeep Holla <sudeep.holla@arm.com>
> Signed-off-by: Lina Iyer <lina.iyer@linaro.org>
> ---
>  Documentation/devicetree/bindings/power/power_domain.txt | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)

With no current users for this, I don't see the point of adding a
compatible now.

IMO, this should wait and be added with the identified user we can
discuss it then.

Kevin

  parent reply	other threads:[~2016-10-25 20:49 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-25 15:26 [PATCH 0/4] PM / Domains: DT bindings for domain idle states and doc updates Lina Iyer
2016-10-25 15:26 ` Lina Iyer
2016-10-25 15:26 ` [PATCH 1/4] PM / doc: Update device documentation for devices in IRQ safe PM domains Lina Iyer
2016-10-25 15:26   ` Lina Iyer
2016-11-01  4:15   ` Rafael J. Wysocki
2016-11-01  4:15     ` Rafael J. Wysocki
2016-10-25 15:26 ` [PATCH 2/4] dt-bindings: add domain-idle-state compatible to arm,idle-state Lina Iyer
2016-10-25 15:26   ` [PATCH 2/4] dt-bindings: add domain-idle-state compatible to arm, idle-state Lina Iyer
     [not found] ` <1477409199-52182-1-git-send-email-lina.iyer-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2016-10-25 15:26   ` [PATCH 3/4] dt-bindings: Update domain-idle-state binding to use correct compatibles Lina Iyer
2016-10-25 15:26     ` Lina Iyer
2016-10-25 15:59     ` Sudeep Holla
2016-10-25 15:59       ` Sudeep Holla
2016-10-25 16:24       ` Lina Iyer
2016-10-25 16:24         ` Lina Iyer
     [not found]         ` <20161025162440.GA48977-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2016-10-25 16:52           ` Sudeep Holla
2016-10-25 16:52             ` Sudeep Holla
2016-10-25 20:49     ` Kevin Hilman [this message]
2016-10-25 20:49       ` Kevin Hilman
2016-10-26  8:57       ` Sudeep Holla
2016-10-26  8:57         ` Sudeep Holla
2016-10-31  2:50         ` Rob Herring
2016-10-31  2:50           ` Rob Herring
2016-10-25 15:26 ` [PATCH 4/4] PM / Domains: Use domain-idle-state compatible Lina Iyer
2016-10-25 15:26   ` Lina Iyer

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=7hh980taqp.fsf@baylibre.com \
    --to=khilman@baylibre.com \
    --cc=Juri.Lelli@arm.com \
    --cc=andy.gross@linaro.org \
    --cc=brendan.jackman@arm.com \
    --cc=devicetree@vger.kernel.org \
    --cc=lina.iyer@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=rjw@rjwysocki.net \
    --cc=robh@kernel.org \
    --cc=sboyd@codeaurora.org \
    --cc=sudeep.holla@arm.com \
    --cc=ulf.hansson@linaro.org \
    /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.