All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] PM / AVS: SmartReflex: remove unused function
@ 2018-10-06 16:40 ` Uwe Kleine-König
  0 siblings, 0 replies; 8+ messages in thread
From: Uwe Kleine-König @ 2018-10-06 16:40 UTC (permalink / raw)
  To: Kevin Hilman
  Cc: Nishanth Menon, LinAdmin, Uwe Kleine-König,
	linux-arm-kernel, linux-pm

From: Uwe Kleine-König <uwe@kleine-koenig.org>

omap_sr_register_pmic() was introduced in 2010 in commit

	984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")

. There was never any caller of this function in mainline resulting in a
warning

	sr_init: No PMIC hook to init smartreflex

for each machine where this driver is enabled. So remove the unused
function and the pr_warn.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
---
Hello,

I didn't test this change, didn't even try to compile it. Just read a
report[1] that this warning is annoying on an Marvell Armada based
machine running a Debian kernel.

Best regards
Uwe

[1] https://lists.debian.org/d6abf9d1-14bf-86fc-58a0-0f11ff1ac345@quickline.ch

 drivers/power/avs/smartreflex.c   | 31 -------------------------------
 include/linux/power/smartreflex.h |  5 -----
 2 files changed, 36 deletions(-)

diff --git a/drivers/power/avs/smartreflex.c b/drivers/power/avs/smartreflex.c
index cb0237143dbe..d29b6313cafc 100644
--- a/drivers/power/avs/smartreflex.c
+++ b/drivers/power/avs/smartreflex.c
@@ -36,7 +36,6 @@
 static LIST_HEAD(sr_list);
 
 static struct omap_sr_class_data *sr_class;
-static struct omap_sr_pmic_data *sr_pmic_data;
 static struct dentry		*sr_dbg_dir;
 
 static inline void sr_write_reg(struct omap_sr *sr, unsigned offset, u32 value)
@@ -779,25 +778,6 @@ void omap_sr_disable_reset_volt(struct voltagedomain *voltdm)
 	sr_class->disable(sr, 1);
 }
 
-/**
- * omap_sr_register_pmic() - API to register pmic specific info.
- * @pmic_data:	The structure containing pmic specific data.
- *
- * This API is to be called from the PMIC specific code to register with
- * smartreflex driver pmic specific info. Currently the only info required
- * is the smartreflex init on the PMIC side.
- */
-void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data)
-{
-	if (!pmic_data) {
-		pr_warn("%s: Trying to register NULL PMIC data structure with smartreflex\n",
-			__func__);
-		return;
-	}
-
-	sr_pmic_data = pmic_data;
-}
-
 /* PM Debug FS entries to enable and disable smartreflex. */
 static int omap_sr_autocomp_show(void *data, u64 *val)
 {
@@ -1064,17 +1044,6 @@ static int __init sr_init(void)
 {
 	int ret = 0;
 
-	/*
-	 * sr_init is a late init. If by then a pmic specific API is not
-	 * registered either there is no need for anything to be done on
-	 * the PMIC side or somebody has forgotten to register a PMIC
-	 * handler. Warn for the second condition.
-	 */
-	if (sr_pmic_data && sr_pmic_data->sr_pmic_init)
-		sr_pmic_data->sr_pmic_init();
-	else
-		pr_warn("%s: No PMIC hook to init smartreflex\n", __func__);
-
 	ret = platform_driver_register(&smartreflex_driver);
 	if (ret) {
 		pr_err("%s: platform driver register failed for SR\n",
diff --git a/include/linux/power/smartreflex.h b/include/linux/power/smartreflex.h
index 7b81dad712de..a586976f4784 100644
--- a/include/linux/power/smartreflex.h
+++ b/include/linux/power/smartreflex.h
@@ -303,9 +303,6 @@ void omap_sr_enable(struct voltagedomain *voltdm);
 void omap_sr_disable(struct voltagedomain *voltdm);
 void omap_sr_disable_reset_volt(struct voltagedomain *voltdm);
 
-/* API to register the pmic specific data with the smartreflex driver. */
-void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data);
-
 /* Smartreflex driver hooks to be called from Smartreflex class driver */
 int sr_enable(struct omap_sr *sr, unsigned long volt);
 void sr_disable(struct omap_sr *sr);
@@ -320,7 +317,5 @@ static inline void omap_sr_enable(struct voltagedomain *voltdm) {}
 static inline void omap_sr_disable(struct voltagedomain *voltdm) {}
 static inline void omap_sr_disable_reset_volt(
 		struct voltagedomain *voltdm) {}
-static inline void omap_sr_register_pmic(
-		struct omap_sr_pmic_data *pmic_data) {}
 #endif
 #endif
-- 
2.19.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH] PM / AVS: SmartReflex: remove unused function
@ 2018-10-06 16:40 ` Uwe Kleine-König
  0 siblings, 0 replies; 8+ messages in thread
