Re: [PATCH] tools/arm: Fix nr_spis handling v2

["Alejandro Vallejo" <alejandro.vallejo@xxxxxxxxx>]

On Mon Mar 24, 2025 at 3:31 PM GMT, Michal Orzel wrote:
>
>
> On 24/03/2025 16:22, Alejandro Vallejo wrote:
> > 
> > 
> > On Mon Mar 24, 2025 at 1:08 PM GMT, Michal Orzel wrote:
> >>
> >>
> >> On 19/03/2025 15:01, Alejandro Vallejo wrote:
> >>>
> >>> Doesn't this regenerate the golang bindings?
> >> FYI, it does not. The bindings are already there for NrSpis and default 
> >> value is
> >> does not result in a change (for verification I checked max_grant_frames 
> >> that
> >> uses LIBXL_MAX_GRANT_DEFAULT).
> >>
> >> ~Michal
> > 
> > Oh, good then. Though it does raise the (completely separate) question of 
> > how
> > correct those bindings might be if they ignore the IDL's default values... 
> > :/
> Why ignore. AFAICS libxl_domain_build_info_init is called there to grab 
> default
> values.
>
> ~Michal

Oh, so the default populator is itself autogenerated C with a golang binding
rather than native golang. Fair enough. The more you learn.

Cheers,
Alejandro