[PATCH v5 3/5] docs: staging: gpio-rpmsg: gpio over rpmsg bus
Linus Walleij
linus.walleij at linaro.org
Tue Nov 11 02:35:37 PST 2025
Hi Shenwei,
thanks for your patch!
Also, a big thanks for working on improving the standardization of rpmsg
so we can get some order here. This is very important work.
On Tue, Nov 4, 2025 at 9:34 PM Shenwei Wang <shenwei.wang at nxp.com> wrote:
> +- **Major**: Major version number.
> +
> +- **Minor**: Minor version number.
I'm not contesting these if they come from similar fields in other rpmsg
devices.
What I'm thinking is that the driver will eventually have to quirk around
bugs in the responding rpmsg CPU, and there will be bugs. This can end
up with this situation:
major,minor = (1,2) NXP implementation, no bug
major,minor = (1,2) Sharp implementation, no bug
major,minor = (1,2) Sony implementation, ooops this has a bug
What is the driver going to do here to work around that bug?
The scheme kind of suppose that all vendors use the same codebase
and they don't.
I would rather have:
**Vendor**: Vendor ID number (such as the PCI or USB ID)
**Version**: Vendor-specific version number (such as SW release)
This will make it possible to identify buggy firmware and apply
quirks.
My apologies if the rpmsg community has already thought about
this.
Bjorns input would be appreciated!
Yours,
Linus Walleij
More information about the linux-arm-kernel
mailing list