[PATCH 0/2] serial: msm-geni: rework oversampling and fix clk API bug

Caleb Connolly caleb.connolly at linaro.org
Tue Nov 14 13:51:10 CET 2023


These patches improve GENI UART support during init by implementing the
parent property read directly rather than via a misc device, and fixing
the error path when the clock can't be found.

In my testing, the first few lines of UART output on platforms with
non-default oversampling values is often garbled, this is because the
parent misc device hasn't yet probed and so the clock rate is incorrect.

It is simpler to just access the geni-se parent device directly rather
than via a misc device, especially as we only need to read a single
register for now.

Additionally, this series makes it a hard requirement that the GENI UART
node is a child of the generic geni-se controller. This allows us to
print a useful error if DTS is incorrect rather than attempting to
continue without reading the oversampling value could result in a lot of
confusion during platform bringup.

---
Caleb Connolly (2):
      serial: msm-geni: don't rely on parent misc device
      serial: msm-geni: handle devm_clk_get() errors

 drivers/misc/Kconfig             |  7 -------
 drivers/misc/Makefile            |  1 -
 drivers/misc/qcom-geni-se.c      | 41 ----------------------------------------
 drivers/serial/Kconfig           |  2 --
 drivers/serial/serial_msm_geni.c | 40 ++++++++++++++++++++++++++-------------
 5 files changed, 27 insertions(+), 64 deletions(-)
---
base-commit: f3fc930d775ef5c1b7b74b1427491a17680e66ae

// Caleb (they/them)



More information about the U-Boot mailing list