<marex>
kabel: s@We can fix@Fix@ ... also describe which problems do occur on affected systems in the commit message, then send V2, and then it can go in, the patch itself is fine
<marex>
kabel: oh, and include the invalid SMI address in the error message, that would be helpful during debugging
ldevulder has quit [Quit: Leaving]
Clamor has quit [Ping timeout: 264 seconds]
Clamor has joined #u-boot
monstr has quit [Ping timeout: 252 seconds]
mmu_man has joined #u-boot
hanetzer has quit [Ping timeout: 255 seconds]
hanetzer has joined #u-boot
goliath has quit [Quit: SIGSEGV]
prabhakarlad has joined #u-boot
<marex>
Kwiboo: hey, does the rk3588 stuff use TFA+U-Boot or is there some blob provider for PSCI ?
naoki has quit [Remote host closed the connection]
<LeSpocky>
just saw u-boot has include/dt-bindings/clock and include/dt-bindings/clk and that seems weird to me
ikarso has quit [Quit: Connection closed for inactivity]
<marex>
LeSpocky: OK
<LeSpocky>
linux has include/dt-bindings/clock only and judging from git it seems some people just put things to include/dt-bindings/clk as well and now we have to at91.h cl{oc,}k binding headers
<LeSpocky>
s/to/two/
<LeSpocky>
where the constants in include/dt-bindings/clk/at91.h do not match with those in linux
<marex>
LeSpocky: fix it and send patch ?
<LeSpocky>
in general yes, I'll have to dig in and find out which dts/drivers uses which, if those off-by-one thing matters, and all that with only one atmel board at hand … how hard can that be? *sigh*
<marex>
git grep + sed
<LeSpocky>
I'll look into it tomorrow, no energy for cleaning up a mess anymore today
edwinistrator2 has joined #u-boot
mmu_man has quit [Ping timeout: 252 seconds]
<sjg1>
mps: Yes, please see 37db20d0a64 video: Support showing a cursor
<mps>
sjg1: I missed this commit. thanks
jmasson has quit [Remote host closed the connection]
mmu_man has joined #u-boot
<sjg1>
apalos: OK I will be able to look at email later today
vagrantc has joined #u-boot
prabhakarlad has quit [Quit: Client closed]
prabhakarlad has joined #u-boot
mmu_man has quit [Ping timeout: 252 seconds]
mckoan is now known as mckoan|away
joeskb7 has quit [Ping timeout: 255 seconds]
<apalos>
sjg1: on the SMBIOS stuff, I cant really find a reasonable way to add tests, so I'll wait for that refactor you were planning
<apalos>
What I can add is move that function that parses the dt string to ofnode.c
<apalos>
because if I read stuff correctly ofnode_read_string_index() does something different
<apalos>
it parses a node that looks like this:
<apalos>
foo = "test1", "test2" etc, and you can select the index to read
<apalos>
What I was is tokenize existing entries on "," and pick the idx
<apalos>
like compatible = foo,bar
prabhakarlad has quit [Quit: Client closed]
___nick___ has quit [Ping timeout: 264 seconds]
<sjg1>
apalos: Oh I see...then perhaps it just needs a comment as I misunderstood the code? Re tests, we keep putting it off, but OK
<apalos>
sjg1: I can work with you on test, but once we setup smbios for *.* from start
<apalos>
Right now x86 inits it on boot, and everyone else on EFI
<apalos>
So I'd rather not waste time creating tests we will change in a release or two
<apalos>
I can help once the smbios problems are sorted
<apalos>
Do you want me me to move that function on ofnode.c? Something like ofnode_read_mode_string_index()
<apalos>
if so, I'll add a test, otherwise I'll resend that patch with the typos corrected,
frieder has quit [Remote host closed the connection]
<calebccff>
super dumb question, how should one generate a GUID for use in U-Boot (in this case capsule update efi_fw_image type_id)
<calebccff>
i notice that online GUID/UUID generators produce GUIDs with 2 bytes more than the EFI_GUID() macro in U-Boot??
<calebccff>
ah, no, i just really can't count
Clamor has quit [Ping timeout: 255 seconds]
Clamor has joined #u-boot
<sjg1>
apalos: I just mean a unit test, not a test of the whole flow. E.g. write SMBIOS tables to memory, read them back and see that it works OK
<sjg1>
apalos: We must write the tables at the start...the EFI approach is a bug :-)
<xypron>
sjg1: EFI is available from the start. You are right in that ACPI and SMBIOS table generation should not require EFI. But for memory allocation we have to find an EFI compliant solution.
<xypron>
sjg1: You can either use 64bit compliant tables (XSDT, SMBIOS3) and malloc or if you want 32bit tables you need to collaborate with EFI for the memory allocation.
<sjg1>
xypron: There are just too many disconnects here across several issues. We are going to have to discuss it somehow
<xypron>
sjg1 apalos are you available on Monday at 17:30 for a call?
prabhakarlad has joined #u-boot
Clamor has quit [Ping timeout: 268 seconds]
Clamor has joined #u-boot
<calebccff>
noticing that the fastboot implementation could use some love, and that the whole flash backend thing is not too different to the DFU backends, could this be de-duped maybe?
<sjg1>
xypron: Which time zone is that? I am free until 17:30 MST on Monday
mmu_man has joined #u-boot
prabhakarlad has quit [Ping timeout: 250 seconds]
Clamor has quit [Read error: Connection reset by peer]