Messages in this thread | | | From | Flavio Suligoi <> | Subject | RE: [PATCH v1 1/2] watchdog: add global watchdog kernel module parameters structure | Date | Tue, 9 Mar 2021 10:38:48 +0000 |
| |
> Hi, >
Hi Randy,
> On 3/8/21 3:21 AM, Flavio Suligoi wrote: > > Different watchdog modules frequently require the same type of > > parameters (for example: timeout, nowayout feature, start wdog on > > module insertion, etc.).
...
> > +In this way, each driver can read these "global" parameters and then, > > +if needed, can implement them, according to the particular hw > > +watchdog > > Please spell out "hardware" (not "hw").
Ok!
> > > +characteristic. > > + > > +Using this approach, it is possible reduce some duplicate code in the > > +*new* > > possible to reduce
Thanks!
...
> > + * Instead of add this kind of module parameters independently to > > +each driver, > > adding
Thanks!
... > > + * if needed, implements them, according to the particular hw > > + watchdog > > s/hw/hardware/
Ok
...
> If I were doing (or using) this, I would probably want 'test_mode' and > 'verbosity' to be unsigned int masks instead of a bool, so that there could be > multiple types of test_mode or verbosity.
I used bool as already done in some watchdog drivers, but your suggestion is better, thanks.
> That's something that some other subsystems do, but maybe watchdog is > simple enough that it's not needed. > If it is needed, then we are back to each driver doing it its own way (until this > patch is updated). > > > HTH. thanks. > -- > ~Randy
Thanks Randy.
Regards, Flavio
| |