Page MenuHomeFreeBSD
Differential D5986

Add support of UART_DEV_DBGPORT to uart_cpu_fdt.c
AbandonedPublic
Actions

Authored by stevek on Apr 18 2016, 4:38 PM.
Tags
None
Referenced Files
F103553664: D5986.diff
Tue, Nov 26, 11:11 AM
Unknown Object (File)
Thu, Nov 21, 10:02 AM
Unknown Object (File)
Thu, Nov 21, 10:02 AM
Unknown Object (File)
Mon, Nov 18, 9:24 AM
Unknown Object (File)
Sat, Nov 16, 1:18 PM
Unknown Object (File)
Oct 25 2024, 5:03 PM
Unknown Object (File)
Oct 20 2024, 5:51 PM
Unknown Object (File)
Oct 4 2024, 7:33 AM
View All 86 Files
Subscribers
andrew
dsl
imp
jhibbits
markj
Contributor Reviews (src)

Details

Reviewers
None
Group Reviewers
ARM
Summary

Suppport UART_DEV_DBGPORT in addition to UART_DEV_CONSOLE in uart_cpu_fdt.c

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

Build and boot on ARMv7 board (in this case this was done with the BCM56340-based board that I am working with) with debug port specified in the FDT.
Attach gdb to the remote debug port.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

stevek updated this revision to Diff 15289.Apr 18 2016, 4:38 PM
stevek retitled this revision from to Add support of UART_DEV_DBGPORT to uart_cpu_fdt.c.
stevek updated this object.
stevek edited the test plan for this revision. (Show Details)
stevek set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptApr 18 2016, 4:38 PM
jhibbits added a reviewer: ARM.Apr 18 2016, 4:47 PM
jhibbits added a subscriber: jhibbits.

An active ARM developer should review this. Looks good to me, though.

andrew added a subscriber: andrew.Apr 26 2016, 11:45 AM

I've discussed this with one of the Linux ARM developers. He suggested the name should be freebsd,debug-path to ensure there is no conflict with Linux.

imp added a comment.Apr 26 2016, 3:23 PM
In D5986#129929, @andrew wrote:

I've discussed this with one of the Linux ARM developers. He suggested the name should be freebsd,debug-path to ensure there is no conflict with Linux.

In general, anything we invent should start with freebsd, while anything standardized should not.

andrew added a comment.Apr 26 2016, 5:08 PM
In D5986#129972, @imp wrote:
In D5986#129929, @andrew wrote:

I've discussed this with one of the Linux ARM developers. He suggested the name should be freebsd,debug-path to ensure there is no conflict with Linux.

In general, anything we invent should start with freebsd, while anything standardized should not.

If it makes sense to standardise it it would pay to raise the issue on devicetree@vger.kernel.org.

What is the advantage of adding this to the dts over just the kenv variable?

stevek added a comment.May 2 2016, 5:24 PM
In D5986#130003, @andrew wrote:

What is the advantage of adding this to the dts over just the kenv variable?

One case that it's been useful for us at Juniper is when we boot up what we call a "bringup kernel", where the FDT is static and a rootfs is embedded in the kernel. The loader (whatever it is, it could be something other that ubldr, loader, or u-boot) doesn't need to be able to pass environment on to the kernel that it can understand and the remote debugger port can "just work" (assuming the UART is set up correctly and the system can get booted far enough.)

imp added a comment.May 2 2016, 5:32 PM

Looks good to me.

sys/dev/uart/uart_cpu_fdt.c
132

most properly, this should be freebsd,debug-path since debug-path isn't a standard thing.

stevek updated this revision to Diff 17424.Jun 8 2016, 4:03 PM

Updated per review comments.

stevek updated this object.Jun 10 2016, 2:10 PM
stevek abandoned this revision.Aug 9 2016, 11:46 PM

It looks like recent changes to uart would require a re-work of this change, abandoning this review for now.

dsl added a subscriber: dsl.Nov 10 2020, 5:24 PM
dsl added a comment.EditedNov 11 2020, 4:37 PM

I've re-worked this patch and opened a PR at https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=251053.

markj added a subscriber: markj.Nov 24 2020, 6:28 PM
In D5986#606736, @dsl_mcusim.org wrote:

I've re-worked this patch and opened a PR at https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=251053.

Would you be willing to create a new phabricator review for the change?

dsl added a comment.Nov 24 2020, 8:17 PM

Would you be willing to create a new phabricator review for the change?

Of course :) But I'll have to understand how to do it properly first.

markj added a comment.Nov 25 2020, 4:40 PM
In D5986#611022, @dsl_mcusim.org wrote:

Would you be willing to create a new phabricator review for the change?

Of course :) But I'll have to understand how to do it properly first.

There's some information here if you haven't found it already: https://wiki.freebsd.org/Phabricator

dsl added a comment.Nov 25 2020, 6:39 PM

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?

markj added a comment.Nov 26 2020, 5:00 PM
In D5986#611427, @dsl_mcusim.org wrote:

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?

It's up to you. If you upload a lot of patches I find it easier to use the arc command-line tool so you can upload git commits. If you upload a patch via the web interface, please ensure to generate a patch with lots of context (-U99999).

dsl mentioned this in D27422: Add support of UART_DEV_DBGPORT to uart_bus_fdt.c.Nov 30 2020, 3:18 PM
mhorne mentioned this in rS368284: uart: allow UART_DEV_DBGPORT for fdt consoles.Dec 2 2020, 9:02 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
uart/
18 lines

Diff 15289

View Options

sys/dev/uart/uart_cpu_fdt.c

Loading...