[PATCH 1/2] arm: bcm283x: remove pointless empty hw_watchdog_disable

Stefan Roese sr at denx.de
Thu Jul 11 10:09:41 CEST 2024


On 7/11/24 09:54, Peter Robinson wrote:
> On Wed, 10 Jul 2024 at 22:17, Rasmus Villemoes
> <rasmus.villemoes at prevas.dk> wrote:
>>
>> This empty stub was originally added as one branch of an #ifdef in
>> commit 45a6d231b2f (bcm2835_wdt: support for the BCM2835/2836
>> watchdog). That incarnation of the rpi watchdog driver was later
>> removed in c7adc0b5f98 (watchdog: bcm2835_wdt: Remove unused BCM283x
>> watchdog driver and its references), but this now-pointless stub was
> 
> You don't need a - in "now-pointless" and I think "now unused" is a
> better term, please update the subject too.
> 
>> left behind. The later (re-)added rpi watchdog driver does not define
>> a hw_watchdog_disable() function, as that is properly integrated in
>> the watchdog framework (i.e. not of the hw_* kind).
>>
>> Since it wasn't declared static, the compiler cannot even elide
>> it (though it does get inlined into the caller here, and the
>> out-of-line definition does get gc'ed by the linker).
> 
> All the things in () don't add value to this, please remove them.

I am a big friend of extensive commit messages, so IMHO all this
should stay in. If a developer is not interested in such details
he can skip these infos. Still if somebody feels strong here to
drop these infos, then I won't object.

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan

>> Signed-off-by: Rasmus Villemoes <rasmus.villemoes at prevas.dk>
>> ---
>>   arch/arm/mach-bcm283x/reset.c | 7 ++-----
>>   1 file changed, 2 insertions(+), 5 deletions(-)
>>
>> diff --git a/arch/arm/mach-bcm283x/reset.c b/arch/arm/mach-bcm283x/reset.c
>> index 9199234917f..1dc7ce50d1d 100644
>> --- a/arch/arm/mach-bcm283x/reset.c
>> +++ b/arch/arm/mach-bcm283x/reset.c
>> @@ -25,8 +25,6 @@
>>   /* max ticks timeout */
>>   #define BCM2835_WDOG_MAX_TIMEOUT       0x000fffff
>>
>> -void hw_watchdog_disable(void) {}
>> -
>>   __efi_runtime_data struct bcm2835_wdog_regs *wdog_regs;
>>
>>   static void __efi_runtime
>> @@ -34,10 +32,9 @@ __reset_cpu(struct bcm2835_wdog_regs *wdog_regs, ulong ticks)
>>   {
>>          uint32_t rstc, timeout;
>>
>> -       if (ticks == 0) {
>> -               hw_watchdog_disable();
>> +       if (ticks == 0)
>>                  timeout = RESET_TIMEOUT;
>> -       } else
>> +       else
>>                  timeout = ticks & BCM2835_WDOG_MAX_TIMEOUT;
>>
>>          rstc = readl(&wdog_regs->rstc);
>> --
>> 2.45.2
>>

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de


More information about the U-Boot mailing list