[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