From: Uwe Kleine-König @ 2018-10-06 16:40 UTC (permalink / raw)
  To: linux-arm-kernel

From: Uwe Kleine-K?nig <uwe@kleine-koenig.org>

omap_sr_register_pmic() was introduced in 2010 in commit

	984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")

. There was never any caller of this function in mainline resulting in a
warning

	sr_init: No PMIC hook to init smartreflex

for each machine where this driver is enabled. So remove the unused
function and the pr_warn.

Signed-off-by: Uwe Kleine-K?nig <uwe@kleine-koenig.org>
---
Hello,

I didn't test this change, didn't even try to compile it. Just read a
report[1] that this warning is annoying on an Marvell Armada based
machine running a Debian kernel.

Best regards
Uwe

[1] https://lists.debian.org/d6abf9d1-14bf-86fc-58a0-0f11ff1ac345 at quickline.ch

 drivers/power/avs/smartreflex.c   | 31 -------------------------------
 include/linux/power/smartreflex.h |  5 -----
 2 files changed, 36 deletions(-)

diff --git a/drivers/power/avs/smartreflex.c b/drivers/power/avs/smartreflex.c
index cb0237143dbe..d29b6313cafc 100644
--- a/drivers/power/avs/smartreflex.c
+++ b/drivers/power/avs/smartreflex.c
@@ -36,7 +36,6 @@
 static LIST_HEAD(sr_list);
 
 static struct omap_sr_class_data *sr_class;
-static struct omap_sr_pmic_data *sr_pmic_data;
 static struct dentry		*sr_dbg_dir;
 
 static inline void sr_write_reg(struct omap_sr *sr, unsigned offset, u32 value)
@@ -779,25 +778,6 @@ void omap_sr_disable_reset_volt(struct voltagedomain *voltdm)
 	sr_class->disable(sr, 1);
 }
 
