warpme has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
vagrantc has joined #u-boot
Jones42_ has joined #u-boot
Jones42 has quit [Ping timeout: 248 seconds]
naoki has quit [Quit: naoki]
npcomp has quit [Ping timeout: 244 seconds]
Jones42 has joined #u-boot
Jones42_ has quit [Ping timeout: 265 seconds]
slobodan has quit [Ping timeout: 252 seconds]
mmu_man has quit [Ping timeout: 252 seconds]
npcomp has joined #u-boot
mmu_man has joined #u-boot
goliath has joined #u-boot
<Tartarus>
calebccff: Do you think you can add a test for that lmb problem (a) at all (b) and today ?
ikarso has quit [Quit: Connection closed for inactivity]
lehmanju has quit [Ping timeout: 272 seconds]
___nick___ has quit [Ping timeout: 252 seconds]
Stat_headcrabbed has quit [Remote host closed the connection]
<calebccff>
Tartarus: lmb notify is not tested currently
<Tartarus>
Right
<Tartarus>
To be clear, I want to pull your patch asap
<Tartarus>
So, if you can't add the test now, can you figure out testing soon?
<Tartarus>
And/or work with sng and apalos to get it tested
<calebccff>
can do, tbh i'd rather just make lmb_add_region() never return a positive integer, doesn't seem like it's useful
<calebccff>
Tartarus: I can't find a single place where the coalesce value from lmb_add_region_flags() is used, and as an API it seems unintuitive. How about we just remove that capability and get rid of the risk vector entirely?
<Tartarus>
calebccff: Saying we need to rework this a bit more is fine too, yeah.
<Tartarus>
We pulled v4 in now for reasons well litigated on the mailing list
<Tartarus>
So another smaller series to further clean it up makes sense
rvalue has quit [Read error: Connection reset by peer]
rvalue has joined #u-boot
slobodan has joined #u-boot
naoki has joined #u-boot
<sjg1>
calebccff: If there are no tests today, then they can come later :-(
slobodan has quit [Ping timeout: 255 seconds]
slobodan has joined #u-boot
slobodan_ has joined #u-boot
* Crofton
wonders if I need to subscribe to the u-boot list to post
<Tartarus>
Crofton: no, i'll just clear the message when I see it
<Tartarus>
Did something not get posted?
<Crofton>
no, I just need to send a one liner, basically fix an implicit declaration