Re: [PATCH net-next 2/2] net: broadcom: share header defining UniMAC registers

On Thu, 7 Jan 2021 09:14:17 -0800 Florian Fainelli wrote:

quoted

I can reproduce that after switching from mips to arm64. Before this
change bgmac.h was not using BIT() macro. Now it does and that macro
forces UL (unsigned long).

Is there any cleaner solution than below one?  

Don't use BIT(), if the constants are 32-bit unsigned integer, maybe
open coding them as (1 << x) is acceptable for that purpose.

No objections from my side, I think we already have a number of drivers
open coding the shifts for that very reason already.