Re: [PATCH net-next 06/15] net: sparx5: add constants to match data

From: Daniel Machon
Date: Wed Oct 02 2024 - 09:32:40 EST


> > +#define SPX5_CONST(const) sparx5->data->consts->const
>
> This is way too ugly too live.
> Please type the code out, there's no prize for having low LoC count.
>

Hi Jakub,

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

or pass in the sparx5 pointer to the macro too, which was the concert that
Jacob raised.

Thanks.

/Daniel