On 15/07/2023 13:25, Jookia wrote: > From: John Watts <contact@xxxxxxxxxx> > > The controllers present in the D1 are extremely similar to the R40 > and require the same reset quirks. This alone can support sending > packets. An extra quirk is needed to support receiving packets. > > Signed-off-by: John Watts <contact@xxxxxxxxxx> > --- > drivers/net/can/Kconfig | 4 ++-- > drivers/net/can/sun4i_can.c | 9 ++++++++- > 2 files changed, 10 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/can/Kconfig b/drivers/net/can/Kconfig > index a5c5036dfb94..e626de33e735 100644 > --- a/drivers/net/can/Kconfig > +++ b/drivers/net/can/Kconfig > @@ -185,10 +185,10 @@ config CAN_SLCAN > > config CAN_SUN4I > tristate "Allwinner A10 CAN controller" > - depends on MACH_SUN4I || MACH_SUN7I || COMPILE_TEST > + depends on MACH_SUN4I || MACH_SUN7I || RISCV || COMPILE_TEST > help > Say Y here if you want to use CAN controller found on Allwinner > - A10/A20 SoCs. > + A10/A20/D1 SoCs. > > To compile this driver as a module, choose M here: the module will > be called sun4i_can. > diff --git a/drivers/net/can/sun4i_can.c b/drivers/net/can/sun4i_can.c > index 0827830bbf28..06f2cf05aaf5 100644 > --- a/drivers/net/can/sun4i_can.c > +++ b/drivers/net/can/sun4i_can.c > @@ -774,6 +774,10 @@ static const struct sun4ican_quirks sun4ican_quirks_r40 = { > .has_reset = true, > }; > > +static const struct sun4ican_quirks sun4ican_quirks_d1 = { > + .has_reset = true, > +}; Isn't this the same as previous? Best regards, Krzysztof