Re: [PATCH] Documentation/bindings: net: fix sfp mod-def0 signal

[Andrew Lunn <andrew@lunn.ch>]

On Mon, Jun 06, 2022 at 11:18:52AM +0200, Angelo Dureghello wrote:
> Checked both by driver code and functionally, module plug works
> when gpio is set as GPIO_ACTIVE_LOW.
> 
> Signed-off-by: Angelo Dureghello <angelo.dureghello@timesys.com>
> ---
>  Documentation/devicetree/bindings/net/sff,sfp.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/sff,sfp.txt b/Documentation/devicetree/bindings/net/sff,sfp.txt
> index 832139919f20..101a025a0c0f 100644
> --- a/Documentation/devicetree/bindings/net/sff,sfp.txt
> +++ b/Documentation/devicetree/bindings/net/sff,sfp.txt
> @@ -13,7 +13,7 @@ Required properties:
>  Optional Properties:
>  
>  - mod-def0-gpios : GPIO phandle and a specifier of the MOD-DEF0 (AKA Mod_ABS)
> -  module presence input gpio signal, active (module absent) high. Must
> +  module presence input gpio signal, active (module absent) low. Must
>    not be present for SFF modules

INF-8074.pdf, "Table 1, Pin Function Definitions", says

	Note 3, Grounded in Module.

and Note 3 says:

	3) Mod-Def 0,1,2. These are the module definition pins. They
	should be pulled up with a 4.7K – 10KΩ resistor on the host
	board. The pull-up voltage shall be VccT or VccR (see Section
	IV for further details).  Mod-Def 0 is grounded by the module
	to indicate that the module is present

So when the module is absent, the pullup on the board will mean the
GPIO is high.

NACK.

Please check you don't have an inverter in the path for your board.

	Andrew