[PATCH v11 3/5] net/enetfec: support queue configuration

Ferruh Yigit ferruh.yigit at intel.com
Mon Nov 15 12:15:51 CET 2021


On 11/15/2021 10:24 AM, Ferruh Yigit wrote:
> On 11/15/2021 10:11 AM, Ferruh Yigit wrote:
>> On 11/15/2021 7:19 AM, Apeksha Gupta wrote:
>>> This patch adds Rx/Tx queue configuration setup operations.
>>> On packet reception the respective BD Ring status bit is set
>>> which is then used for packet processing.
>>>
>>> Signed-off-by: Sachin Saxena<sachin.saxena at nxp.com>
>>> Signed-off-by: Apeksha Gupta<apeksha.gupta at nxp.com>
>>> Acked-by: Hemant Agrawal<hemant.agrawal at nxp.com>
>>
>> I put a comment to v9 on using 'fep->bd_addr_p_r[queue_idx]' &
>> 'fep->bd_addr_p_t[queue_idx])' 64 bit variables as 32 bit, can you please
>> check it?
> 
> hmm, 'fep->bd_addr_p_r[queue_idx]' & 'fep->bd_addr_p_t[queue_idx])' are no more
> 64-bit variables. Not sure why gcc12 complains about it, can we follow it on the
> v9 comment?

Since the 64-bit variable issue is addressed and
it is not clear from the build error if there is a real issue in the code
and gcc12 is experimental, I will continue with patch, ignoring the build
error for now, we can address it later if the issue is still there when
gcc12 is released.




More information about the dev mailing list