On 2023/2/2 6:46, Guenter Roeck wrote: > On Mon, Dec 19, 2022 at 05:42:32PM +0800, Xingyu Wu wrote: >> Add watchdog driver for the StarFive JH7110 SoC. >> >> Signed-off-by: Xingyu Wu <xingyu.wu@xxxxxxxxxxxxxxxx> >> + >>[...] >> + >> +static const struct watchdog_info starfive_wdt_ident = { >> + .options = OPTIONS, >> + .firmware_version = 0, > > It is not necessary to initilize a static variable with 0. > >> + .identity = "StarFive Watchdog", >> +}; Hi Guenter, It would be a compilation error if drop the '0'. I found that other files initialize this static variable as well. Best regards, Xingyu Wu