Am Sonntag, 18. Dezember 2022, 05:44:02 CET schrieb Gerhard Sittig:
> On Sun, 2022-12-18 at 02:29 +0100, Markus Heidelberg wrote:
> >
> > Am Samstag, 17. Dezember 2022, 20:26:42 CET schrieb Gerhard Sittig:
> > >
> > > The other issue that you noticed is that current bindings seem to
> > > completely lack u32 conversion support. Haven't checked in detail
> > > which config properties of drivers and inputs are affected. Your
> > > commit message listed a few, am not aware of seeing other users'
> > > reports for any of these. But fixing them is worthwile.
> >
> > I noticed it with the CSV input and checked for occurrences of
> > g_variant_new_uint32.
> > Since there is no command line option to pass driver config to
> > PulseView, I didn't know how these might be affected as well.
>
> There is, or should be. Passing options to -d even predates the
> -I option passing. The syntax would be:
>
>   $ pulseview -d demo:analog_channels=0

Oh, indeed! I might even have read about it a short while ago...

sr_key_info_config[] in src/hwdriver.c does not yet use u32 (wouldn't be
possible because SR_T_UINT32 didn't exist yet), so drivers won't be
affected.

Markus





_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to