mbox series

[GIT,PULL] firmware: arm_scmi: Fixes for v6.12

Message ID 20241015185128.1000604-1-sudeep.holla@arm.com (mailing list archive)
State New
Headers show
Series [GIT,PULL] firmware: arm_scmi: Fixes for v6.12 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git tags/scmi-fixes-6.12

Message

Sudeep Holla Oct. 15, 2024, 6:51 p.m. UTC
Hi ARM SoC Team,

Please pull !

Regards,
Sudeep

-->8

The following changes since commit 9852d85ec9d492ebef56dc5f229416c925758edc:

  Linux 6.12-rc1 (2024-09-29 15:06:19 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git tags/scmi-fixes-6.12

for you to fetch changes up to da1642bc97c4ef67f347edcd493bd0a52f88777b:

  firmware: arm_scmi: Queue in scmi layer for mailbox implementation (2024-10-14 21:36:46 +0100)

----------------------------------------------------------------
Arm SCMI fixes for v6.12

Couple of fixes to address the issues found and reported on Broadcom
STB platforms following the recent refactor of all the SCMI transports
as standalone drivers.

One of the issue is that the effective timeout value is much less than
the intended value due to the way mailbox messages are queues in the
mailbox framework. Since we block or serialise the shmem access anyway,
there is no point in utilizing mailbox queues. The issue is fixed with
exclusive lock on the channel when sending the message.

The other issues is actually non-issue for upstream, but the workaround
is just changing the link order of the transport drivers which enables
Broadcom STB platforms to run both upstream and custom downstream kernel
without any device tree changes. So pushing this to help them test upstream
seamlessly as it has no practical or theoretical impact for others.

There is also a fix to address possible double freeing of the name string
in scmi_debugfs_common_cleanup() when devm_add_action_or_reset() fails.

----------------------------------------------------------------
Florian Fainelli (1):
      firmware: arm_scmi: Give SMC transport precedence over mailbox

Justin Chen (1):
      firmware: arm_scmi: Queue in scmi layer for mailbox implementation

Su Hui (1):
      firmware: arm_scmi: Fix the double free in scmi_debugfs_common_setup()

 drivers/firmware/arm_scmi/driver.c             |  4 +---
 drivers/firmware/arm_scmi/transports/Makefile  |  6 +++--
 drivers/firmware/arm_scmi/transports/mailbox.c | 32 +++++++++++++++++---------
 3 files changed, 26 insertions(+), 16 deletions(-)