[PATCH v3 27/32] rtc: s5m: add support for S2MPG10 RTC

[Krzysztof Kozlowski]

On 03/04/2025 10:59, André Draszik wrote:
> Add support for Samsung's S2MPG10 PMIC RTC, which is similar to the
> existing PMIC RTCs supported by this driver.
> 
> S2MPG10 doesn't use I2C, so we expect the core driver to have created a
> regmap for us.
> 
> Additionally, it can be used for doing a cold-reset. If requested to do
> so (via DT), S2MPG10 is programmed with a watchdog configuration that
> will perform a full power cycle upon watchdog expiry.
> 
> Signed-off-by: André Draszik <andre.draszik at linaro.org>
> ---
>  drivers/rtc/rtc-s5m.c | 60 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 60 insertions(+)
> 
> diff --git a/drivers/rtc/rtc-s5m.c b/drivers/rtc/rtc-s5m.c
> index 86ccf666c68059408907c97f2647716ffaad10c6..0d8783577bab4f4ebe61050dbd68387d970773bd 100644
> --- a/drivers/rtc/rtc-s5m.c
> +++ b/drivers/rtc/rtc-s5m.c
> @@ -11,6 +11,7 @@
>  #include <linux/i2c.h>
>  #include <linux/bcd.h>
>  #include <linux/regmap.h>
> +#include <linux/reboot.h>

You just sorted the MFD headers in other patch, but here you break the
sorting.

For me patches resorting the addresses (as a separate patch) is churn,
but sure. If you do this however, it means you really believe that it is
worth to sort them, so you must keep that standard for all your further
patches in that series.

Rest is fine, so with this fixed:

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>

Best regards,
Krzysztof