diff mbox

[2/2] ARM: dts: dra72-evm-revc: enable irqs for dp83867 eth phys

Message ID 20170106205543.4748-1-grygorii.strashko@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Grygorii Strashko Jan. 6, 2017, 8:55 p.m. UTC
TI DRA72-EVM Rev C has two DP83867 ethernet phys which support IRQ
generation in case of phy/link status changes. The INT/PWDN lines from both
DP83867 phys are wired to DRA7 gpio6.16, so reflect the same in DT.

Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 arch/arm/boot/dts/dra72-evm-revc.dts | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Tony Lindgren Jan. 6, 2017, 9:54 p.m. UTC | #1
* Grygorii Strashko <grygorii.strashko@ti.com> [170106 12:56]:
> TI DRA72-EVM Rev C has two DP83867 ethernet phys which support IRQ
> generation in case of phy/link status changes. The INT/PWDN lines from both
> DP83867 phys are wired to DRA7 gpio6.16, so reflect the same in DT.

Hmm not seeing the patch 1/2 here.. Can this one be queued separately?
Is it for v4.11 or a fix?

Regards,

Tony

> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
> ---
>  arch/arm/boot/dts/dra72-evm-revc.dts | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/dra72-evm-revc.dts b/arch/arm/boot/dts/dra72-evm-revc.dts
> index c3d939c..3ecac56 100644
> --- a/arch/arm/boot/dts/dra72-evm-revc.dts
> +++ b/arch/arm/boot/dts/dra72-evm-revc.dts
> @@ -68,6 +68,8 @@
>  		ti,tx-internal-delay = <DP83867_RGMIIDCTL_250_PS>;
>  		ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
>  		ti,min-output-impedance;
> +		interrupt-parent = <&gpio6>;
> +		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
>  	};
>  
>  	dp83867_1: ethernet-phy@3 {
> @@ -75,6 +77,8 @@
>  		ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_25_NS>;
>  		ti,tx-internal-delay = <DP83867_RGMIIDCTL_250_PS>;
>  		ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
> -		ti,min-output-imepdance;
> +		ti,min-output-impedance;
> +		interrupt-parent = <&gpio6>;
> +		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
>  	};
>  };
> -- 
> 2.10.1.dirty
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Grygorii Strashko Jan. 9, 2017, 7:25 p.m. UTC | #2
On 01/06/2017 03:54 PM, Tony Lindgren wrote:
> * Grygorii Strashko <grygorii.strashko@ti.com> [170106 12:56]:
>> TI DRA72-EVM Rev C has two DP83867 ethernet phys which support IRQ
>> generation in case of phy/link status changes. The INT/PWDN lines from both
>> DP83867 phys are wired to DRA7 gpio6.16, so reflect the same in DT.
>
> Hmm not seeing the patch 1/2 here.. Can this one be queued separately?
> Is it for v4.11 or a fix?

This is for next (v4.11) and there is just a mistake in subj, Sry.

>> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
>> ---
>>  arch/arm/boot/dts/dra72-evm-revc.dts | 6 +++++-
>>  1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/dra72-evm-revc.dts b/arch/arm/boot/dts/dra72-evm-revc.dts
>> index c3d939c..3ecac56 100644
>> --- a/arch/arm/boot/dts/dra72-evm-revc.dts
>> +++ b/arch/arm/boot/dts/dra72-evm-revc.dts
>> @@ -68,6 +68,8 @@
>>  		ti,tx-internal-delay = <DP83867_RGMIIDCTL_250_PS>;
>>  		ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
>>  		ti,min-output-impedance;
>> +		interrupt-parent = <&gpio6>;
>> +		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
>>  	};
>>
>>  	dp83867_1: ethernet-phy@3 {
>> @@ -75,6 +77,8 @@
>>  		ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_25_NS>;
>>  		ti,tx-internal-delay = <DP83867_RGMIIDCTL_250_PS>;
>>  		ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
>> -		ti,min-output-imepdance;
>> +		ti,min-output-impedance;
>> +		interrupt-parent = <&gpio6>;
>> +		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
>>  	};
>>  };
>> --
>> 2.10.1.dirty
>>
Tony Lindgren Jan. 12, 2017, 10:27 p.m. UTC | #3
* Grygorii Strashko <grygorii.strashko@ti.com> [170109 11:26]:
> 
> 
> On 01/06/2017 03:54 PM, Tony Lindgren wrote:
> > * Grygorii Strashko <grygorii.strashko@ti.com> [170106 12:56]:
> > > TI DRA72-EVM Rev C has two DP83867 ethernet phys which support IRQ
> > > generation in case of phy/link status changes. The INT/PWDN lines from both
> > > DP83867 phys are wired to DRA7 gpio6.16, so reflect the same in DT.
> > 
> > Hmm not seeing the patch 1/2 here.. Can this one be queued separately?
> > Is it for v4.11 or a fix?
> 
> This is for next (v4.11) and there is just a mistake in subj, Sry.

OK applying into omap-for-v4.11/dt thanks.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/dra72-evm-revc.dts b/arch/arm/boot/dts/dra72-evm-revc.dts
index c3d939c..3ecac56 100644
--- a/arch/arm/boot/dts/dra72-evm-revc.dts
+++ b/arch/arm/boot/dts/dra72-evm-revc.dts
@@ -68,6 +68,8 @@ 
 		ti,tx-internal-delay = <DP83867_RGMIIDCTL_250_PS>;
 		ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
 		ti,min-output-impedance;
+		interrupt-parent = <&gpio6>;
+		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
 	};
 
 	dp83867_1: ethernet-phy@3 {
@@ -75,6 +77,8 @@ 
 		ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_25_NS>;
 		ti,tx-internal-delay = <DP83867_RGMIIDCTL_250_PS>;
 		ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
-		ti,min-output-imepdance;
+		ti,min-output-impedance;
+		interrupt-parent = <&gpio6>;
+		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
 	};
 };