[PATCH v5 0/2] add VCP mailbox driver

Jjian Zhou posted 2 patches 1 month, 1 week ago
There is a newer version of this series
.../mailbox/mediatek,mt8196-vcp-mbox.yaml     |  49 +++++
drivers/mailbox/Kconfig                       |   9 +
drivers/mailbox/Makefile                      |   2 +
drivers/mailbox/mtk-vcp-mailbox.c             | 174 ++++++++++++++++++
include/linux/mailbox/mtk-vcp-mailbox.h       |  32 ++++
5 files changed, 266 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mailbox/mediatek,mt8196-vcp-mbox.yaml
create mode 100644 drivers/mailbox/mtk-vcp-mailbox.c
create mode 100644 include/linux/mailbox/mtk-vcp-mailbox.h
[PATCH v5 0/2] add VCP mailbox driver
Posted by Jjian Zhou 1 month, 1 week ago
Hi everyone,

This is v5 of my VCP mailbox driver.

Changes since v5:
- bindngs:
  - Patch 1 fix 'make dt_binding_check' errors.

Changes since v4:
- binding:
  - Match the binding file name and compatible.
- mtk-vcp-mailbox.c:
  - Drop 'dev_dbg(dev, "MTK VCP mailbox initialized\n")'.
- Since the reviewer hopes to combine the VCP IPC driver and
  the VCP driver for a unified review, the original three patches
  have been split into two parts: the VCP mailbox driver and
  the binding remain together, while the VCP IPC driver is merged
  with the VCP driver and submitted as one.
- Link to v4
  https://lore.kernel.org/all/20250820094545.23821-1-jjian.zhou@mediatek.com/

Changes since v3:
- binding:
  - Remove unused lable '|' and 'vcp_mailbox0'.
- Link to v3
  https://lore.kernel.org/all/20250317110331.2776-1-jjian.zhou@mediatek.com/

Changes since v1:
- Link to v1
  https://lore.kernel.org/all/20250305082047.15746-1-jjian.zhou@mediatek.com/

In the v2 version, there is ongoing discussion about whether the VCP's
IPC should use mailbox or rpmsg. To prevent the discussion records
from being lost, the previous discussion link is attached.
https://lore.kernel.org/all/CAGXv+5FXqZb_v2dQNgCKbFpJrLhbVk3f0sWrrMCVk3jaWwoBqA@mail.gmail.com/

Jjian Zhou (2):
  dt-bindings: mailbox: mediatek,mt8196-vcp-mbox: add mtk vcp-mbox
    document
  mailbox: mediatek: Add mtk-vcp-mailbox driver

 .../mailbox/mediatek,mt8196-vcp-mbox.yaml     |  49 +++++
 drivers/mailbox/Kconfig                       |   9 +
 drivers/mailbox/Makefile                      |   2 +
 drivers/mailbox/mtk-vcp-mailbox.c             | 174 ++++++++++++++++++
 include/linux/mailbox/mtk-vcp-mailbox.h       |  32 ++++
 5 files changed, 266 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mailbox/mediatek,mt8196-vcp-mbox.yaml
 create mode 100644 drivers/mailbox/mtk-vcp-mailbox.c
 create mode 100644 include/linux/mailbox/mtk-vcp-mailbox.h

--
2.46.0
Re: [PATCH v5 0/2] add VCP mailbox driver
Posted by Jjian Zhou (周建) 3 weeks, 3 days ago
On Fri, 2025-08-22 at 10:12 +0800, Jjian Zhou wrote:

This email is just a reminder to review the patch:


https://lore.kernel.org/all/CAGXv+5GHSinkMHhBxRekfCxFK8my8JqeeRBsGXGuWDqH=MjmTQ@mail.gmail.com/

> Hi everyone,
> 
> This is v5 of my VCP mailbox driver.
> 
> Changes since v5:
> - bindngs:
>   - Patch 1 fix 'make dt_binding_check' errors.
> 
> Changes since v4:
> - binding:
>   - Match the binding file name and compatible.
> - mtk-vcp-mailbox.c:
>   - Drop 'dev_dbg(dev, "MTK VCP mailbox initialized\n")'.
> - Since the reviewer hopes to combine the VCP IPC driver and
>   the VCP driver for a unified review, the original three patches
>   have been split into two parts: the VCP mailbox driver and
>   the binding remain together, while the VCP IPC driver is merged
>   with the VCP driver and submitted as one.
> - Link to v4
>   
> https://lore.kernel.org/all/20250820094545.23821-1-jjian.zhou@mediatek.com/
> 
> Changes since v3:
> - binding:
>   - Remove unused lable '|' and 'vcp_mailbox0'.
> - Link to v3
>   
> https://lore.kernel.org/all/20250317110331.2776-1-jjian.zhou@mediatek.com/
> 
> Changes since v1:
> - Link to v1
>   
> https://lore.kernel.org/all/20250305082047.15746-1-jjian.zhou@mediatek.com/
> 
> In the v2 version, there is ongoing discussion about whether the
> VCP's
> IPC should use mailbox or rpmsg. To prevent the discussion records
> from being lost, the previous discussion link is attached.
> 
https://lore.kernel.org/all/CAGXv+5FXqZb_v2dQNgCKbFpJrLhbVk3f0sWrrMCVk3jaWwoBqA@mail.gmail.com/
> 
> Jjian Zhou (2):
>   dt-bindings: mailbox: mediatek,mt8196-vcp-mbox: add mtk vcp-mbox
>     document
>   mailbox: mediatek: Add mtk-vcp-mailbox driver
> 
>  .../mailbox/mediatek,mt8196-vcp-mbox.yaml     |  49 +++++
>  drivers/mailbox/Kconfig                       |   9 +
>  drivers/mailbox/Makefile                      |   2 +
>  drivers/mailbox/mtk-vcp-mailbox.c             | 174
> ++++++++++++++++++
>  include/linux/mailbox/mtk-vcp-mailbox.h       |  32 ++++
>  5 files changed, 266 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/mailbox/mediatek,mt8196-vcp-
> mbox.yaml
>  create mode 100644 drivers/mailbox/mtk-vcp-mailbox.c
>  create mode 100644 include/linux/mailbox/mtk-vcp-mailbox.h
> 
> --
> 2.46.0
>