[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1e16ab86-ccc1-433a-a482-76d9ba567fb9@lunn.ch>
Date: Tue, 14 Oct 2025 21:41:42 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Parvathi Pudi <parvathi@...thit.com>
Cc: andrew+netdev@...n.ch, davem@...emloft.net, edumazet@...gle.com,
kuba@...nel.org, pabeni@...hat.com, danishanwar@...com,
rogerq@...nel.org, pmohan@...thit.com, basharath@...thit.com,
afd@...com, linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, pratheesh@...com,
prajith@...com, vigneshr@...com, praneeth@...com, srk@...com,
rogerq@...com, krishna@...thit.com, mohan@...thit.com
Subject: Re: [PATCH net-next v3 1/3] net: ti: icssm-prueth: Adds helper
functions to configure and maintain FDB
> +/* 4 bytes */
> +struct fdb_index_tbl_entry_t {
> + /* Bucket Table index of first Bucket with this MAC address */
> + u16 bucket_idx;
> + u16 bucket_entries; /* Number of entries in this bucket */
> +};
Please drop the _t. That is normally used to indicate a type, but this
is actually a struct
> +void icssm_prueth_sw_fdb_tbl_init(struct prueth *prueth)
> +{
> + struct fdb_tbl *t = prueth->fdb_tbl;
> +
> + t->index_a = (struct fdb_index_array_t *)((__force const void *)
> + prueth->mem[V2_1_FDB_TBL_LOC].va +
> + V2_1_FDB_TBL_OFFSET);
You cast it to a const void * and then a non-const fdb_index_array_t *?
Powered by blists - more mailing lists