-/**
- * omap_sr_register_pmic() - API to register pmic specific info.
- * @pmic_data:	The structure containing pmic specific data.
- *
- * This API is to be called from the PMIC specific code to register with
- * smartreflex driver pmic specific info. Currently the only info required
- * is the smartreflex init on the PMIC side.
- */
-void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data)
-{
-	if (!pmic_data) {
-		pr_warn("%s: Trying to register NULL PMIC data structure with smartreflex\n",
-			__func__);
-		return;
-	}
-
-	sr_pmic_data = pmic_data;
-}
-
 /* PM Debug FS entries to enable and disable smartreflex. */
 static int omap_sr_autocomp_show(void *data, u64 *val)
 {
@@ -1064,17 +1044,6 @@ static int __init sr_init(void)
 {
 	int ret = 0;
 
-	/*
-	 * sr_init is a late init. If by then a pmic specific API is not
-	 * registered either there is no need for anything to be done on
-	 * the PMIC side or somebody has forgotten to register a PMIC
-	 * handler. Warn for the second condition.
-	 */
-	if (sr_pmic_data && sr_pmic_data->sr_pmic_init)
-		sr_pmic_data->sr_pmic_init();
-	else
-		pr_warn("%s: No PMIC hook to init smartreflex\n", __func__);
-
 	ret = platform_driver_register(&smartreflex_driver);
 	if (ret) {
 		pr_err("%s: platform driver register failed for SR\n",
diff --git a/include/linux/power/smartreflex.h b/include/linux/power/smartreflex.h
index 7b81dad712de..a586976f4784 100644
--- a/include/linux/power/smartreflex.h
+++ b/include/linux/power/smartreflex.h
@@ -303,9 +303,6 @@ void omap_sr_enable(struct voltagedomain *voltdm);
 void omap_sr_disable(struct voltagedomain *voltdm);
 void omap_sr_disable_reset_volt(struct voltagedomain *voltdm);
 
-/* API to register the pmic specific data with the smartreflex driver. */
-void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data);
-
 /* Smartreflex driver hooks to be called from Smartreflex class driver */
 int sr_enable(struct omap_sr *sr, unsigned long volt);
 void sr_disable(struct omap_sr *sr);
@@ -320,7 +317,5 @@ static inline void omap_sr_enable(struct voltagedomain *voltdm) {}
 static inline void omap_sr_disable(struct voltagedomain *voltdm) {}
 static inline void omap_sr_disable_reset_volt(
 		struct voltagedomain *voltdm) {}
-static inline void omap_sr_register_pmic(
-		struct omap_sr_pmic_data *pmic_data) {}
 #endif
 #endif
-- 
2.19.0

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* Re: [PATCH] PM / AVS: SmartReflex: remove unused function
  2018-10-06 16:40 ` Uwe Kleine-König
@ 2018-10-16  9:09   ` Rafael J. Wysocki
  -1 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2018-10-16  9:09 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Nishanth Menon, Uwe Kleine-König, linux-pm, Kevin Hilman,
	LinAdmin, linux-arm-kernel

On Saturday, October 6, 2018 6:40:59 PM CEST Uwe Kleine-König wrote:
> From: Uwe Kleine-König <uwe@kleine-koenig.org>
> 
> omap_sr_register_pmic() was introduced in 2010 in commit
> 
> 	984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")
> 
> . There was never any caller of this function in mainline resulting in a
> warning
> 
> 	sr_init: No PMIC hook to init smartreflex
> 
> for each machine where this driver is enabled. So remove the unused
> function and the pr_warn.
> 
> Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
> ---
> Hello,
> 
> I didn't test this change, didn't even try to compile it. Just read a
> report[1] that this warning is annoying on an Marvell Armada based
> machine running a Debian kernel.
> 
> Best regards
> Uwe
> 
> [1] https://lists.debian.org/d6abf9d1-14bf-86fc-58a0-0f11ff1ac345@quickline.ch
> 
>  drivers/power/avs/smartreflex.c   | 31 -------------------------------
>  include/linux/power/smartreflex.h |  5 -----
>  2 files changed, 36 deletions(-)
> 
> diff --git a/drivers/power/avs/smartreflex.c b/drivers/power/avs/smartreflex.c
> index cb0237143dbe..d29b6313cafc 100644
> --- a/drivers/power/avs/smartreflex.c
> +++ b/drivers/power/avs/smartreflex.c
> @@ -36,7 +36,6 @@
>  static LIST_HEAD(sr_list);
>  
>  static struct omap_sr_class_data *sr_class;
> -static struct omap_sr_pmic_data *sr_pmic_data;
>  static struct dentry		*sr_dbg_dir;
>  
>  static inline void sr_write_reg(struct omap_sr *sr, unsigned offset, u32 value)
> @@ -779,25 +778,6 @@ void omap_sr_disable_reset_volt(struct voltagedomain *voltdm)
>  	sr_class->disable(sr, 1);
>  }
>  
> -/**
> - * omap_sr_register_pmic() - API to register pmic specific info.
> - * @pmic_data:	The structure containing pmic specific data.
> - *
> - * This API is to be called from the PMIC specific code to register with
> - * smartreflex driver pmic specific info. Currently the only info required
> - * is the smartreflex init on the PMIC side.
> - */
> -void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data)
> -{
> -	if (!pmic_data) {
> -		pr_warn("%s: Trying to register NULL PMIC data structure with smartreflex\n",
> -			__func__);
> -		return;
> -	}
> -
> -	sr_pmic_data = pmic_data;
> -}
> -
>  /* PM Debug FS entries to enable and disable smartreflex. */
>  static int omap_sr_autocomp_show(void *data, u64 *val)
>  {
> @@ -1064,17 +1044,6 @@ static int __init sr_init(void)
>  {
>  	int ret = 0;
>  
> -	/*
> -	 * sr_init is a late init. If by then a pmic specific API is not
> -	 * registered either there is no need for anything to be done on
> -	 * the PMIC side or somebody has forgotten to register a PMIC
> -	 * handler. Warn for the second condition.
> -	 */
> -	if (sr_pmic_data && sr_pmic_data->sr_pmic_init)
> -		sr_pmic_data->sr_pmic_init();
> -	else
> -		pr_warn("%s: No PMIC hook to init smartreflex\n", __func__);
> -
>  	ret = platform_driver_register(&smartreflex_driver);
>  	if (ret) {
>  		pr_err("%s: platform driver register failed for SR\n",
> diff --git a/include/linux/power/smartreflex.h b/include/linux/power/smartreflex.h
> index 7b81dad712de..a586976f4784 100644
> --- a/include/linux/power/smartreflex.h
> +++ b/include/linux/power/smartreflex.h
> @@ -303,9 +303,6 @@ void omap_sr_enable(struct voltagedomain *voltdm);
>  void omap_sr_disable(struct voltagedomain *voltdm);
>  void omap_sr_disable_reset_volt(struct voltagedomain *voltdm);
>  
> -/* API to register the pmic specific data with the smartreflex driver. */
> -void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data);
> -
>  /* Smartreflex driver hooks to be called from Smartreflex class driver */
>  int sr_enable(struct omap_sr *sr, unsigned long volt);
>  void sr_disable(struct omap_sr *sr);
> @@ -320,7 +317,5 @@ static inline void omap_sr_enable(struct voltagedomain *voltdm) {}
>  static inline void omap_sr_disable(struct voltagedomain *voltdm) {}
>  static inline void omap_sr_disable_reset_volt(
>  		struct voltagedomain *voltdm) {}
> -static inline void omap_sr_register_pmic(
> -		struct omap_sr_pmic_data *pmic_data) {}
>  #endif
>  #endif
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH] PM / AVS: SmartReflex: remove unused function
@ 2018-10-16  9:09   ` Rafael J. Wysocki
  0 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2018-10-16  9:09 UTC (permalink / raw)
  To: linux-arm-kernel

On Saturday, October 6, 2018 6:40:59 PM CEST Uwe Kleine-K?nig wrote:
> From: Uwe Kleine-K?nig <uwe@kleine-koenig.org>
> 
> omap_sr_register_pmic() was introduced in 2010 in commit
> 
> 	984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")
> 
> . There was never any caller of this function in mainline resulting in a
> warning
> 
> 	sr_init: No PMIC hook to init smartreflex
> 
> for each machine where this driver is enabled. So remove the unused
> function and the pr_warn.
> 
> Signed-off-by: Uwe Kleine-K?nig <uwe@kleine-koenig.org>
> ---
> Hello,
> 
> I didn't test this change, didn't even try to compile it. Just read a
> report[1] that this warning is annoying on an Marvell Armada based
> machine running a Debian kernel.
> 
> Best regards
> Uwe
> 
> [1] https://lists.debian.org/d6abf9d1-14bf-86fc-58a0-0f11ff1ac345 at quickline.ch
> 
>  drivers/power/avs/smartreflex.c   | 31 -------------------------------
>  include/linux/power/smartreflex.h |  5 -----
>  2 files changed, 36 deletions(-)
> 
> diff --git a/drivers/power/avs/smartreflex.c b/drivers/power/avs/smartreflex.c
> index cb0237143dbe..d29b6313cafc 100644
> --- a/drivers/power/avs/smartreflex.c
> +++ b/drivers/power/avs/smartreflex.c
> @@ -36,7 +36,6 @@
>  static LIST_HEAD(sr_list);
>  
>  static struct omap_sr_class_data *sr_class;
> -static struct omap_sr_pmic_data *sr_pmic_data;
>  static struct dentry		*sr_dbg_dir;
>  
>  static inline void sr_write_reg(struct omap_sr *sr, unsigned offset, u32 value)
> @@ -779,25 +778,6 @@ void omap_sr_disable_reset_volt(struct voltagedomain *voltdm)
>  	sr_class->disable(sr, 1);
>  }
>  
> -/**
> - * omap_sr_register_pmic() - API to register pmic specific info.
> - * @pmic_data:	The structure containing pmic specific data.
> - *
> - * This API is to be called from the PMIC specific code to register with
> - * smartreflex driver pmic specific info. Currently the only info required
> - * is the smartreflex init on the PMIC side.
> - */
> -void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data)
> -{
> -	if (!pmic_data) {
> -		pr_warn("%s: Trying to register NULL PMIC data structure with smartreflex\n",
> -			__func__);
> -		return;
> -	}
> -
> -	sr_pmic_data = pmic_data;
> -}
> -
>  /* PM Debug FS entries to enable and disable smartreflex. */
>  static int omap_sr_autocomp_show(void *data, u64 *val)
>  {
> @@ -1064,17 +1044,6 @@ static int __init sr_init(void)
>  {
>  	int ret = 0;
>  
> -	/*
> -	 * sr_init is a late init. If by then a pmic specific API is not
> -	 * registered either there is no need for anything to be done on
> -	 * the PMIC side or somebody has forgotten to register a PMIC
> -	 * handler. Warn for the second condition.
> -	 */
> -	if (sr_pmic_data && sr_pmic_data->sr_pmic_init)
> -		sr_pmic_data->sr_pmic_init();
> -	else
> -		pr_warn("%s: No PMIC hook to init smartreflex\n", __func__);
> -
>  	ret = platform_driver_register(&smartreflex_driver);
>  	if (ret) {
>  		pr_err("%s: platform driver register failed for SR\n",
> diff --git a/include/linux/power/smartreflex.h b/include/linux/power/smartreflex.h
> index 7b81dad712de..a586976f4784 100644
> --- a/include/linux/power/smartreflex.h
> +++ b/include/linux/power/smartreflex.h
> @@ -303,9 +303,6 @@ void omap_sr_enable(struct voltagedomain *voltdm);
>  void omap_sr_disable(struct voltagedomain *voltdm);
>  void omap_sr_disable_reset_volt(struct voltagedomain *voltdm);
>  
> -/* API to register the pmic specific data with the smartreflex driver. */
> -void omap_sr_register_pmic(struct omap_sr_pmic_data *pmic_data);
> -
>  /* Smartreflex driver hooks to be called from Smartreflex class driver */
>  int sr_enable(struct omap_sr *sr, unsigned long volt);
>  void sr_disable(struct omap_sr *sr);
> @@ -320,7 +317,5 @@ static inline void omap_sr_enable(struct voltagedomain *voltdm) {}
>  static inline void omap_sr_disable(struct voltagedomain *voltdm) {}
>  static inline void omap_sr_disable_reset_volt(
>  		struct voltagedomain *voltdm) {}
> -static inline void omap_sr_register_pmic(
> -		struct omap_sr_pmic_data *pmic_data) {}
>  #endif
>  #endif
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] PM / AVS: SmartReflex: remove unused function
  2018-10-16  9:09   ` Rafael J. Wysocki
@ 2018-11-14  9:42     ` Uwe Kleine-König
  -1 siblings, 0 replies; 8+ messages in thread
From: Uwe Kleine-König @ 2018-11-14  9:42 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Nishanth Menon, linux-pm, Kevin Hilman, kernel, LinAdmin,
	linux-arm-kernel

Hello,

On Tue, Oct 16, 2018 at 11:09:11AM +0200, Rafael J. Wysocki wrote:
> On Saturday, October 6, 2018 6:40:59 PM CEST Uwe Kleine-König wrote:
> > From: Uwe Kleine-König <uwe@kleine-koenig.org>
> > 
> > omap_sr_register_pmic() was introduced in 2010 in commit
> > 
> > 	984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")
> > 
> > . There was never any caller of this function in mainline resulting in a
> > warning
> > 
> > 	sr_init: No PMIC hook to init smartreflex
> > 
> > for each machine where this driver is enabled. So remove the unused
> > function and the pr_warn.
>
> Applied, thanks!

This patch didn't make it into next yet, is this expected?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH] PM / AVS: SmartReflex: remove unused function
@ 2018-11-14  9:42     ` Uwe Kleine-König
  0 siblings, 0 replies; 8+ messages in thread
From: Uwe Kleine-König @ 2018-11-14  9:42 UTC (permalink / raw)
  To: linux-arm-kernel

Hello,

On Tue, Oct 16, 2018 at 11:09:11AM +0200, Rafael J. Wysocki wrote:
> On Saturday, October 6, 2018 6:40:59 PM CEST Uwe Kleine-K?nig wrote:
> > From: Uwe Kleine-K?nig <uwe@kleine-koenig.org>
> > 
> > omap_sr_register_pmic() was introduced in 2010 in commit
> > 
> > 	984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")
> > 
> > . There was never any caller of this function in mainline resulting in a
> > warning
> > 
> > 	sr_init: No PMIC hook to init smartreflex
> > 
> > for each machine where this driver is enabled. So remove the unused
> > function and the pr_warn.
>
> Applied, thanks!

This patch didn't make it into next yet, is this expected?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] PM / AVS: SmartReflex: remove unused function
  2018-11-14  9:42     ` Uwe Kleine-König
@ 2018-11-15  2:28       ` Rafael J. Wysocki
  -1 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2018-11-15  2:28 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Nishanth Menon, Linux PM, Kevin Hilman, Rafael J. Wysocki,
	Sascha Hauer, linadmin, Linux ARM

On Wed, Nov 14, 2018 at 10:42 AM Uwe Kleine-König
<u.kleine-koenig@pengutronix.de> wrote:
>
> Hello,
>
> On Tue, Oct 16, 2018 at 11:09:11AM +0200, Rafael J. Wysocki wrote:
> > On Saturday, October 6, 2018 6:40:59 PM CEST Uwe Kleine-König wrote:
> > > From: Uwe Kleine-König <uwe@kleine-koenig.org>
> > >
> > > omap_sr_register_pmic() was introduced in 2010 in commit
> > >
> > >     984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")
> > >
> > > . There was never any caller of this function in mainline resulting in a
> > > warning
> > >
> > >     sr_init: No PMIC hook to init smartreflex
> > >
> > > for each machine where this driver is enabled. So remove the unused
> > > function and the pr_warn.
> >
> > Applied, thanks!
>
> This patch didn't make it into next yet, is this expected?

It did some time ago, but then the branch it's on was neglected by
mistake.  Sorry about that.

I will merge it into my linux-next branch next week.

Thanks,
Rafael

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH] PM / AVS: SmartReflex: remove unused function
@ 2018-11-15  2:28       ` Rafael J. Wysocki
  0 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2018-11-15  2:28 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Nov 14, 2018 at 10:42 AM Uwe Kleine-K?nig
<u.kleine-koenig@pengutronix.de> wrote:
>
> Hello,
>
> On Tue, Oct 16, 2018 at 11:09:11AM +0200, Rafael J. Wysocki wrote:
> > On Saturday, October 6, 2018 6:40:59 PM CEST Uwe Kleine-K?nig wrote:
> > > From: Uwe Kleine-K?nig <uwe@kleine-koenig.org>
> > >
> > > omap_sr_register_pmic() was introduced in 2010 in commit
> > >
> > >     984aa6dbf4ca ("OMAP3: PM: Adding smartreflex driver support.")
> > >
> > > . There was never any caller of this function in mainline resulting in a
> > > warning
> > >
> > >     sr_init: No PMIC hook to init smartreflex
> > >
> > > for each machine where this driver is enabled. So remove the unused
> > > function and the pr_warn.
> >
> > Applied, thanks!
>
> This patch didn't make it into next yet, is this expected?

It did some time ago, but then the branch it's on was neglected by
mistake.  Sorry about that.

I will merge it into my linux-next branch next week.

Thanks,
Rafael

^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-11-15  2:28 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-06 16:40 [PATCH] PM / AVS: SmartReflex: remove unused function Uwe Kleine-König
2018-10-06 16:40 ` Uwe Kleine-König
2018-10-16  9:09 ` Rafael J. Wysocki
2018-10-16  9:09   ` Rafael J. Wysocki
2018-11-14  9:42   ` Uwe Kleine-König
2018-11-14  9:42     ` Uwe Kleine-König
2018-11-15  2:28     ` Rafael J. Wysocki
2018-11-15  2:28       ` Rafael J. Wysocki

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.