Re: [PATCH v2 1/2] Documentation: DT: bindings: Add Broadcom STB PCIe bindings

From: Arnd Bergmann
Date: Mon May 09 2016 - 09:12:45 EST


On Thursday 05 May 2016 14:46:22 Florian Fainelli wrote:
>
> >
> >> +- brcm,gen: integer that indicates desired forced generation of link: 1 => 2.5
> >> + Gbps, 2 => 5.0 Gbps, 3 => 8.0 Gbps. Will override the auto-negotation if
> >> + specified.
> >
> > to repeat my earlier comment from v1:
> >
> > Shouldn't the link generation be probed automatically?
>
> It is, but if the property is present, we can foce the link negotiation,
> We have had cases where this was desired and helpful.
>

Right, I missed that this is listed as 'optional', so that seems
perfectly fine.

Thanks,

Arnd