Tartarus changed the topic of #u-boot to: SOURCE MOVED TO https://source.denx.de/u-boot/u-boot.git / U-Boot v2023.07.02, v2023.10-rc2 are OUT / Merge Window is CLOSED, next branch is OPEN / Release v2023.10 is scheduled for 02 October 2023 / Channel archives at https://libera.irclog.whitequark.org/u-boot
<sjg1> marex: Yes you can cast. That is common
<sjg1> marex: Re pointers, I mean that EFI code uses a u64 to hold a pointer and (I think) a void * to hold an address. But it is a long time since I looked at it
<marex> sjg1: ignore those two patches for now
<marex> sjg1: I think you will have to revisit it soon due to the binman EFI capsule stuff, no ?
<sjg1> marex: I think it is separate, for now.
<marex> sjg1: well, I finally misused CI to spit out .img files for the remaining 7 or so tests
<marex> so ...
<marex> ugh
<sjg1> marex: So your local setup can't do that?
<marex> sjg1: it was easier to spent 10 seconds on having CI do it
<marex> *send
<marex> *spend
<marex> sjg1: frankly, I would expect userspace tool test suite to not spam host system with sudo ...
<sjg1> marex: Perhaps you can fix it? It annoys me too. I remember when I had to sudo to write to a USB stick, but now that has been fixed
<marex> sjg1: perhaps I can add it as another entry on the todo ?
persmule has quit [Remote host closed the connection]
mmu_man has quit [Ping timeout: 245 seconds]
<sjg1> marex: SGTM
<marex> sjg1: perhaps we need a public todo for the project
<marex> aha ... so ... another bug fixed
<marex> sjg1: is that Tartarus only thing ?
<sjg1> No, but there is one for each custodian, I believe
persmule has joined #u-boot
___nick___ has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
___nick___ has joined #u-boot
___nick___ has quit [Client Quit]
___nick___ has joined #u-boot
vagrantc has joined #u-boot
persmule has quit [Remote host closed the connection]
persmule has joined #u-boot
pgreco has quit [Remote host closed the connection]
pgreco has joined #u-boot
alpernebbi has quit [Ping timeout: 248 seconds]
alpernebbi has joined #u-boot
camus has joined #u-boot
pbsds has quit [Quit: The Lounge - https://thelounge.chat]
pbsds has joined #u-boot
swiftgeek has quit [Ping timeout: 246 seconds]
vagrantc has quit [Quit: leaving]
redbrain has quit [Read error: Connection reset by peer]
redbrain has joined #u-boot
swiftgeek has joined #u-boot
sng has joined #u-boot
sng has quit [Remote host closed the connection]
sng has joined #u-boot
Clamor has joined #u-boot
Wouter0100670440 has quit [Quit: The Lounge - https://thelounge.chat]
Wouter0100670440 has joined #u-boot
naoki has joined #u-boot
gsz has joined #u-boot
devarsh has joined #u-boot
dormito has quit [Ping timeout: 240 seconds]
goliath has joined #u-boot
sng has quit [Remote host closed the connection]
sng has joined #u-boot
dormito has joined #u-boot
sng has quit [Read error: Connection reset by peer]
stefanro has joined #u-boot
sng has joined #u-boot
sng has quit [Remote host closed the connection]
sng has joined #u-boot
ezulian has joined #u-boot
frieder has joined #u-boot
ikarso has joined #u-boot
sszy has joined #u-boot
naoki has quit [Quit: naoki]
sng has quit [Remote host closed the connection]
Wouter0100670440 has quit [Quit: The Lounge - https://thelounge.chat]
Wouter0100670440 has joined #u-boot
sng has joined #u-boot
Clamor has quit [Ping timeout: 248 seconds]
Clamor has joined #u-boot
<mps> do anyone here use starfive visionfive v2 with mainline kernel and mainline u-boot? For me mainline kernel doesn't work with mainline u-boot, while it works fine with starfive u-boot
frieder has quit [Read error: Connection reset by peer]
vigneshr has quit [Quit: Connection closed for inactivity]
frieder has joined #u-boot
camus has quit [Ping timeout: 256 seconds]
camus has joined #u-boot
Brainzman has joined #u-boot
sng has quit [Remote host closed the connection]
prabhakarlad has quit [Quit: Client closed]
<marex> Clamor: what is that ?
<marex> Clamor: I see ton of ads and other goo
<Clamor> This is a log from uboot I took. I will switch paste service if this is crowded
<marex> paste.debian.net seems to work well, and no ads
<Clamor> I have inserted logs in pmic, regualtors and i2c probes to track calls
<marex> ahh, lemme check the driver in mainline U-Boot source tree, since I assume that's what you're using (there is no version information in the incomplete log)
<marex> $ git grep max77663_sd_probe | wc -l
<marex> hmmmmm
<marex> 0
<Clamor> It is not merged yet
<Clamor> I was testing these regulator updates https://github.com/clamor-s/u-boot/commits/pmic-uclass-new
<Clamor> Last 4 commits
mmu_man has joined #u-boot
<Clamor> Pmic and regulator drivers can be found here https://github.com/clamor-s/u-boot/commits/pmic-devices
<Clamor> Though they are fairly simple and should not fail
<Clamor> I am using the latest mainline u-boot with those patches on top
<marex> if you need drivers to be probed after bind, there is a PROBE_AFTER_BIND DM flag for it
<Clamor> No it is not included
<marex> dont use the post bind hack
<Clamor> I am not, though it fixes situation
<Clamor> I am aware of probe_after_bind flag
<marex> yep, just yesterday I spent like 5 hours debugging memory leak caused by such misuse
<Clamor> I am testing commits from pmic-uclass-new with just max77663 drivers
<marex> Clamor: and probe of some of the driver is called twice, right ?
<Clamor> Yes, regulator drivers as you can see
<Clamor> First time regulator drivers probe is called it triggers all chain of probes, but not the second time
<marex> Clamor: do you know dm_dump_tree() ?
<marex> might give you an idea about the state of DM tree before/after the probe
<marex> maybe some things are not probed because they are already probed
<Clamor> dm tree? Yes I have it
<marex> Clamor: also, try and add dummy .remove and .unbind calls to your drivers, could it be those are called ?
<marex> Clamor: maybe the driver is unbound at some point too, and then reprobed
<Clamor> It feels like i2c is unbound but uboot assumes it is still there
<Clamor> Good advice, thanks
<marex> Clamor: if I wasn't fighting with similar stuff a few hours ago ... oh well
<marex> Clamor: you can spam the code with dm_dump_tree() to check the state of the DM (and what is probed and not probed)
<marex> there is this [+] in the listing if device is probed , [ ] otherwise
<Clamor> Yes, I know how tree works
<Clamor> Thanks
<marex> Clamor: you can also always just add printf()s to DM core and print dev->name and dev->drv->name all over the place
<Clamor> Ok, I see no unbinds and remove
<marex> (yeah, stack unwinding for backtraces would be nice to have)
<marex> Clamor: if you have no unbind/remove implemented, DM core calls the generic ones
<Clamor> I have dummy calls with prints
<marex> and neither is called ?
<Clamor> Neither
<marex> Clamor: what about in the regulator(s) ?
<marex> Clamor: stick printf into device_remove() and device_unbind() for maximum ease
<Clamor> I did
<Clamor> No calls
<Clamor> marex: may you check those 4 commits with your board (if pmics are implemented)?
<marex> btw are you able to reproduce it on sandbox too ?
<Clamor> Bo
sng has joined #u-boot
<Clamor> No, I cannot reproduce it on sandbox
<Clamor> That post probe stuff is required to set up regulator according to tree
<marex> which tree ? DT ?
<marex> is that some regulator configuration stuff ?
<Clamor> Yes, device tree
<Clamor> It calls regulator autoconfiguration
<Clamor> So there is no need is calls from board
<marex> Clamor: and that cannot happen at probe time because the regulators might not all be available yet ?
<marex> Clamor: hence the post probe ?
sng has quit [Remote host closed the connection]
sng has joined #u-boot
<Clamor> Not to call it in every regulator driver probe
<marex> Clamor: why not ?
<marex> I think mmc subsystem does it too
<marex> for example ... it does call generic function in each driver probe I think
<Clamor> Driver probe replaces generic probe
<Clamor> Post probe in this case is uclass extension of driver probe
<marex> Clamor: you might want to double check how the uclass post* functions really behave
<marex> Clamor: the only other reason I can think of why the driver would be removed is some internal DM state corruption, which would be real bad
<Clamor> marex: It behaves exactly as intended, and I see no reason why the post probe should not be used in a way it was used.
Brainzman has quit [Quit: Brainzman]
Clamor has quit [Ping timeout: 246 seconds]
Clamor has joined #u-boot
ikarso has quit [Quit: Connection closed for inactivity]
prabhakarlad has joined #u-boot
Wouter0100670440 has quit [Quit: The Lounge - https://thelounge.chat]
Wouter0100670440 has joined #u-boot
sng has quit [Remote host closed the connection]
sng has joined #u-boot
Clamor has quit [Ping timeout: 256 seconds]
sakman has quit [Remote host closed the connection]
swiftgeek has quit [Quit: swiftgeek]
sakman has joined #u-boot
swiftgeek has joined #u-boot
swiftgeek has quit [Quit: swiftgeek]
swiftgeek has joined #u-boot
Perflosopher has quit [Quit: Ping timeout (120 seconds)]
swiftgeek has quit [Client Quit]
Perflosopher has joined #u-boot
swiftgeek has joined #u-boot
frieder has quit [Ping timeout: 245 seconds]
macromorgan has quit [Ping timeout: 245 seconds]
macromorgan has joined #u-boot
Brainzman has joined #u-boot
swiftgeek has quit [Quit: swiftgeek]
swiftgeek has joined #u-boot
frieder has joined #u-boot
frieder has quit [Remote host closed the connection]
macromorgan has quit [Ping timeout: 246 seconds]
macromorgan has joined #u-boot
prabhakarlad has quit [Quit: Client closed]
macromorgan is now known as Guest7868
macromorgan has joined #u-boot
sng has quit [Remote host closed the connection]
Guest7868 has quit [Ping timeout: 245 seconds]
mmu_man has quit [Ping timeout: 246 seconds]
Clamor has joined #u-boot
mmu_man has joined #u-boot
prabhakarlad has joined #u-boot
Clamor has quit [Ping timeout: 250 seconds]
prabhakarlad has quit [Quit: Client closed]
Brainzman has quit [Ping timeout: 256 seconds]
tepperson has joined #u-boot
ikarso has joined #u-boot
tepperson has quit [Quit: Client closed]
mmu_man has quit [Ping timeout: 244 seconds]
ezulian has quit [Ping timeout: 248 seconds]
sng has joined #u-boot
mmu_man has joined #u-boot
sszy has quit [Quit: http://quassel-irc.org - Chat comfortably. Anywhere.]
silurian_invader has joined #u-boot
silurian_invader has quit [Read error: Connection reset by peer]
sng has quit [Remote host closed the connection]
sng has joined #u-boot
___nick___ has quit [Ping timeout: 260 seconds]
sng has quit [Remote host closed the connection]
silurian_invader has joined #u-boot
prabhakarlad has joined #u-boot
prabhakarlad has quit [Quit: Client closed]
gsz has quit [Ping timeout: 260 seconds]
prabhakarlad has joined #u-boot
lucascastro has joined #u-boot
ikarso has quit [Quit: Connection closed for inactivity]
Wouter0100670440 has quit [Quit: The Lounge - https://thelounge.chat]
Wouter0100670440 has joined #u-boot
lucascastro has quit [Ping timeout: 245 seconds]
camus1 has joined #u-boot
camus has quit [Ping timeout: 248 seconds]
camus1 is now known as camus
cakes has quit [Quit: No Ping reply in 180 seconds.]
cakes has joined #u-boot
cakes has joined #u-boot
persmule has quit [Remote host closed the connection]
persmule has joined #u-boot
goliath has quit [Quit: SIGSEGV]
naoki has joined #u-boot