[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <547EDD91.2000000@atmel.com>
Date: Wed, 3 Dec 2014 10:53:21 +0100
From: Nicolas Ferre <nicolas.ferre@...el.com>
To: Arnd Bergmann <arnd@...db.de>,
Boris BREZILLON <boris.brezillon@...e-electrons.com>,
Alexandre Belloni <alexandre.belloni@...e-electrons.com>,
Linus Walleij <linus.walleij@...aro.org>
CC: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>,
"Ludovic Desroches" <ludovic.desroches@...el.com>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-kernel@...r.kernel.org>, "Olof Johansson" <olof@...om.net>,
<arm@...nel.org>
Subject: Re: [PATCH 7/8] pinctrl: at91: add registers definitions directly
into driver
Le 02/12/2014 18:27, Nicolas Ferre a écrit :
> As the old gpio.c driver is gone, we can merge the registers definitions with
> the pinctrl driver.
> This also allows us to empty the AT91 include/mach directory.
>
> Reported-by: Arnd Bergmann <arnd@...db.de>
> Signed-off-by: Nicolas Ferre <nicolas.ferre@...el.com>
> CC: Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>
Please disregard this patch as a similar one is already queued in
pinctrl git tree (pinctrl: at91: use own header) by Alexandre.
The file will be removed after both arm-soc and pinctrl git tree are
pulled by Linus (T.).
So the -rc1 time-frame seems the good one for this kind of dead code
removal action...
Bye,
> ---
> arch/arm/mach-at91/include/mach/at91_pio.h | 80 ------------------------------
> drivers/pinctrl/pinctrl-at91.c | 65 ++++++++++++++++++++++--
> 2 files changed, 62 insertions(+), 83 deletions(-)
> delete mode 100644 arch/arm/mach-at91/include/mach/at91_pio.h
>
> diff --git a/arch/arm/mach-at91/include/mach/at91_pio.h b/arch/arm/mach-at91/include/mach/at91_pio.h
> deleted file mode 100644
> index 7b7366253ceb..000000000000
> --- a/arch/arm/mach-at91/include/mach/at91_pio.h
> +++ /dev/null
> @@ -1,80 +0,0 @@
> -/*
> - * arch/arm/mach-at91/include/mach/at91_pio.h
> - *
> - * Copyright (C) 2005 Ivan Kokshaysky
> - * Copyright (C) SAN People
> - *
> - * Parallel I/O Controller (PIO) - System peripherals registers.
> - * Based on AT91RM9200 datasheet revision E.
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - */
> -
> -#ifndef AT91_PIO_H
> -#define AT91_PIO_H
> -
> -#define PIO_PER 0x00 /* Enable Register */
> -#define PIO_PDR 0x04 /* Disable Register */
> -#define PIO_PSR 0x08 /* Status Register */
> -#define PIO_OER 0x10 /* Output Enable Register */
> -#define PIO_ODR 0x14 /* Output Disable Register */
> -#define PIO_OSR 0x18 /* Output Status Register */
> -#define PIO_IFER 0x20 /* Glitch Input Filter Enable */
> -#define PIO_IFDR 0x24 /* Glitch Input Filter Disable */
> -#define PIO_IFSR 0x28 /* Glitch Input Filter Status */
> -#define PIO_SODR 0x30 /* Set Output Data Register */
> -#define PIO_CODR 0x34 /* Clear Output Data Register */
> -#define PIO_ODSR 0x38 /* Output Data Status Register */
> -#define PIO_PDSR 0x3c /* Pin Data Status Register */
> -#define PIO_IER 0x40 /* Interrupt Enable Register */
> -#define PIO_IDR 0x44 /* Interrupt Disable Register */
> -#define PIO_IMR 0x48 /* Interrupt Mask Register */
> -#define PIO_ISR 0x4c /* Interrupt Status Register */
> -#define PIO_MDER 0x50 /* Multi-driver Enable Register */
> -#define PIO_MDDR 0x54 /* Multi-driver Disable Register */
> -#define PIO_MDSR 0x58 /* Multi-driver Status Register */
> -#define PIO_PUDR 0x60 /* Pull-up Disable Register */
> -#define PIO_PUER 0x64 /* Pull-up Enable Register */
> -#define PIO_PUSR 0x68 /* Pull-up Status Register */
> -#define PIO_ASR 0x70 /* Peripheral A Select Register */
> -#define PIO_ABCDSR1 0x70 /* Peripheral ABCD Select Register 1 [some sam9 only] */
> -#define PIO_BSR 0x74 /* Peripheral B Select Register */
> -#define PIO_ABCDSR2 0x74 /* Peripheral ABCD Select Register 2 [some sam9 only] */
> -#define PIO_ABSR 0x78 /* AB Status Register */
> -#define PIO_IFSCDR 0x80 /* Input Filter Slow Clock Disable Register */
> -#define PIO_IFSCER 0x84 /* Input Filter Slow Clock Enable Register */
> -#define PIO_IFSCSR 0x88 /* Input Filter Slow Clock Status Register */
> -#define PIO_SCDR 0x8c /* Slow Clock Divider Debouncing Register */
> -#define PIO_SCDR_DIV (0x3fff << 0) /* Slow Clock Divider Mask */
> -#define PIO_PPDDR 0x90 /* Pad Pull-down Disable Register */
> -#define PIO_PPDER 0x94 /* Pad Pull-down Enable Register */
> -#define PIO_PPDSR 0x98 /* Pad Pull-down Status Register */
> -#define PIO_OWER 0xa0 /* Output Write Enable Register */
> -#define PIO_OWDR 0xa4 /* Output Write Disable Register */
> -#define PIO_OWSR 0xa8 /* Output Write Status Register */
> -#define PIO_AIMER 0xb0 /* Additional Interrupt Modes Enable Register */
> -#define PIO_AIMDR 0xb4 /* Additional Interrupt Modes Disable Register */
> -#define PIO_AIMMR 0xb8 /* Additional Interrupt Modes Mask Register */
> -#define PIO_ESR 0xc0 /* Edge Select Register */
> -#define PIO_LSR 0xc4 /* Level Select Register */
> -#define PIO_ELSR 0xc8 /* Edge/Level Status Register */
> -#define PIO_FELLSR 0xd0 /* Falling Edge/Low Level Select Register */
> -#define PIO_REHLSR 0xd4 /* Rising Edge/ High Level Select Register */
> -#define PIO_FRLHSR 0xd8 /* Fall/Rise - Low/High Status Register */
> -#define PIO_SCHMITT 0x100 /* Schmitt Trigger Register */
> -
> -#define ABCDSR_PERIPH_A 0x0
> -#define ABCDSR_PERIPH_B 0x1
> -#define ABCDSR_PERIPH_C 0x2
> -#define ABCDSR_PERIPH_D 0x3
> -
> -#define SAMA5D3_PIO_DRIVER1 0x118 /*PIO Driver 1 register offset*/
> -#define SAMA5D3_PIO_DRIVER2 0x11C /*PIO Driver 2 register offset*/
> -
> -#define AT91SAM9X5_PIO_DRIVER1 0x114 /*PIO Driver 1 register offset*/
> -#define AT91SAM9X5_PIO_DRIVER2 0x118 /*PIO Driver 2 register offset*/
> -
> -#endif
> diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
> index 354a81d40925..47d755861b5a 100644
> --- a/drivers/pinctrl/pinctrl-at91.c
> +++ b/drivers/pinctrl/pinctrl-at91.c
> @@ -25,11 +25,70 @@
> /* Since we request GPIOs from ourself */
> #include <linux/pinctrl/consumer.h>
>
> -#include <mach/hardware.h>
> -#include <mach/at91_pio.h>
> -
> #include "core.h"
>
> +/* PIO register definitions */
> +#define PIO_PER 0x00 /* Enable Register */
> +#define PIO_PDR 0x04 /* Disable Register */
> +#define PIO_PSR 0x08 /* Status Register */
> +#define PIO_OER 0x10 /* Output Enable Register */
> +#define PIO_ODR 0x14 /* Output Disable Register */
> +#define PIO_OSR 0x18 /* Output Status Register */
> +#define PIO_IFER 0x20 /* Glitch Input Filter Enable */
> +#define PIO_IFDR 0x24 /* Glitch Input Filter Disable */
> +#define PIO_IFSR 0x28 /* Glitch Input Filter Status */
> +#define PIO_SODR 0x30 /* Set Output Data Register */
> +#define PIO_CODR 0x34 /* Clear Output Data Register */
> +#define PIO_ODSR 0x38 /* Output Data Status Register */
> +#define PIO_PDSR 0x3c /* Pin Data Status Register */
> +#define PIO_IER 0x40 /* Interrupt Enable Register */
> +#define PIO_IDR 0x44 /* Interrupt Disable Register */
> +#define PIO_IMR 0x48 /* Interrupt Mask Register */
> +#define PIO_ISR 0x4c /* Interrupt Status Register */
> +#define PIO_MDER 0x50 /* Multi-driver Enable Register */
> +#define PIO_MDDR 0x54 /* Multi-driver Disable Register */
> +#define PIO_MDSR 0x58 /* Multi-driver Status Register */
> +#define PIO_PUDR 0x60 /* Pull-up Disable Register */
> +#define PIO_PUER 0x64 /* Pull-up Enable Register */
> +#define PIO_PUSR 0x68 /* Pull-up Status Register */
> +#define PIO_ASR 0x70 /* Peripheral A Select Register */
> +#define PIO_ABCDSR1 0x70 /* Peripheral ABCD Select Register 1 [some SoCs only] */
> +#define PIO_BSR 0x74 /* Peripheral B Select Register */
> +#define PIO_ABCDSR2 0x74 /* Peripheral ABCD Select Register 2 [some SoCs only] */
> +#define PIO_ABSR 0x78 /* AB Status Register */
> +#define PIO_IFSCDR 0x80 /* Input Filter Slow Clock Disable Register */
> +#define PIO_IFSCER 0x84 /* Input Filter Slow Clock Enable Register */
> +#define PIO_IFSCSR 0x88 /* Input Filter Slow Clock Status Register */
> +#define PIO_SCDR 0x8c /* Slow Clock Divider Debouncing Register */
> +#define PIO_SCDR_DIV (0x3fff << 0) /* Slow Clock Divider Mask */
> +#define PIO_PPDDR 0x90 /* Pad Pull-down Disable Register */
> +#define PIO_PPDER 0x94 /* Pad Pull-down Enable Register */
> +#define PIO_PPDSR 0x98 /* Pad Pull-down Status Register */
> +#define PIO_OWER 0xa0 /* Output Write Enable Register */
> +#define PIO_OWDR 0xa4 /* Output Write Disable Register */
> +#define PIO_OWSR 0xa8 /* Output Write Status Register */
> +#define PIO_AIMER 0xb0 /* Additional Interrupt Modes Enable Register */
> +#define PIO_AIMDR 0xb4 /* Additional Interrupt Modes Disable Register */
> +#define PIO_AIMMR 0xb8 /* Additional Interrupt Modes Mask Register */
> +#define PIO_ESR 0xc0 /* Edge Select Register */
> +#define PIO_LSR 0xc4 /* Level Select Register */
> +#define PIO_ELSR 0xc8 /* Edge/Level Status Register */
> +#define PIO_FELLSR 0xd0 /* Falling Edge/Low Level Select Register */
> +#define PIO_REHLSR 0xd4 /* Rising Edge/ High Level Select Register */
> +#define PIO_FRLHSR 0xd8 /* Fall/Rise - Low/High Status Register */
> +#define PIO_SCHMITT 0x100 /* Schmitt Trigger Register */
> +
> +#define ABCDSR_PERIPH_A 0x0
> +#define ABCDSR_PERIPH_B 0x1
> +#define ABCDSR_PERIPH_C 0x2
> +#define ABCDSR_PERIPH_D 0x3
> +
> +#define SAMA5D3_PIO_DRIVER1 0x118 /*PIO Driver 1 register offset*/
> +#define SAMA5D3_PIO_DRIVER2 0x11C /*PIO Driver 2 register offset*/
> +
> +#define AT91SAM9X5_PIO_DRIVER1 0x114 /*PIO Driver 1 register offset*/
> +#define AT91SAM9X5_PIO_DRIVER2 0x118 /*PIO Driver 2 register offset*/
> +
> #define MAX_GPIO_BANKS 5
> #define MAX_NB_GPIO_PER_BANK 32
>
>
--
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists