Re: [PATCH] dt-bindings: tegra186-gpio: Add Tegra186 specific prefix

[Jon Hunter <jonathanh@xxxxxxxxxx>]

On 23/11/2018 12:43, Thierry Reding wrote:
> From: Thierry Reding <treding@xxxxxxxxxx>
> 
> Subsequent generations of Tegra, such as Tegra194, contain a completely
> different set of GPIOs. In order to clarify that the Tegra186 defines
> are indeed specific to Tegra186, change the prefix from TEGRA_ to
> TEGRA186_.
> 
> Note that for now we need to keep the old definitions in place to avoid
> breaking compilation in file that use this header. Once all users have
> been converted to use the new defines, the old ones can be removed.
> 
> Also note that this is only a naming change and doesn't affect device
> tree ABI.
> 
> Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> ---
>  include/dt-bindings/gpio/tegra186-gpio.h | 41 ++++++++++++++++++++++++
>  1 file changed, 41 insertions(+)
> 
> diff --git a/include/dt-bindings/gpio/tegra186-gpio.h b/include/dt-bindings/gpio/tegra186-gpio.h
> index 463ad398fe3e..cabc5712e745 100644
> --- a/include/dt-bindings/gpio/tegra186-gpio.h
> +++ b/include/dt-bindings/gpio/tegra186-gpio.h
> @@ -14,6 +14,34 @@
>  #include <dt-bindings/gpio/gpio.h>
>  
>  /* GPIOs implemented by main GPIO controller */
> +#define TEGRA186_MAIN_GPIO_PORT_A 0
> +#define TEGRA186_MAIN_GPIO_PORT_B 1
> +#define TEGRA186_MAIN_GPIO_PORT_C 2
> +#define TEGRA186_MAIN_GPIO_PORT_D 3
> +#define TEGRA186_MAIN_GPIO_PORT_E 4
> +#define TEGRA186_MAIN_GPIO_PORT_F 5
> +#define TEGRA186_MAIN_GPIO_PORT_G 6
> +#define TEGRA186_MAIN_GPIO_PORT_H 7
> +#define TEGRA186_MAIN_GPIO_PORT_I 8
> +#define TEGRA186_MAIN_GPIO_PORT_J 9
> +#define TEGRA186_MAIN_GPIO_PORT_K 10
> +#define TEGRA186_MAIN_GPIO_PORT_L 11
> +#define TEGRA186_MAIN_GPIO_PORT_M 12
> +#define TEGRA186_MAIN_GPIO_PORT_N 13
> +#define TEGRA186_MAIN_GPIO_PORT_O 14
> +#define TEGRA186_MAIN_GPIO_PORT_P 15
> +#define TEGRA186_MAIN_GPIO_PORT_Q 16
> +#define TEGRA186_MAIN_GPIO_PORT_R 17
> +#define TEGRA186_MAIN_GPIO_PORT_T 18
> +#define TEGRA186_MAIN_GPIO_PORT_X 19
> +#define TEGRA186_MAIN_GPIO_PORT_Y 20
> +#define TEGRA186_MAIN_GPIO_PORT_BB 21
> +#define TEGRA186_MAIN_GPIO_PORT_CC 22
> +
> +#define TEGRA186_MAIN_GPIO(port, offset) \
> +	((TEGRA186_MAIN_GPIO_PORT_##port * 8) + offset)
> +
> +/* need to keep these for backwards-compatibility */
>  #define TEGRA_MAIN_GPIO_PORT_A 0
>  #define TEGRA_MAIN_GPIO_PORT_B 1
>  #define TEGRA_MAIN_GPIO_PORT_C 2
> @@ -42,6 +70,19 @@
>  	((TEGRA_MAIN_GPIO_PORT_##port * 8) + offset)
>  
>  /* GPIOs implemented by AON GPIO controller */
> +#define TEGRA186_AON_GPIO_PORT_S 0
> +#define TEGRA186_AON_GPIO_PORT_U 1
> +#define TEGRA186_AON_GPIO_PORT_V 2
> +#define TEGRA186_AON_GPIO_PORT_W 3
> +#define TEGRA186_AON_GPIO_PORT_Z 4
> +#define TEGRA186_AON_GPIO_PORT_AA 5
> +#define TEGRA186_AON_GPIO_PORT_EE 6
> +#define TEGRA186_AON_GPIO_PORT_FF 7
> +
> +#define TEGRA186_AON_GPIO(port, offset) \
> +	((TEGRA186_AON_GPIO_PORT_##port * 8) + offset)
> +
> +/* need to keep these for backwards-compatibility */
>  #define TEGRA_AON_GPIO_PORT_S 0
>  #define TEGRA_AON_GPIO_PORT_U 1
>  #define TEGRA_AON_GPIO_PORT_V 2

I guess this will make it more consistent with what we have for
Tegra194, so looks fine to me ...

Acked-by: Jon Hunter <jonathanh@xxxxxxxxxx>

Cheers
Jon

-- 
nvpublic