persmule has quit [Remote host closed the connection]
persmule has joined #u-boot
mmu_man has joined #u-boot
<deathcamel57>
Per the "Debugging driver model" documentation, it recommends using `dm_dump_all()`, which isn't in the U-Boot codebase. Is there an alternative function? I'd also love to update the documentation (would like advice on how to submit a patch).
<deathcamel57>
Found it, it's been renamed to `dm_dump_tree()`.
monstr has joined #u-boot
monstr has quit [Ping timeout: 255 seconds]
slobodan_ has quit [Read error: Connection reset by peer]
<deathcamel57>
Anyone ever mainlined anything? I'd like some input on my commit message for the documentation change.
<dsimic>
perhaps no quotation marks are needed around the function names, for example
persmule has quit [Remote host closed the connection]
<dsimic>
the message needs to wrap around column 78 or so
<dsimic>
I'd also reword it a bit, to read better
persmule has joined #u-boot
<deathcamel57>
Updated that gist
<dsimic>
"Replace dm_dump_all() with dm_dump_tree() in driver model documentation, to reflect changes introduced in commit 14528704."
<dsimic>
that's how I'd write it
<deathcamel57>
Updated. Do you mean like that?
<deathcamel57>
Sorry, fixed wrapping now.
<dsimic>
the wording looks fine to me
slobodan has joined #u-boot
persmule has quit [Remote host closed the connection]
<deathcamel57>
Patman's giving me `Missing commit description`. Where would I add that? I thought that was in the commit message?
persmule has joined #u-boot
<mkorpershoek>
deathcamel57: you need to add a blank line between the title and the description
<dsimic>
true
<deathcamel57>
I got the awaiting moderator approval message! Thanks so much dsimic and mkorpershoek!
naoki has quit [Quit: naoki]
<dsimic>
I'm glad to help
<mkorpershoek>
thank you for contributing !
<dsimic>
that too :)
prabhakarlad has quit [Quit: Client closed]
tnovotny has joined #u-boot
stipa has joined #u-boot
matthias_bgg has quit [Ping timeout: 258 seconds]
<paulbarker>
marex: Damn. The patch changelogs are in the wrong place in the patches I just sent. Want me to resend or can you fixup when you apply them?
<deathcamel57>
I'm trying to set up SoC detection in my `mach-%` folder. I've written a SOC driver, and put the node in my DTS. However, `print_cpuinfo()`, isn't able to `soc_get()` it. Using `dm_dump_tree()`, I'm seeing a few nodes, but most of my DTS nodes are missing, including the SoC driver node. Why would that be?
<deathcamel57>
That's how I ran across the `dm_dump_tree()` issue in the docs btw.
matthias_bgg has joined #u-boot
frieder has quit [Remote host closed the connection]