From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH 3/4] PM / Domains: Improve error handling while adding/removing devices Date: Wed, 29 Oct 2014 16:53:50 -0700 Message-ID: <7h8ujy5t69.fsf@deeprootsystems.com> References: <1414507090-516-1-git-send-email-ulf.hansson@linaro.org> <1414507090-516-4-git-send-email-ulf.hansson@linaro.org> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from mail-pd0-f173.google.com ([209.85.192.173]:42951 "EHLO mail-pd0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757540AbaJ2Xxz (ORCPT ); Wed, 29 Oct 2014 19:53:55 -0400 Received: by mail-pd0-f173.google.com with SMTP id v10so3941303pde.18 for ; Wed, 29 Oct 2014 16:53:55 -0700 (PDT) In-Reply-To: <1414507090-516-4-git-send-email-ulf.hansson@linaro.org> (Ulf Hansson's message of "Tue, 28 Oct 2014 15:38:09 +0100") Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Ulf Hansson Cc: "Rafael J. Wysocki" , Len Brown , Pavel Machek , linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, Geert Uytterhoeven , Alan Stern , Greg Kroah-Hartman , Tomasz Figa , Simon Horman , Magnus Damm , Ben Dooks , Kukjin Kim , Philipp Zabel , Mark Brown , Wolfram Sang , Russell King , Dmitry Torokhov , Jack Dai , Jinkun Hong , Aaron Lu , Sylwester Nawrocki Ulf Hansson writes: > To improve error handling while adding/removing devices from their PM > domains, we need to restructure the code a bit. Let's do this by moving > the device specific parts into a separate function. > > Signed-off-by: Ulf Hansson This looks like just restructuring, but with these kinds of patches, its hard to be sure that it's just refactoring, with no functional changes, so it's nice to be clear in the changelog whether there are (meant to be) any functional changes. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@kernel.org (Kevin Hilman) Date: Wed, 29 Oct 2014 16:53:50 -0700 Subject: [PATCH 3/4] PM / Domains: Improve error handling while adding/removing devices In-Reply-To: <1414507090-516-4-git-send-email-ulf.hansson@linaro.org> (Ulf Hansson's message of "Tue, 28 Oct 2014 15:38:09 +0100") References: <1414507090-516-1-git-send-email-ulf.hansson@linaro.org> <1414507090-516-4-git-send-email-ulf.hansson@linaro.org> Message-ID: <7h8ujy5t69.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Ulf Hansson writes: > To improve error handling while adding/removing devices from their PM > domains, we need to restructure the code a bit. Let's do this by moving > the device specific parts into a separate function. > > Signed-off-by: Ulf Hansson This looks like just restructuring, but with these kinds of patches, its hard to be sure that it's just refactoring, with no functional changes, so it's nice to be clear in the changelog whether there are (meant to be) any functional changes. Kevin