Add support of UART_DEV_DBGPORT to uart_bus_fdt.c

[dsl]

Summary

NOTE: This patch is a re-worked copy of an abandoned one attached to the review D5986.

Support of UART_DEV_DBGPORT in addition to UART_DEV_CONSOLE added to
uart_bus_fdt.c where "hw.fdt.debug" can override the FDT settings.
The following properties are looked at in /chosen for the debug node:
freebsd,debug-path.

Test Plan

I tested it on BeagleBone Black with kgdb connected, but I also have BeagleBone (white)
and RPi2 model B to check this patch if necessary.

[mhorne]

Hi! Thanks for updating this change, I was actually looking just last week at why this isn't supported.

sys/dev/uart/uart_bus_fdt.c

182

propnames_dbg seems preferable to me since this is a list.

193

Perhaps this was already discussed in the old review, but hw.fdt.debug seems vague to me. Maybe hw.fdt.debug_console or hw.fdt.dbgport?

196

Although it should not be an issue in practice, UART_DEV_CONSOLE should be an explicit case with default being a catch-all returning ENXIO.

sys/dev/uart/uart_cpu_arm64.c

104

You will need to allow UART_DEV_DBGPORT here as well or it will fail to attach.

sys/dev/uart/uart_cpu_fdt.c

91

I suspect you don't need the line break. FreeBSD's style(9) allows lines to be 80 characters in length.

[dsl]

I've tried to follow all of the comments during the review.

[dsl]

Please, take a look at the updated patch.

sys/dev/uart/uart_bus_fdt.c

182

I've re-named it as propnames_dbgport in order to align with hw.fdt.dbgport.

[andrew]

sys/dev/uart/uart_cpu_arm64.c

104

It should be removed with uart_cpu_fdt_probe checking for console types it supports.

sys/dev/uart/uart_cpu_fdt.c

90

You don't need the extra parentheses

[dsl]

Unnecessary tests of the devtype argument have been removed from uart_cpu_getdev() in sys/dev/uart/uart_cpu_arm64.c and sys/dev/uart/uart_cpu_fdt.c. uart_cpu_fdt_probe() checks for console types it supports.

[dsl]

@mhorne Should I update a patch attached to the PR also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=251053 ?

[mhorne]

In D27422#613267, @dsl_mcusim.org wrote:

@mhorne Should I update a patch attached to the PR also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=251053 ?

Linking the review in the PR is enough. We should wait a little longer for any further comments, and if there are none I can commit the change.

Edit: I missed andrew's acceptance. I can commit it shortly.

[dsl]

@mhorne, @andrew, thanks for help here :)