rvalue has quit [Read error: Connection reset by peer]
rvalue has joined #u-boot
thopiekar has joined #u-boot
<cambrian_invader>
sjg1: is there an example of refactoring out helper python functions used in tests?
<cambrian_invader>
I'd like to reuse the helpers from test_fit
<cambrian_invader>
but they seem to reference local variables like cons
<paulbarker>
sjg1: I have a good guess why enabling CONFIG_SPL_OF_CONTROL prevents boot on the BeagleBone but I'm getting a bit lost in the code trying to confirm my suspicions
<paulbarker>
The board detect code uses `uclass_get_device_by_seq()` to get the relevant i2c bus
<paulbarker>
But the board detect code is called from board_init_f
<paulbarker>
I wonder if, when OF_CONTROL is enabled, the dtb hasn't been parsed yet at this stage and so no i2c busses exist
<yang2>
WoC-: I can place U-boot on the USB key (this is what I did, when flashing) or onto SD card (slot)
rvalue has quit [Read error: Connection reset by peer]
rvalue has joined #u-boot
sicelo_ has joined #u-boot
sicelo_ has joined #u-boot
lucascastro has joined #u-boot
<WoC->
Looks good to me
sicelo_ has quit [Quit: Bye!]
sicelo_ has joined #u-boot
sicelo_ has joined #u-boot
sicelo_ has quit [Changing host]
ladis has quit [Quit: Leaving]
___nick___ has quit [Ping timeout: 260 seconds]
sicelo_ has quit [Quit: Bye!]
sicelo_ has joined #u-boot
<yang2>
WoC-: a few days ago, I've reported a bricked utilite device
prabhakarlad has joined #u-boot
<sjg1>
paulbarker: Eek, yes that code should move, if possible. Ideally to spl_board_init()
<sjg1>
paulbarker: The only other option is to call spl_early_init() first, from your board_init_f() function
<sjg1>
cambrian_invader: Hmmm I imagined I sent a series for that but did not
<sjg1>
cambrian_invader: See u-boot-dm/vp3-working 'vbe: Use a manual test'
<cambrian_invader>
well, it turned out I only needed 4 or so lines for a basic test
<cambrian_invader>
hm, I'm not sure if the linked commit addresses what I had in mind
persmule has quit [Remote host closed the connection]
persmule has joined #u-boot
<sjg1>
cambrian_invader: Oh right, my commit addresses having test setup in Python with the rest of the test in C
<cambrian_invader>
that would have worked fine for the test I was writing as well
<paulbarker>
sjg1: It's not as simple as I thought, spl_early_init() is called first and parses the dtb, however only the top-level nodes are parsed
<paulbarker>
It doesn't descend the tree and find the i2c nodes
<paulbarker>
I expect the top level `ocp` node, which uses the simple-bus driver, needs to be probed as well
<paulbarker>
Huh, actually, no, simple_bus_post_bind() calls dm_scan_fdt_dev(dev) so it should recursive down and bind the devices
vagrantc has quit [Quit: leaving]
<sjg1>
paulbarker: You might need u-boot,dm-spl tags in the nodes that you want to appear in SPL
<sjg1>
paulbarker: Also check dm_dump_tree() which you can call in SPL to see what devices are present
sicelo_ has quit [Quit: Bye!]
sicelo_ has joined #u-boot
<paulbarker>
sjg1: dm-spl tags may be what's missing
<paulbarker>
I expect `arch/arm/dts/am33xx-u-boot.dtsi` needs expanding a little, and we may need a beaglebone u-boot.dtsi file as well to add those tags
sicelo_ has quit [Quit: Bye!]
sicelo_ has joined #u-boot
mncheck has quit [Ping timeout: 260 seconds]
sicelo_ has quit [Remote host closed the connection]
<paulbarker>
I'll pick it up tomorrow
sicelo_ has joined #u-boot
sicelo_ has quit [Changing host]
sicelo_ has joined #u-boot
sicelo_ has quit [Remote host closed the connection]
sicelo_ has joined #u-boot
sicelo_ has joined #u-boot
minimal has joined #u-boot
lucascastro has quit [Remote host closed the connection]