HomeFreeBSD

uart: Honour clock-frequency in FDT for UART_FDT_CLASS if present

Description

uart: Honour clock-frequency in FDT for UART_FDT_CLASS if present

The StarFive VisionFive 2 has a Synopsys DesignWare ABP UART, whose
driver uses UART_FDT_CLASS rather than UART_FDT_CLASS_AND_DEVICE as it
has its own separate newbus driver. This UART is driven by a 24MHz clock
as specified in the FDT, but we don't currently look at the property
here, instead passing down 0 and letting the default value be used in
the 8250 driver (~1.8MHz). As a result the divisor is misconfigured for
the current baud rate for the entire kernel boot process. Once the
newbus driver attaches the correct frequency is saved in the softc, but
that does not take effect until the next time ns8250_param is called and
the divisor is recalculated, namely when userspace runs and /dev/console
is opened (note that ns8250_init does not get called when the newbus
device corresponding to the current console attaches).

Fix this issue by attemmpting to get the current clock frequency as for
the UART_FDT_CLASS_AND_DEVICE_CASE, but falling back to 0 rather than
failing on error.

Reviewed by: imp, mhorne
Differential Revision: https://reviews.freebsd.org/D45159

Details

Provenance
jrtc27Authored on May 15 2024, 3:53 PM
Reviewer
imp
Differential Revision
D45159: uart: Honour clock-frequency in FDT for UART_FDT_CLASS if present
Parents
rGea3751fb53b2: freebsd-tips: Tweak fortune to find font providing Unicode character
Branches
Unknown
Tags
Unknown