[PATCH 1/1] clk: sifive: append missing \n to messages
Heinrich Schuchardt
heinrich.schuchardt at canonical.com
Thu Apr 11 10:37:37 CEST 2024
On 11.04.24 05:13, Sean Anderson wrote:
> On 2/16/24 11:35, Heinrich Schuchardt wrote:
>> If multiple messages are written, line-feeds improve the readability.
>>
>> Fixes: c40b6df87fc0 ("clk: Add SiFive FU540 PRCI clock driver")
>> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
>> ---
>> drivers/clk/analogbits/wrpll-cln28hpc.c | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/clk/analogbits/wrpll-cln28hpc.c
>> b/drivers/clk/analogbits/wrpll-cln28hpc.c
>> index a3cb109d357..537c696b727 100644
>> --- a/drivers/clk/analogbits/wrpll-cln28hpc.c
>> +++ b/drivers/clk/analogbits/wrpll-cln28hpc.c
>> @@ -81,7 +81,7 @@ static int __wrpll_calc_filter_range(unsigned long
>> post_divr_freq)
>> {
>> if (post_divr_freq < MIN_POST_DIVR_FREQ ||
>> post_divr_freq > MAX_POST_DIVR_FREQ) {
>> - WARN(1, "%s: post-divider reference freq out of range: %lu",
>> + WARN(1, "%s: post-divider reference freq out of range: %lu\n",
>> __func__, post_divr_freq);
>> return -ERANGE;
>> }
>> @@ -229,7 +229,7 @@ int wrpll_configure_for_rate(struct wrpll_cfg *c,
>> u32 target_rate,
>> int range;
>> if (c->flags == 0) {
>> - WARN(1, "%s called with uninitialized PLL config", __func__);
>> + WARN(1, "%s called with uninitialized PLL config\n", __func__);
>> return -EINVAL;
>> }
>> @@ -335,7 +335,7 @@ unsigned long wrpll_calc_output_rate(const struct
>> wrpll_cfg *c,
>> u64 n;
>> if (c->flags & WRPLL_FLAGS_EXT_FEEDBACK_MASK) {
>> - WARN(1, "external feedback mode not yet supported");
>> + WARN(1, "external feedback mode not yet supported\n");
>> return ULONG_MAX;
>> }
>
> Reviewed-by: Sean Anderson <seanga2 at gmail.com>
>
> But maybe these should be dev_dbg?
The messages look like indicating misconfiguration. So the user should
see these.
The kernel code also uses WARN() (and also lacks the line-feeds).
CCing Paul as the Linux maintainer and author.
Best regards
Heinrich
More information about the U-Boot
mailing list