All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-09  9:37 ` Weiyi Lu
  0 siblings, 0 replies; 12+ messages in thread
From: Weiyi Lu @ 2020-11-09  9:37 UTC (permalink / raw)
  To: Matthias Brugger, Stephen Boyd
  Cc: linux-arm-kernel, linux-kernel, linux-mediatek, linux-clk,
	srv_heupstream, stable, Weiyi Lu, Owen Chen

mtk_clk_register_mux() should be a static function

Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
Cc: <stable@vger.kernel.org>
Signed-off-by: Weiyi Lu <weiyi.lu@mediatek.com>
---
 drivers/clk/mediatek/clk-mux.c | 2 +-
 drivers/clk/mediatek/clk-mux.h | 4 ----
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/clk/mediatek/clk-mux.c b/drivers/clk/mediatek/clk-mux.c
index 14e127e..dcc1352 100644
--- a/drivers/clk/mediatek/clk-mux.c
+++ b/drivers/clk/mediatek/clk-mux.c
@@ -155,7 +155,7 @@ static int mtk_clk_mux_set_parent_setclr_lock(struct clk_hw *hw, u8 index)
 	.set_parent = mtk_clk_mux_set_parent_setclr_lock,
 };
 
-struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
+static struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
 				 struct regmap *regmap,
 				 spinlock_t *lock)
 {
diff --git a/drivers/clk/mediatek/clk-mux.h b/drivers/clk/mediatek/clk-mux.h
index f5625f4..8e2f927 100644
--- a/drivers/clk/mediatek/clk-mux.h
+++ b/drivers/clk/mediatek/clk-mux.h
@@ -77,10 +77,6 @@ struct mtk_mux {
 			_width, _gate, _upd_ofs, _upd,			\
 			CLK_SET_RATE_PARENT)
 
-struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
-				 struct regmap *regmap,
-				 spinlock_t *lock);
-
 int mtk_clk_register_muxes(const struct mtk_mux *muxes,
 			   int num, struct device_node *node,
 			   spinlock_t *lock,
-- 
1.8.1.1.dirty

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-09  9:37 ` Weiyi Lu
  0 siblings, 0 replies; 12+ messages in thread
From: Weiyi Lu @ 2020-11-09  9:37 UTC (permalink / raw)
  To: Matthias Brugger, Stephen Boyd
  Cc: Weiyi Lu, srv_heupstream, linux-kernel, stable, linux-mediatek,
	Owen Chen, linux-clk, linux-arm-kernel

mtk_clk_register_mux() should be a static function

Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
Cc: <stable@vger.kernel.org>
Signed-off-by: Weiyi Lu <weiyi.lu@mediatek.com>
---
 drivers/clk/mediatek/clk-mux.c | 2 +-
 drivers/clk/mediatek/clk-mux.h | 4 ----
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/clk/mediatek/clk-mux.c b/drivers/clk/mediatek/clk-mux.c
index 14e127e..dcc1352 100644
--- a/drivers/clk/mediatek/clk-mux.c
+++ b/drivers/clk/mediatek/clk-mux.c
@@ -155,7 +155,7 @@ static int mtk_clk_mux_set_parent_setclr_lock(struct clk_hw *hw, u8 index)
 	.set_parent = mtk_clk_mux_set_parent_setclr_lock,
 };
 
-struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
+static struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
 				 struct regmap *regmap,
 				 spinlock_t *lock)
 {
diff --git a/drivers/clk/mediatek/clk-mux.h b/drivers/clk/mediatek/clk-mux.h
index f5625f4..8e2f927 100644
--- a/drivers/clk/mediatek/clk-mux.h
+++ b/drivers/clk/mediatek/clk-mux.h
@@ -77,10 +77,6 @@ struct mtk_mux {
 			_width, _gate, _upd_ofs, _upd,			\
 			CLK_SET_RATE_PARENT)
 
-struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
-				 struct regmap *regmap,
-				 spinlock_t *lock);
-
 int mtk_clk_register_muxes(const struct mtk_mux *muxes,
 			   int num, struct device_node *node,
 			   spinlock_t *lock,
-- 
1.8.1.1.dirty
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-09  9:37 ` Weiyi Lu
  0 siblings, 0 replies; 12+ messages in thread
From: Weiyi Lu @ 2020-11-09  9:37 UTC (permalink / raw)
  To: Matthias Brugger, Stephen Boyd
  Cc: Weiyi Lu, srv_heupstream, linux-kernel, stable, linux-mediatek,
	Owen Chen, linux-clk, linux-arm-kernel

mtk_clk_register_mux() should be a static function

Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
Cc: <stable@vger.kernel.org>
Signed-off-by: Weiyi Lu <weiyi.lu@mediatek.com>
---
 drivers/clk/mediatek/clk-mux.c | 2 +-
 drivers/clk/mediatek/clk-mux.h | 4 ----
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/clk/mediatek/clk-mux.c b/drivers/clk/mediatek/clk-mux.c
index 14e127e..dcc1352 100644
--- a/drivers/clk/mediatek/clk-mux.c
+++ b/drivers/clk/mediatek/clk-mux.c
@@ -155,7 +155,7 @@ static int mtk_clk_mux_set_parent_setclr_lock(struct clk_hw *hw, u8 index)
 	.set_parent = mtk_clk_mux_set_parent_setclr_lock,
 };
 
-struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
+static struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
 				 struct regmap *regmap,
 				 spinlock_t *lock)
 {
diff --git a/drivers/clk/mediatek/clk-mux.h b/drivers/clk/mediatek/clk-mux.h
index f5625f4..8e2f927 100644
--- a/drivers/clk/mediatek/clk-mux.h
+++ b/drivers/clk/mediatek/clk-mux.h
@@ -77,10 +77,6 @@ struct mtk_mux {
 			_width, _gate, _upd_ofs, _upd,			\
 			CLK_SET_RATE_PARENT)
 
-struct clk *mtk_clk_register_mux(const struct mtk_mux *mux,
-				 struct regmap *regmap,
-				 spinlock_t *lock);
-
 int mtk_clk_register_muxes(const struct mtk_mux *muxes,
 			   int num, struct device_node *node,
 			   spinlock_t *lock,
-- 
1.8.1.1.dirty
_______________________________________________
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] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
  2020-11-09  9:37 ` Weiyi Lu
  (?)
@ 2020-11-09 10:20   ` Greg KH
  -1 siblings, 0 replies; 12+ messages in thread
From: Greg KH @ 2020-11-09 10:20 UTC (permalink / raw)
  To: Weiyi Lu
  Cc: Matthias Brugger, Stephen Boyd, linux-arm-kernel, linux-kernel,
	linux-mediatek, linux-clk, srv_heupstream, stable, Owen Chen

On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
> mtk_clk_register_mux() should be a static function
> 
> Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
> Cc: <stable@vger.kernel.org>

Why is this for stable trees?


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-09 10:20   ` Greg KH
  0 siblings, 0 replies; 12+ messages in thread
From: Greg KH @ 2020-11-09 10:20 UTC (permalink / raw)
  To: Weiyi Lu
  Cc: srv_heupstream, Stephen Boyd, linux-kernel, stable,
	linux-mediatek, Matthias Brugger, Owen Chen, linux-clk,
	linux-arm-kernel

On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
> mtk_clk_register_mux() should be a static function
> 
> Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
> Cc: <stable@vger.kernel.org>

Why is this for stable trees?


_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-09 10:20   ` Greg KH
  0 siblings, 0 replies; 12+ messages in thread
From: Greg KH @ 2020-11-09 10:20 UTC (permalink / raw)
  To: Weiyi Lu
  Cc: srv_heupstream, Stephen Boyd, linux-kernel, stable,
	linux-mediatek, Matthias Brugger, Owen Chen, linux-clk,
	linux-arm-kernel

On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
> mtk_clk_register_mux() should be a static function
> 
> Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
> Cc: <stable@vger.kernel.org>

Why is this for stable trees?


_______________________________________________
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] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
  2020-11-09 10:20   ` Greg KH
  (?)
@ 2020-11-10  1:38     ` Weiyi Lu
  -1 siblings, 0 replies; 12+ messages in thread
From: Weiyi Lu @ 2020-11-10  1:38 UTC (permalink / raw)
  To: Greg KH
  Cc: Matthias Brugger, Stephen Boyd, linux-arm-kernel, linux-kernel,
	linux-mediatek, linux-clk, srv_heupstream, stable, Owen Chen

On Mon, 2020-11-09 at 11:20 +0100, Greg KH wrote:
> On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
> > mtk_clk_register_mux() should be a static function
> > 
> > Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
> > Cc: <stable@vger.kernel.org>
> 
> Why is this for stable trees?

Hi Greg,

My Mistake. Indeed, this is not a bug fix for stable tree.
And there are simple questions.
Will I be allowed to keep the fixes tag in this patch to indicate the
mistakes we made in previous commit if it's not a bug fix for stable
tree?
And all I need to do now is to remove stable tree from cc list. Is it
correct?

Many thanks.

> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-10  1:38     ` Weiyi Lu
  0 siblings, 0 replies; 12+ messages in thread
From: Weiyi Lu @ 2020-11-10  1:38 UTC (permalink / raw)
  To: Greg KH
  Cc: srv_heupstream, Stephen Boyd, linux-kernel, stable,
	linux-mediatek, Matthias Brugger, Owen Chen, linux-clk,
	linux-arm-kernel

On Mon, 2020-11-09 at 11:20 +0100, Greg KH wrote:
> On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
> > mtk_clk_register_mux() should be a static function
> > 
> > Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
> > Cc: <stable@vger.kernel.org>
> 
> Why is this for stable trees?

Hi Greg,

My Mistake. Indeed, this is not a bug fix for stable tree.
And there are simple questions.
Will I be allowed to keep the fixes tag in this patch to indicate the
mistakes we made in previous commit if it's not a bug fix for stable
tree?
And all I need to do now is to remove stable tree from cc list. Is it
correct?

Many thanks.

> 

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-10  1:38     ` Weiyi Lu
  0 siblings, 0 replies; 12+ messages in thread
From: Weiyi Lu @ 2020-11-10  1:38 UTC (permalink / raw)
  To: Greg KH
  Cc: srv_heupstream, Stephen Boyd, linux-kernel, stable,
	linux-mediatek, Matthias Brugger, Owen Chen, linux-clk,
	linux-arm-kernel

On Mon, 2020-11-09 at 11:20 +0100, Greg KH wrote:
> On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
> > mtk_clk_register_mux() should be a static function
> > 
> > Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
> > Cc: <stable@vger.kernel.org>
> 
> Why is this for stable trees?

Hi Greg,

My Mistake. Indeed, this is not a bug fix for stable tree.
And there are simple questions.
Will I be allowed to keep the fixes tag in this patch to indicate the
mistakes we made in previous commit if it's not a bug fix for stable
tree?
And all I need to do now is to remove stable tree from cc list. Is it
correct?

Many thanks.

> 

_______________________________________________
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] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
  2020-11-10  1:38     ` Weiyi Lu
  (?)
@ 2020-11-10 10:50       ` Matthias Brugger
  -1 siblings, 0 replies; 12+ messages in thread
From: Matthias Brugger @ 2020-11-10 10:50 UTC (permalink / raw)
  To: Weiyi Lu, Greg KH
  Cc: Stephen Boyd, linux-arm-kernel, linux-kernel, linux-mediatek,
	linux-clk, srv_heupstream, stable, Owen Chen



On 10/11/2020 02:38, Weiyi Lu wrote:
> On Mon, 2020-11-09 at 11:20 +0100, Greg KH wrote:
>> On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
>>> mtk_clk_register_mux() should be a static function
>>>
>>> Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
>>> Cc: <stable@vger.kernel.org>
>>
>> Why is this for stable trees?
> 
> Hi Greg,
> 
> My Mistake. Indeed, this is not a bug fix for stable tree.
> And there are simple questions.
> Will I be allowed to keep the fixes tag in this patch to indicate the
> mistakes we made in previous commit if it's not a bug fix for stable
> tree?
> And all I need to do now is to remove stable tree from cc list. Is it
> correct?

That's my understanding, yes. Keep fixes tag but delete cc to stable.

Regards,
Matthias

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-10 10:50       ` Matthias Brugger
  0 siblings, 0 replies; 12+ messages in thread
From: Matthias Brugger @ 2020-11-10 10:50 UTC (permalink / raw)
  To: Weiyi Lu, Greg KH
  Cc: srv_heupstream, Stephen Boyd, linux-kernel, stable,
	linux-mediatek, Owen Chen, linux-clk, linux-arm-kernel



On 10/11/2020 02:38, Weiyi Lu wrote:
> On Mon, 2020-11-09 at 11:20 +0100, Greg KH wrote:
>> On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
>>> mtk_clk_register_mux() should be a static function
>>>
>>> Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
>>> Cc: <stable@vger.kernel.org>
>>
>> Why is this for stable trees?
> 
> Hi Greg,
> 
> My Mistake. Indeed, this is not a bug fix for stable tree.
> And there are simple questions.
> Will I be allowed to keep the fixes tag in this patch to indicate the
> mistakes we made in previous commit if it's not a bug fix for stable
> tree?
> And all I need to do now is to remove stable tree from cc list. Is it
> correct?

That's my understanding, yes. Keep fixes tag but delete cc to stable.

Regards,
Matthias

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function
@ 2020-11-10 10:50       ` Matthias Brugger
  0 siblings, 0 replies; 12+ messages in thread
From: Matthias Brugger @ 2020-11-10 10:50 UTC (permalink / raw)
  To: Weiyi Lu, Greg KH
  Cc: srv_heupstream, Stephen Boyd, linux-kernel, stable,
	linux-mediatek, Owen Chen, linux-clk, linux-arm-kernel



On 10/11/2020 02:38, Weiyi Lu wrote:
> On Mon, 2020-11-09 at 11:20 +0100, Greg KH wrote:
>> On Mon, Nov 09, 2020 at 05:37:07PM +0800, Weiyi Lu wrote:
>>> mtk_clk_register_mux() should be a static function
>>>
>>> Fixes: a3ae549917f16 ("clk: mediatek: Add new clkmux register API")
>>> Cc: <stable@vger.kernel.org>
>>
>> Why is this for stable trees?
> 
> Hi Greg,
> 
> My Mistake. Indeed, this is not a bug fix for stable tree.
> And there are simple questions.
> Will I be allowed to keep the fixes tag in this patch to indicate the
> mistakes we made in previous commit if it's not a bug fix for stable
> tree?
> And all I need to do now is to remove stable tree from cc list. Is it
> correct?

That's my understanding, yes. Keep fixes tag but delete cc to stable.

Regards,
Matthias

_______________________________________________
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] 12+ messages in thread

end of thread, other threads:[~2020-11-10 10:50 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-09  9:37 [PATCH] clk: mediatek: fix mtk_clk_register_mux() as static function Weiyi Lu
2020-11-09  9:37 ` Weiyi Lu
2020-11-09  9:37 ` Weiyi Lu
2020-11-09 10:20 ` Greg KH
2020-11-09 10:20   ` Greg KH
2020-11-09 10:20   ` Greg KH
2020-11-10  1:38   ` Weiyi Lu
2020-11-10  1:38     ` Weiyi Lu
2020-11-10  1:38     ` Weiyi Lu
2020-11-10 10:50     ` Matthias Brugger
2020-11-10 10:50       ` Matthias Brugger
2020-11-10 10:50       ` Matthias Brugger

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.