On 16/07/2023 18:52, John Watts wrote: > Hello, > > On Sun, Jul 16, 2023 at 06:36:03PM +0200, Krzysztof Kozlowski wrote: >>> +static const struct sun4ican_quirks sun4ican_quirks_d1 = { >>> + .has_reset = true, >>> +}; >> >> Isn't this the same as previous? > > Yes, but I wanted to split up the new quirk in to its own patch. I don't understand why you need this new, duplicated entry. Aren't devices compatible? I also do not understand what 'own patch' has anything to do with it. > Is > there a better way of doing that? Devices are compatible, right? So express it in the bindings and DTS. Best regards, Krzysztof