[PATCH v2] drivers: watchdog: Fix dev_read_addr error check

[Stefan Roese]

On 5/25/26 22:33, Francois Berder wrote:
> dev_read_addr does not return a void* but fdt_addr_t.
> Replace invalid usage of dev_read_addr by dev_read_addr_ptr.
> 
> v2:
>   - Replace dev_read_addr by dev_read_addr_ptr
>   - Change error to EINVAL
> 
> Signed-off-by: Francois Berder <fberder at outlook.fr>

Reviewed-by: Stefan Roese <stefan.roese at mailbox.org>

Thanks,
Stefan

> ---
>   drivers/watchdog/starfive_wdt.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/watchdog/starfive_wdt.c b/drivers/watchdog/starfive_wdt.c
> index ee9ec4cdc3a..d2c16150f4c 100644
> --- a/drivers/watchdog/starfive_wdt.c
> +++ b/drivers/watchdog/starfive_wdt.c
> @@ -290,9 +290,9 @@ static int starfive_wdt_of_to_plat(struct udevice *dev)
>   {
>   	struct starfive_wdt_priv *wdt = dev_get_priv(dev);
>   
> -	wdt->base = (void *)dev_read_addr(dev);
> +	wdt->base = dev_read_addr_ptr(dev);
>   	if (!wdt->base)
> -		return -ENODEV;
> +		return -EINVAL;
>   
>   	wdt->apb_clk = devm_clk_get(dev, "apb");
>   	if (IS_ERR(wdt->apb_clk))