[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