[PATCH] mailbox: exynos: Drop unused register definitions

Jassi Brar jassisinghbrar at gmail.com
Mon May 18 12:00:49 PDT 2026


On Thu, Apr 30, 2026 at 6:13 AM Tudor Ambarus <tudor.ambarus at linaro.org> wrote:
>
> Leaving these dead definitions in place hides which registers are
> actually being used by the hardware, making the driver harder to read
> and maintain. Remove them to clean up the file.
>
> Signed-off-by: Tudor Ambarus <tudor.ambarus at linaro.org>
> ---
>  drivers/mailbox/exynos-mailbox.c | 7 -------
>  1 file changed, 7 deletions(-)
>
> diff --git a/drivers/mailbox/exynos-mailbox.c b/drivers/mailbox/exynos-mailbox.c
> index 5f2d3b81c1db..953e715a89b1 100644
> --- a/drivers/mailbox/exynos-mailbox.c
> +++ b/drivers/mailbox/exynos-mailbox.c
> @@ -16,15 +16,8 @@
>  #include <linux/platform_device.h>
>  #include <linux/slab.h>
>
> -#define EXYNOS_MBOX_MCUCTRL            0x0     /* Mailbox Control Register */
> -#define EXYNOS_MBOX_INTCR0             0x24    /* Interrupt Clear Register 0 */
>  #define EXYNOS_MBOX_INTMR0             0x28    /* Interrupt Mask Register 0 */
> -#define EXYNOS_MBOX_INTSR0             0x2c    /* Interrupt Status Register 0 */
> -#define EXYNOS_MBOX_INTMSR0            0x30    /* Interrupt Mask Status Register 0 */
>  #define EXYNOS_MBOX_INTGR1             0x40    /* Interrupt Generation Register 1 */
> -#define EXYNOS_MBOX_INTMR1             0x48    /* Interrupt Mask Register 1 */
> -#define EXYNOS_MBOX_INTSR1             0x4c    /* Interrupt Status Register 1 */
> -#define EXYNOS_MBOX_INTMSR1            0x50    /* Interrupt Mask Status Register 1 */
>
>  #define EXYNOS_MBOX_INTMR0_MASK                GENMASK(15, 0)
>  #define EXYNOS_MBOX_INTGR1_MASK                GENMASK(15, 0)
>
> ---
Applied to mailbox/for-next
Thanks
Jassi



More information about the linux-arm-kernel mailing list