[PATCH v3] mailbox: remove superfluous internal header
Jassi Brar
jassisinghbrar at gmail.com
Sun Mar 29 09:18:36 PDT 2026
On Fri, Mar 27, 2026 at 10:11 AM Wolfram Sang
<wsa+renesas at sang-engineering.com> wrote:
>
> Quite some controller drivers use the defines from the internal header
> already. This prevents controller drivers outside the mailbox directory.
> Move the defines to the public controller header to allow this again as
> the defines are not strictly internal anyhow.
>
> Signed-off-by: Wolfram Sang <wsa+renesas at sang-engineering.com>
> Reviewed-by: Sudeep Holla <sudeep.holla at kernel.org>
> Reviewed-by: Daniel Baluta <daniel.baluta at nxp.com>
> ---
>
> Changes since v2:
> * rebased to 7.0-rc5
> * add tag (Thanks, Daniel!)
>
> drivers/mailbox/cix-mailbox.c | 2 --
> drivers/mailbox/hi3660-mailbox.c | 2 --
> drivers/mailbox/imx-mailbox.c | 2 --
> drivers/mailbox/mailbox-sti.c | 2 --
> drivers/mailbox/mailbox.c | 2 --
> drivers/mailbox/mailbox.h | 12 ------------
> drivers/mailbox/omap-mailbox.c | 2 --
> drivers/mailbox/pcc.c | 2 --
> drivers/mailbox/tegra-hsp.c | 2 --
> include/linux/mailbox_controller.h | 5 +++++
> 10 files changed, 5 insertions(+), 28 deletions(-)
> delete mode 100644 drivers/mailbox/mailbox.h
>
> diff --git a/drivers/mailbox/cix-mailbox.c b/drivers/mailbox/cix-mailbox.c
> index 443620e8ae37..864f98f21fc3 100644
> --- a/drivers/mailbox/cix-mailbox.c
> +++ b/drivers/mailbox/cix-mailbox.c
> @@ -12,8 +12,6 @@
> #include <linux/module.h>
> #include <linux/platform_device.h>
>
> -#include "mailbox.h"
> -
> /*
> * The maximum transmission size is 32 words or 128 bytes.
> */
> diff --git a/drivers/mailbox/hi3660-mailbox.c b/drivers/mailbox/hi3660-mailbox.c
> index 17c29e960fbf..9b727a2b54a5 100644
> --- a/drivers/mailbox/hi3660-mailbox.c
> +++ b/drivers/mailbox/hi3660-mailbox.c
> @@ -15,8 +15,6 @@
> #include <linux/platform_device.h>
> #include <linux/slab.h>
>
> -#include "mailbox.h"
> -
> #define MBOX_CHAN_MAX 32
>
> #define MBOX_RX 0x0
> diff --git a/drivers/mailbox/imx-mailbox.c b/drivers/mailbox/imx-mailbox.c
> index 003f9236c35e..22331b579489 100644
> --- a/drivers/mailbox/imx-mailbox.c
> +++ b/drivers/mailbox/imx-mailbox.c
> @@ -23,8 +23,6 @@
> #include <linux/slab.h>
> #include <linux/workqueue.h>
>
> -#include "mailbox.h"
> -
> #define IMX_MU_CHANS 24
> /* TX0/RX0/RXDB[0-3] */
> #define IMX_MU_SCU_CHANS 6
> diff --git a/drivers/mailbox/mailbox-sti.c b/drivers/mailbox/mailbox-sti.c
> index b4b5bdd503cf..b6c9ecbbc8ec 100644
> --- a/drivers/mailbox/mailbox-sti.c
> +++ b/drivers/mailbox/mailbox-sti.c
> @@ -21,8 +21,6 @@
> #include <linux/property.h>
> #include <linux/slab.h>
>
> -#include "mailbox.h"
> -
> #define STI_MBOX_INST_MAX 4 /* RAM saving: Max supported instances */
> #define STI_MBOX_CHAN_MAX 20 /* RAM saving: Max supported channels */
>
> diff --git a/drivers/mailbox/mailbox.c b/drivers/mailbox/mailbox.c
> index e63b2292ee7a..9d41a1ab9018 100644
> --- a/drivers/mailbox/mailbox.c
> +++ b/drivers/mailbox/mailbox.c
> @@ -18,8 +18,6 @@
> #include <linux/property.h>
> #include <linux/spinlock.h>
>
> -#include "mailbox.h"
> -
> static LIST_HEAD(mbox_cons);
> static DEFINE_MUTEX(con_mutex);
>
> diff --git a/drivers/mailbox/mailbox.h b/drivers/mailbox/mailbox.h
> deleted file mode 100644
> index e1ec4efab693..000000000000
> --- a/drivers/mailbox/mailbox.h
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -/* SPDX-License-Identifier: GPL-2.0-only */
> -
> -#ifndef __MAILBOX_H
> -#define __MAILBOX_H
> -
> -#include <linux/bits.h>
> -
> -#define TXDONE_BY_IRQ BIT(0) /* controller has remote RTR irq */
> -#define TXDONE_BY_POLL BIT(1) /* controller can read status of last TX */
> -#define TXDONE_BY_ACK BIT(2) /* S/W ACK received by Client ticks the TX */
> -
> -#endif /* __MAILBOX_H */
> diff --git a/drivers/mailbox/omap-mailbox.c b/drivers/mailbox/omap-mailbox.c
> index d9f100c18895..5772c6b9886a 100644
> --- a/drivers/mailbox/omap-mailbox.c
> +++ b/drivers/mailbox/omap-mailbox.c
> @@ -22,8 +22,6 @@
> #include <linux/pm_runtime.h>
> #include <linux/mailbox_controller.h>
>
> -#include "mailbox.h"
> -
> #define MAILBOX_REVISION 0x000
> #define MAILBOX_MESSAGE(m) (0x040 + 4 * (m))
> #define MAILBOX_FIFOSTATUS(m) (0x080 + 4 * (m))
> diff --git a/drivers/mailbox/pcc.c b/drivers/mailbox/pcc.c
> index 22e70af1ae5d..636879ae1db7 100644
> --- a/drivers/mailbox/pcc.c
> +++ b/drivers/mailbox/pcc.c
> @@ -59,8 +59,6 @@
> #include <linux/io-64-nonatomic-lo-hi.h>
> #include <acpi/pcc.h>
>
> -#include "mailbox.h"
> -
> #define MBOX_IRQ_NAME "pcc-mbox"
>
> /**
> diff --git a/drivers/mailbox/tegra-hsp.c b/drivers/mailbox/tegra-hsp.c
> index ed9a0bb2bcd8..2231050bb5a9 100644
> --- a/drivers/mailbox/tegra-hsp.c
> +++ b/drivers/mailbox/tegra-hsp.c
> @@ -16,8 +16,6 @@
>
> #include <dt-bindings/mailbox/tegra186-hsp.h>
>
> -#include "mailbox.h"
> -
> #define HSP_INT_IE(x) (0x100 + ((x) * 4))
> #define HSP_INT_IV 0x300
> #define HSP_INT_IR 0x304
> diff --git a/include/linux/mailbox_controller.h b/include/linux/mailbox_controller.h
> index 80a427c7ca29..16fef421c30c 100644
> --- a/include/linux/mailbox_controller.h
> +++ b/include/linux/mailbox_controller.h
> @@ -3,6 +3,7 @@
> #ifndef __MAILBOX_CONTROLLER_H
> #define __MAILBOX_CONTROLLER_H
>
> +#include <linux/bits.h>
> #include <linux/completion.h>
> #include <linux/device.h>
> #include <linux/hrtimer.h>
> @@ -11,6 +12,10 @@
>
> struct mbox_chan;
>
> +#define TXDONE_BY_IRQ BIT(0) /* controller has remote RTR irq */
> +#define TXDONE_BY_POLL BIT(1) /* controller can read status of last TX */
> +#define TXDONE_BY_ACK BIT(2) /* S/W ACK received by Client ticks the TX */
> +
> /**
> * struct mbox_chan_ops - methods to control mailbox channels
> * @send_data: The API asks the MBOX controller driver, in atomic
> --
> 2.51.0
>
Applied to mailbox/for-next
Thanks
Jassi
More information about the linux-arm-kernel
mailing list