mirror of
https://github.com/torvalds/linux.git
synced 2024-11-22 04:38:03 +00:00
rtc: rx8111: demote warnings to debug level
The proper way for userspace to react on a read time error is to have a look at the voltage low information. There is no point in cluttering dmesg as it is often not even visible to the end user. Reviewed-by: Waqar Hameed <waqar.hameed@axis.com> Link: https://lore.kernel.org/r/20240417191937.33790-1-alexandre.belloni@bootlin.com Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
This commit is contained in:
parent
6a216cbadb
commit
b7ff9efebc
@ -170,13 +170,13 @@ static int rx8111_read_time(struct device *dev, struct rtc_time *tm)
|
||||
}
|
||||
|
||||
if (FIELD_GET(RX8111_FLAG_XST_BIT, regval)) {
|
||||
dev_warn(data->dev,
|
||||
dev_dbg(data->dev,
|
||||
"Crystal oscillation stopped, time is not reliable\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
if (FIELD_GET(RX8111_FLAG_VLF_BIT, regval)) {
|
||||
dev_warn(data->dev,
|
||||
dev_dbg(data->dev,
|
||||
"Low voltage detected, time is not reliable\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
@ -188,7 +188,7 @@ static int rx8111_read_time(struct device *dev, struct rtc_time *tm)
|
||||
}
|
||||
|
||||
if (regval) {
|
||||
dev_warn(data->dev, "Clock stopped, time is not reliable\n");
|
||||
dev_dbg(data->dev, "Clock stopped, time is not reliable\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user