Re: Additional patches in my watchdog-next branch

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Hello,

On Wed, 30 Dec 2015 21:30:08 +0800, Fu Wei wrote:

> > Maybe we should enforce that all users of the driver use two Device
> > Tree compatible strings:
> >
> >         compatible = "<vendor>,sbsa-gwdt", "arm,sbsa-gwdt";
> >
> > So that the SBSA watchdog driver can implement vendor specific quirks
> > if needed. Though it is true that the HW has some identification
> > registers that should allow to differentiate between the implementation.
> 
> I think the implementation can be different, but all the SBSA watchdog
> should be able to use the same driver.

Yes, they *should*, but as Guenter said, since the specification is not
very clear, it is quite likely that some HW implementations will not
behave exactly like others, so the driver will likely need to have some
quirks to adapt to those subtle differences between implementations.

> > For now, I'm personally fine with a SBSA driver that does not support
> > the pretimeout. Is anyone working on submitting something like this ?
> 
> I am working on a new non-pretimeout  SBSA driver.

Ok, great. Can you Cc: me when you send this new version ?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com