Dec 2 2020
@mhorne Should I update a patch attached to the PR also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=251053 ?
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.
Dec 1 2020
Please, take a look at the updated patch.
I've tried to follow all of the comments during the review.
Nov 30 2020
Nov 25 2020
There's some information here if you haven't found it already: https://wiki.freebsd.org/Phabricator
No, I haven't seen it yet - thanks for sharing! What's a preferred way to create a revision: via command line or web interface?
Nov 24 2020
Would you be willing to create a new phabricator review for the change?
Nov 11 2020
I've re-worked this patch and opened a PR at https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=251053.