Hello Dmitry,
Thank you for your work to create a UI for flashrom!
Overall question to all your 1,2,3 points: if we get to implementing
these, would you be interested to get involved, for example test a
patch or involved in any other way? Especially that you would be using
the new API functions.
On Fri, Mar 14, 2025 at 10:18 AM Peter Marheine <pmarheine@chromium.org> wrote:
>
> Hey Dmitry,
>
> Thanks for the notes! There aren't many general-purpose users of the library API so I'm sure it does currently have some rough edges- it's nice to have somebody interested in improving it.
>
>> - how to obtain the list of programmers , as far as I see there is no any way to do it for now. Maybe using the log routine but it is definitely not the best way
>
>
> There isn't currently an API for this, but it would be nice to add one. I imagine a function returning strings would be sufficient, and that could possibly be expanded later with a way to discover the parameters supported by each programmer (a function taking a programmer name and returning some kind of parameter info struct).
>
>>
>> - could you add userdata to the callbacks you are calling . just a void* without any obligations
>
>
> https://review.coreboot.org/c/flashrom/+/86031 added that to the progress callback, but it looks like we should also have one for log callbacks; is that right?
>
>> - probing: in case of multiple chips it returns just an error about it. but there is no option to enumerate what was found. BTW relating to some previous thread. It would be great that libflasrom probe works like cli version (to probe everything but not only first 2)
>
>
> That doesn't seem too difficult either, and I agree it would be good to have. I imagine adding a new API similar to the internal probe_flash function allowing the user to iterate through detected chips (rather than detecting all of them and returning a list).
> _______________________________________________
> flashrom mailing list -- flashrom@flashrom.org
> To unsubscribe send an email to flashrom-leave@flashrom.org
--
Anastasia.