On 17.07.2023 08:41:07, Krzysztof Kozlowski wrote:
> 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?
According to patch 4/4 the devices are not compatible.
> I also do not understand what 'own patch' has anything to do with it.
Patch 4/4 adds a new quirk to the new device.
Jookia, please let the patches build on each other so that the resulting
kernel is consistent. The kernel at the state 3/4 will build and load
the driver on the D1, but it will not work, as the new quirk is missing.
Please exchange patches 3/4 and 4/4 (add the sun4ican_quirks_d1 in patch
4/4 only).
> > Is
> > there a better way of doing that?
>
> Devices are compatible, right? So express it in the bindings and DTS.
regards,
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung Nürnberg | Phone: +49-5121-206917-129 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 |
Attachment:
signature.asc
Description: PGP signature
