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