Page MenuHomeFreeBSD
Differential D3559

Allow the FDT uart driver to select witch device node to use
ClosedPublic
Actions

Authored by andrew on Sep 3 2015, 4:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 13, 11:08 PM
Unknown Object (File)
Thu, Nov 13, 11:08 PM
Unknown Object (File)
Thu, Nov 13, 8:26 PM
Unknown Object (File)
Thu, Nov 13, 5:28 PM
Unknown Object (File)
Thu, Nov 13, 2:36 AM
Unknown Object (File)
Thu, Nov 13, 1:24 AM
Unknown Object (File)
Sat, Nov 1, 5:57 PM
Unknown Object (File)
Tue, Oct 28, 10:09 PM
View All Files
Subscribers
imp

Details

Reviewers
None
Group Reviewers
MIPS
ARM
Commits
rS287565: Allow us to set the console device tree node. This is needed as not all
Summary

Some upstream dts files lack a stdout in /choosen. In these cases
we default to serial0, however this may be incorrect. This adds
support to set the node we should use in loader.

An example of this would be to add the following to loader.conf:
hw.fdt.console="/smb/uart@f7113000"

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

andrew updated this revision to Diff 8460.Sep 3 2015, 4:57 PM
andrew retitled this revision from to Allow the FDT uart driver to select witch device node to use.
andrew updated this object.
andrew edited the test plan for this revision. (Show Details)
andrew added reviewers: ARM, MIPS.
Herald added a subscriber: imp. · View Herald TranscriptSep 3 2015, 4:57 PM
imp added a comment.Sep 3 2015, 5:33 PM

Generally I like this, though hw.fdt.console seems limiting. hw.uart.console already exists, and also lets you set UART parameters. I see that we do this to a limited extent for current-speed below and hard-wire 8:1:N. We've done this for a long time, so I don't know how important it is, but I thought I'd point out the inconsistency.

sys/dev/uart/uart_cpu_fdt.c
156 ↗(On Diff #8460)

There are other things than /chose/stdin /chosen/stdout. Perhaps you should limit things explicitly to those?

166–167 ↗(On Diff #8460)

I'm torn. Is there some reason we don't do this when the user has specified a node that doesn't exist (eg node == -1 when cp != NULL)

andrew added a comment.Sep 3 2015, 6:25 PM

We could follow the stdout-path bindings [1] in this node, but this would need us to parse them first.

I'm also open to a better name for the environment variable.

[1] https://www.kernel.org/doc/Documentation/devicetree/bindings/chosen.txt

sys/dev/uart/uart_cpu_fdt.c
156 ↗(On Diff #8460)

This is just changing the indentation of existing code, there is extra data in some of these, but that is out of the scope for this change.

166–167 ↗(On Diff #8460)

The user could disable the fdt console with this by setting it to an invalid node, e.g. hw.fdt.console="none"

Closed by commit rS287565: Allow us to set the console device tree node. This is needed as not all (authored by andrew). · Explain WhySep 8 2015, 4:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
uart/
28 lines

Diff 8591

View Options

head/sys/dev/uart/uart_cpu_fdt.c

Loading...