On Sat, Aug 25, 2018 at 11:00:39AM +0200, Felix Fietkau wrote:
> On 2018-08-24 13:15, Stanislaw Gruszka wrote:
> > MAC registers are the same for mt76x0 and mt76x2. Rename mt76x2_regs.h
> > file to reflect that.
> > 
> > Signed-off-by: Stanislaw Gruszka <[email protected]>
> > ---
> >  drivers/net/wireless/mediatek/mt76/mt76x2.h      |   2 +-
> >  drivers/net/wireless/mediatek/mt76/mt76x2_regs.h | 619 
> > -----------------------
> >  drivers/net/wireless/mediatek/mt76/mt76xx_regs.h | 619 
> > +++++++++++++++++++++++
> I think mt76xx_regs.h is named too generic. MT7603, MT7628, MT7615 and
> all newer chips use a completely different register layout.
> I'd prefer to stick with mt76x2_regs for now and use it for x0 as well.

I thought about that too, but choose it will be better to have some
prefix that will indicate the function/structure/header is shared between
the x0 and x2 drivers to not confuse mt76x2 code that is mt76x2 only
and one that is shared.

I choose mt76xx name, considered mt76 to be generic prefix shared also 
between old and new chips. But yeah this mt76xx name also "contains" new
chips. How about mt76old_ or mt76og_ (old generation) ?

Thanks
Stanislaw


Reply via email to