Hi, On Fri, Jun 24, 2022 at 06:52:26PM +0200, Andrew Lunn wrote: > > > So it seems to me you are changing the documented default. You cannot > > > do that, this is ABI. > > > > Right. I suppose we either need a disable value or an extra property. I > > can add support for supplying (-1) from DT. Does that sounds ok to > > everyone? > > I'm missing the big picture. > > Does the hardware you are adding not support delays? If so, rather > than using the defaults, don't do anything. And if a value is > supplied, -EINVAL? The Rockchip hardware supports enabling/disabling delays and configuring a delay value of 0x00-0x7f. For the RK3588 evaluation board the RX delay should be disabled. -- Sebastian
Attachment:
signature.asc
Description: PGP signature
