Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1420398 > unrolled thread

[PATCH] mfd: qcom_rpm: Remove unused define

Started byLinus Walleij <linus.walleij@linaro.org>
First post2016-06-13 02:40 +0200
Last post2016-06-15 16:10 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] mfd: qcom_rpm: Remove unused define Linus Walleij <linus.walleij@linaro.org> - 2016-06-13 02:40 +0200
    Re: [PATCH] mfd: qcom_rpm: Remove unused define Stephen Boyd <sboyd@codeaurora.org> - 2016-06-14 19:50 +0200
    Re: [PATCH] mfd: qcom_rpm: Remove unused define Lee Jones <lee.jones@linaro.org> - 2016-06-15 16:10 +0200

#1420398 — [PATCH] mfd: qcom_rpm: Remove unused define

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-06-13 02:40 +0200
Subject[PATCH] mfd: qcom_rpm: Remove unused define
Message-ID<rJo53-7kX-13@gated-at.bofh.it>
This define RPM_SIGNAL probably pertains to the IPC signal which
we ended up fetching from the device tree instead. the define is
unused, remove it.

Cc: Stephen Boyd <sboyd@codeaurora.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Andy Gross <andy.gross@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
 drivers/mfd/qcom_rpm.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c
index 1be47ad6441b..4f589cf75549 100644
--- a/drivers/mfd/qcom_rpm.c
+++ b/drivers/mfd/qcom_rpm.c
@@ -70,8 +70,6 @@ struct qcom_rpm {
 #define RPM_NOTIFICATION	BIT(30)
 #define RPM_REJECTED		BIT(31)
 
-#define RPM_SIGNAL		BIT(2)
-
 static const struct qcom_rpm_resource apq8064_rpm_resource_table[] = {
 	[QCOM_RPM_CXO_CLK] =			{ 25, 9, 5, 1 },
 	[QCOM_RPM_PXO_CLK] =			{ 26, 10, 6, 1 },
-- 
2.4.11

[toc] | [next] | [standalone]


#1422144

FromStephen Boyd <sboyd@codeaurora.org>
Date2016-06-14 19:50 +0200
Message-ID<rK0Do-80b-49@gated-at.bofh.it>
In reply to#1420398
On 06/13, Linus Walleij wrote:
> This define RPM_SIGNAL probably pertains to the IPC signal which
> we ended up fetching from the device tree instead. the define is
> unused, remove it.
> 
> Cc: Stephen Boyd <sboyd@codeaurora.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Andy Gross <andy.gross@linaro.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@codeaurora.org>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [prev] | [next] | [standalone]


#1423059

FromLee Jones <lee.jones@linaro.org>
Date2016-06-15 16:10 +0200
Message-ID<rKjG2-3xJ-23@gated-at.bofh.it>
In reply to#1420398
On Mon, 13 Jun 2016, Linus Walleij wrote:

> This define RPM_SIGNAL probably pertains to the IPC signal which
> we ended up fetching from the device tree instead. the define is
> unused, remove it.
> 
> Cc: Stephen Boyd <sboyd@codeaurora.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Andy Gross <andy.gross@linaro.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
>  drivers/mfd/qcom_rpm.c | 2 --
>  1 file changed, 2 deletions(-)

Applied, thanks.

> diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c
> index 1be47ad6441b..4f589cf75549 100644
> --- a/drivers/mfd/qcom_rpm.c
> +++ b/drivers/mfd/qcom_rpm.c
> @@ -70,8 +70,6 @@ struct qcom_rpm {
>  #define RPM_NOTIFICATION	BIT(30)
>  #define RPM_REJECTED		BIT(31)
>  
> -#define RPM_SIGNAL		BIT(2)
> -
>  static const struct qcom_rpm_resource apq8064_rpm_resource_table[] = {
>  	[QCOM_RPM_CXO_CLK] =			{ 25, 9, 5, 1 },
>  	[QCOM_RPM_PXO_CLK] =			{ 26, 10, 6, 1 },

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web