RE: [PATCH net v2 1/1] net: fec: fix the PTP periodic output sysfs interface

[Wei Fang]

> When the PPS channel configuration was implemented, the number of
> supported periodic outputs (`n_per_out`) was left at 1.
> 
> This prohibits using channels 1..3 from the sysfs interface, since
> period_store() rejects channel numbers greater than `n_per_out`.
> 
> Fix by increasing `n_per_out` to the number of channels supported
> by the hardware.
> 
> Fixes: 566c2d83887f ("net: fec: make PPS channel configurable")
> Signed-off-by: Buday Csaba <buday.csaba@xxxxxxxxx>
> ---
>  drivers/net/ethernet/freescale/fec_ptp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/freescale/fec_ptp.c
> b/drivers/net/ethernet/freescale/fec_ptp.c
> index 4b7bad9a485d..1a7aa280e7f6 100644
> --- a/drivers/net/ethernet/freescale/fec_ptp.c
> +++ b/drivers/net/ethernet/freescale/fec_ptp.c
> @@ -756,7 +756,7 @@ void fec_ptp_init(struct platform_device *pdev, int
> irq_idx)
>  	fep->ptp_caps.max_adj = 250000000;
>  	fep->ptp_caps.n_alarm = 0;
>  	fep->ptp_caps.n_ext_ts = 0;
> -	fep->ptp_caps.n_per_out = 1;
> +	fep->ptp_caps.n_per_out = 4;
>  	fep->ptp_caps.n_pins = 0;
>  	fep->ptp_caps.pps = 1;
>  	fep->ptp_caps.adjfine = fec_ptp_adjfine;
> 
> base-commit: 8a63baadf08453f66eb582fdb6dd234f72024723
> --
> 2.39.5
> 

Reviewed-by: Wei Fang <wei.fang@xxxxxxx>