On 2020-07-21, Jonas Smedegaard wrote:
> Quoting Jonas Smedegaard (2020-07-21 02:28:11)
>> Quoting Vagrant Cascadian (2020-07-20 22:05:43)
>> > I think could probably be trimmed down to a more minimal patch. e.g. 
>> > It doesn't look like CONFIG_USE_PREBOOT is needed at all; I *think* 
>> > you could just use CONFIG_PREBOOT without adding Kconfig support 
>> > (needs testing) and then use ifdef/ifndef directly where the preboot 
>> > command is added.
>> 
>> Sorry, I don't follow how you think it could be done without per-board 
>> definitions.
>
> Sleeping on it helped: I figured it out - here's a much shorter patch: 
> https://salsa.debian.org/debian/u-boot/-/commit/17bcf50

Yeah, that's more like what I was thinking! Clever figuring out
configuration options only present on the teres-i. :)


> Tested to work with TERES-I laptop.
>
> I also issued a minor update to the upstream patch: 
> https://patchwork.ozlabs.org/project/uboot/list/?series=191157

Yeah, wondered about usb "reset" vs usb "start".


Nice work!


live well,
  vagrant

Attachment: signature.asc
Description: PGP signature

Reply via email to