[PATCH net-next 06/15] net: sparx5: add constants to match data
Daniel Machon
daniel.machon at microchip.com
Wed Oct 2 11:28:00 PDT 2024
> > By "type the code out" - are you saying that we should not be using a macro
> > for accessing the const at all? and rather just:
> >
> > struct sparx5_consts *consts = sparx5->data->consts;
> > consts->some_var
>
> This.
>
> > or pass in the sparx5 pointer to the macro too, which was the concert that
> > Jacob raised.
>
> The implicit arguments are part of the ugliness, and should also go
> away. But in this case the entire macro should go.
Ack.
Will get rid of the macro in v2.
/Daniel
More information about the linux-arm-kernel
mailing list