Aux uart gets wrong clock speed on RPi4

[db]

Summary

Ths uart uses clock from VPU on the VC4 GPU which is locked
to either 500 or 250Mhz depending on RPi4 or lower. Check which
model is running to set appropriate clock speed.

Test Plan

I've tested under RPi3 and RPi4. Baud rate is correct now on both.

[mikael]

sys/dev/uart/uart_dev_mu.c

519

wrong link,
https://www.raspberrypi.org/documentation/configuration/uart.md

[db]

Fixed typo in www.raspberrypi.org/documentation/configuration/uart.md

[manu]

If we start using the root compatible properties I would prefer to have it stored in a variable in sys/dev/ofw/ofw_fdt.c and both accessible by the kernel and userland (via sysctl under hw.fdt.compatible/compatibles, the first one will return the first string in the prop and the second all the strings).

sys/dev/uart/uart_dev_mu.c

548

Why not comparing the full "raspberrypi,4-model-b" string ?

[db]

sys/dev/uart/uart_dev_mu.c

519

I can't believe I did that. Fixed.

548

Sure. I don't have strong feelings on it.

Thinking about it I guess since I saw a ',' in that string I thought there might is a slight chance it might end up with an embedded space. I'd also prefer the full string. I'll change.

[db]

In D24339#535434, @manu wrote:

If we start using the root compatible properties I would prefer to have it stored in a variable in sys/dev/ofw/ofw_fdt.c and both accessible by the kernel and userland (via sysctl under hw.fdt.compatible/compatibles, the first one will return the first string in the prop and the second all the strings).

The uart is attached before cpufreq is attached. I had wrongly assumed it would be the other way around.

[manu]

In D24339#536095, @db wrote:

In D24339#535434, @manu wrote:

If we start using the root compatible properties I would prefer to have it stored in a variable in sys/dev/ofw/ofw_fdt.c and both accessible by the kernel and userland (via sysctl under hw.fdt.compatible/compatibles, the first one will return the first string in the prop and the second all the strings).

The uart is attached before cpufreq is attached. I had wrongly assumed it would be the other way around.

This doesn't change what I said.

[db]

Update to correct RPi baud rate on mu serial port

Add a command to ofw_fdt.c to retrieve Manufacturer model name
modified uart_dev_mu.c to use this new command then added
a sysctl hw.manufacturer to retrieve this model name from user land.
hw.model is already in use.

Tested on both RPi3 and RPi4

[db]

openfirm.h missing

[manu]

sys/dev/ofw/ofw_fdt.c

153

that should be in hw.fdt.model

463

What's the point of this function ?

[db]

Add model and compatible strings
add string sysctl's for both under hw.fdt
lookup model and compatible once and store

[• ian]

I don't understand the need to add new methods to ofw_if. The new methods are just extracting standard properties from a standard dtb in memory; if new functions are needed at all, why aren't they just a couple new convenience functions in ofw_subr.{c,h}?

It might be a good idea to get Nathan Whitehorn's input on this before committing.