From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lina Iyer Subject: Re: [PATCH v2 02/14] dt/bindings: update binding for PM domain idle states Date: Thu, 4 Aug 2016 15:23:59 -0600 Message-ID: <20160804212359.GF1207@linaro.org> References: <1469829385-11511-1-git-send-email-lina.iyer@linaro.org> <1469829385-11511-3-git-send-email-lina.iyer@linaro.org> <20160804152453.GB1732@brendan-thinkstation> <20160804162844.GC1207@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Return-path: Content-Disposition: inline In-Reply-To: <20160804162844.GC1207@linaro.org> Sender: linux-pm-owner@vger.kernel.org To: Brendan Jackman Cc: ulf.hansson@linaro.org, khilman@kernel.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, Axel Haslam , devicetree@vger.kernel.org, Marc Titinger , Mark Rutland List-Id: devicetree@vger.kernel.org On Thu, Aug 04 2016 at 10:28 -0600, Lina Iyer wrote: >On Thu, Aug 04 2016 at 09:24 -0600, Brendan Jackman wrote: >>idle-states.txt (to which this file refers) says that idle state nodes must come >>under /cpus/idle-states. I don't think power domain idle states belong there, so >>the documentation should be updated to reflect that. >> >Absolutely right. This should be fixed. Sorry, I take it back. idle-states are cpuidle specific idle states and possibly include Platform coordinated cluster modes as well (which is fine). Its rightful place is under /cpus/idle-states. However, domain idle states that are compatible with "arm,idle-state" would not have idle-states as the parent node. They would be defined under the domain provider node and not the CPUs. Thanks, Lina