> On 09.06.2020, at 15:37, Heiko Stuebner <he...@sntech.de> wrote:
>
> From: Heiko Stuebner <heiko.stueb...@theobroma-systems.com>
>
> The Linux kernel does set the clock delays to
> - 0.2 ns (their default, and lowest, hardware value) if delays should
> not be enabled
> - 2.0 ns (which causes the data to be sampled at exactly half way between
> clock transitions at 1000 Mbps) if delays should be enabled
> depending on the interface mode
>
> See
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/phy/mscc/mscc_main.c#n523
>
> So instead of using arbitrary delay values like now, mimic this behaviour.
>
> The behaviour is the same for all of vsc8530/8531/8540/8541 so move that
> to a shared function while at it.
>
> Signed-off-by: Heiko Stuebner <heiko.stueb...@theobroma-systems.com>
Reviewed-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com>