Re: [PATCH v5] can: virtio: Initial virtio CAN driver.

Marc Kleine-Budde
2025-03-12T13:36:38Z

[-- Attachment #1: Type: text/plain, Size: 1929 bytes --]

On 12.03.2025 14:28:10, Matias Ezequiel Vara Larsen wrote:
> On Wed, Mar 12, 2025 at 11:41:26AM +0100, Marc Kleine-Budde wrote:
> > On 12.03.2025 11:31:12, Matias Ezequiel Vara Larsen wrote:
> > > On Thu, Feb 01, 2024 at 07:57:45PM +0100, Harald Mommer wrote:
> > > > Hello,
> > > >
> > > > I thought there would be some more comments coming and I could address
> > > > everything in one chunk. Not the case, besides your comments silence.
> > > >
> > > > On 08.01.24 20:34, Christophe JAILLET wrote:
> > > > >
> > > > > Hi,
> > > > > a few nits below, should there be a v6.
> > > > >
> > > >
> > > > I'm sure there will be but not so soon. Probably after acceptance of the
> > > > virtio CAN specification or after change requests to the specification are
> > > > received and the driver has to be adapted to an updated draft.
> > > >
> > > What is the status of this series?
> >
> > There has been no movement from the Linux side. The patch series is
> > quite extensive. To get this mainline, we need not only a proper Linux
> > CAN driver, but also a proper VirtIO specification.
>
> Thanks for your answer. AFAIK the spec has been merged (see
> https://github.com/oasis-tcs/virtio-spec/tree/virtio-1.4).

Yes, the spec was merged. I think it was written with a specific
use-case (IIRC: automotive, Linux on-top of a specific hypervisor) in
mind, in Linux we have other use cases that might not be covered.

> > This whole project is too big for me to do it as a collaborative
> > effort.
>
> What do you mean?

I mean the driver is too big to review on a non-paid community based
effort.

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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]