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


Groups > linux.kernel > #1490001 > unrolled thread

[PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int

Started byMasahiro Yamada <yamada.masahiro@socionext.com>
First post2016-09-23 14:30 +0200
Last post2016-09-23 23:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int Masahiro Yamada <yamada.masahiro@socionext.com> - 2016-09-23 14:30 +0200
    Re: [PATCH] clk: change the type of clk_hw_onecell_data.num to  unsigned int Stephen Boyd <sboyd@codeaurora.org> - 2016-09-23 23:50 +0200

#1490001 — [PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2016-09-23 14:30 +0200
Subject[PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int
Message-ID<skxM5-3BX-7@gated-at.bofh.it>
The "num" is the number of clk_hw entries in the structure, so
"unsigned int" would be a better fit.  (size_t looks like data
size we count by byte.)

Besides, struct clk_onecell_data already uses unsigned int for
"clk_num".

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 include/linux/clk-provider.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
index a39c0c5..e91d31f 100644
--- a/include/linux/clk-provider.h
+++ b/include/linux/clk-provider.h
@@ -772,7 +772,7 @@ struct clk_onecell_data {
 };
 
 struct clk_hw_onecell_data {
-	size_t num;
+	unsigned int num;
 	struct clk_hw *hws[];
 };
 
-- 
1.9.1

[toc] | [next] | [standalone]


#1490475 — Re: [PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int

FromStephen Boyd <sboyd@codeaurora.org>
Date2016-09-23 23:50 +0200
SubjectRe: [PATCH] clk: change the type of clk_hw_onecell_data.num to unsigned int
Message-ID<skGw1-Ig-11@gated-at.bofh.it>
In reply to#1490001
On 09/23, Masahiro Yamada wrote:
> The "num" is the number of clk_hw entries in the structure, so
> "unsigned int" would be a better fit.  (size_t looks like data
> size we count by byte.)
> 
> Besides, struct clk_onecell_data already uses unsigned int for
> "clk_num".
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---

Seems fair.

Applied to clk-next

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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web