diff mbox

[v2,2/3] ARM: at91/dt: at91: use new pinctrl id for sama5

Message ID 1408849926-9603-2-git-send-email-mark.roszko@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mark Roszko Aug. 24, 2014, 3:12 a.m. UTC
This switches the SAMA5D3 to use the new atmel,sama5d3-pinctrl id that was
added with the drive strength options patch.

Signed-off-by: Marek Roszko <mark.roszko@gmail.com>
---
 arch/arm/boot/dts/sama5d3.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Linus Walleij Sept. 5, 2014, 7:34 a.m. UTC | #1
On Sun, Aug 24, 2014 at 5:12 AM, Marek Roszko <mark.roszko@gmail.com> wrote:

> This switches the SAMA5D3 to use the new atmel,sama5d3-pinctrl id that was
> added with the drive strength options patch.
>
> Signed-off-by: Marek Roszko <mark.roszko@gmail.com>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Funnel this through the AT91 kernel tree (Nicolas).

Yours,
Linus Walleij
Nicolas Ferre Sept. 8, 2014, 8:07 a.m. UTC | #2
On 24/08/2014 05:12, Marek Roszko :
> This switches the SAMA5D3 to use the new atmel,sama5d3-pinctrl id that was
> added with the drive strength options patch.
> 
> Signed-off-by: Marek Roszko <mark.roszko@gmail.com>
> ---
>  arch/arm/boot/dts/sama5d3.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
> index 45013b8..7db0e85 100644
> --- a/arch/arm/boot/dts/sama5d3.dtsi
> +++ b/arch/arm/boot/dts/sama5d3.dtsi
> @@ -428,7 +428,7 @@
>  			pinctrl@fffff200 {
>  				#address-cells = <1>;
>  				#size-cells = <1>;
> -				compatible = "atmel,at91sam9x5-pinctrl", "atmel,at91rm9200-pinctrl", "simple-bus";
> +				compatible = "atmel,sama5d3-pinctrl", "atmel,at91rm9200-pinctrl", "simple-bus";

I'd better keep the compatibility with at91sam9x5, like it
was before your patch: that will remove the dependency on
the pinctrl series: what about:

+				compatible = "atmel,sama5d3-pinctrl", "atmel,at91sam9x5-pinctrl", "simple-bus";

>  				ranges = <0xfffff200 0xfffff200 0xa00>;
>  				atmel,mux-mask = <
>  					/*   A          B          C  */
>
Nicolas Ferre Sept. 18, 2014, 3:32 p.m. UTC | #3
On 08/09/2014 10:07, Nicolas Ferre :
> On 24/08/2014 05:12, Marek Roszko :
>> This switches the SAMA5D3 to use the new atmel,sama5d3-pinctrl id that was
>> added with the drive strength options patch.
>>
>> Signed-off-by: Marek Roszko <mark.roszko@gmail.com>
>> ---
>>  arch/arm/boot/dts/sama5d3.dtsi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
>> index 45013b8..7db0e85 100644
>> --- a/arch/arm/boot/dts/sama5d3.dtsi
>> +++ b/arch/arm/boot/dts/sama5d3.dtsi
>> @@ -428,7 +428,7 @@
>>  			pinctrl@fffff200 {
>>  				#address-cells = <1>;
>>  				#size-cells = <1>;
>> -				compatible = "atmel,at91sam9x5-pinctrl", "atmel,at91rm9200-pinctrl", "simple-bus";
>> +				compatible = "atmel,sama5d3-pinctrl", "atmel,at91rm9200-pinctrl", "simple-bus";
> 
> I'd better keep the compatibility with at91sam9x5, like it
> was before your patch: that will remove the dependency on
> the pinctrl series: what about:
> 
> +				compatible = "atmel,sama5d3-pinctrl", "atmel,at91sam9x5-pinctrl", "simple-bus";

So, with the modification that I did above:

Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

I also stack it to my at91-3.18-dt3 that will soon reach arm-soc (there
is no dependency with the other patches of the series anymore).

Linus, are the other two of the series ready to land in Mainline for
next kernel release (3.18)? (I'm just asking because you said you would
do a "tentative").

Thanks, bye,

> 
>>  				ranges = <0xfffff200 0xfffff200 0xa00>;
>>  				atmel,mux-mask = <
>>  					/*   A          B          C  */
>>
> 
>
diff mbox

Patch

diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
index 45013b8..7db0e85 100644
--- a/arch/arm/boot/dts/sama5d3.dtsi
+++ b/arch/arm/boot/dts/sama5d3.dtsi
@@ -428,7 +428,7 @@ 
 			pinctrl@fffff200 {
 				#address-cells = <1>;
 				#size-cells = <1>;
-				compatible = "atmel,at91sam9x5-pinctrl", "atmel,at91rm9200-pinctrl", "simple-bus";
+				compatible = "atmel,sama5d3-pinctrl", "atmel,at91rm9200-pinctrl", "simple-bus";
 				ranges = <0xfffff200 0xfffff200 0xa00>;
 				atmel,mux-mask = <
 					/*   A          B          C  */