2021-07-14 10:20:29

by Geert Uytterhoeven

Subject: [PATCH resend] clk: Align provider-specific CLK_* bit definitions

The definition of CLK_MULTIPLIER_ROUND_CLOSEST is not aligned to the two
bit definitions next to it. A deeper inspection reveals that the
alignment of CLK_MULTIPLIER_ROUND_CLOSEST does match the most common
alignment.

Align the bit definitions for the various provider types throughout the
file at 40 columns, to increase uniformity.

Signed-off-by: Geert Uytterhoeven <[email protected]>
---
include/linux/clk-provider.h | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
index d83b829305c05d40..7be81d5fcf8c4844 100644
--- a/include/linux/clk-provider.h
+++ b/include/linux/clk-provider.h
@@ -342,7 +342,7 @@ struct clk_fixed_rate {
unsigned long flags;
};

-#define CLK_FIXED_RATE_PARENT_ACCURACY BIT(0)
+#define CLK_FIXED_RATE_PARENT_ACCURACY BIT(0)

extern const struct clk_ops clk_fixed_rate_ops;
struct clk_hw *__clk_hw_register_fixed_rate(struct device *dev,
@@ -1020,8 +1020,8 @@ struct clk_fractional_divider {

#define to_clk_fd(_hw) container_of(_hw, struct clk_fractional_divider, hw)

-#define CLK_FRAC_DIVIDER_ZERO_BASED BIT(0)
-#define CLK_FRAC_DIVIDER_BIG_ENDIAN BIT(1)
+#define CLK_FRAC_DIVIDER_ZERO_BASED BIT(0)
+#define CLK_FRAC_DIVIDER_BIG_ENDIAN BIT(1)

extern const struct clk_ops clk_fractional_divider_ops;
struct clk *clk_register_fractional_divider(struct device *dev,
@@ -1069,9 +1069,9 @@ struct clk_multiplier {

#define to_clk_multiplier(_hw) container_of(_hw, struct clk_multiplier, hw)

-#define CLK_MULTIPLIER_ZERO_BYPASS BIT(0)
+#define CLK_MULTIPLIER_ZERO_BYPASS BIT(0)
#define CLK_MULTIPLIER_ROUND_CLOSEST BIT(1)
-#define CLK_MULTIPLIER_BIG_ENDIAN BIT(2)
+#define CLK_MULTIPLIER_BIG_ENDIAN BIT(2)

extern const struct clk_ops clk_multiplier_ops;

--
2.25.1

2021-08-06 07:02:01

by Stephen Boyd

Subject: Re: [PATCH resend] clk: Align provider-specific CLK_* bit definitions

Quoting Geert Uytterhoeven (2021-07-14 03:17:36)
> The definition of CLK_MULTIPLIER_ROUND_CLOSEST is not aligned to the two
> bit definitions next to it. A deeper inspection reveals that the
> alignment of CLK_MULTIPLIER_ROUND_CLOSEST does match the most common
> alignment.
>
> Align the bit definitions for the various provider types throughout the
> file at 40 columns, to increase uniformity.
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> ---

Applied to clk-next