| Message ID | 1474633776-11586-1-git-send-email-yamada.masahiro@socionext.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Stephen Boyd |
| Headers | show |
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
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[]; };
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(-)