[PATCH v4 2/4] phy: ralink: Add PHY driver for MT7621 PCIe PHY

Sergio Paracuellos sergio.paracuellos at gmail.com
Mon Nov 30 14:34:26 UTC 2020


Hi Dan,

On Mon, Nov 30, 2020 at 1:15 PM Dan Carpenter <dan.carpenter at oracle.com> wrote:
>
> On Thu, Nov 19, 2020 at 07:05:22AM +0100, Sergio Paracuellos wrote:
> > > > +static inline void phy_write(struct mt7621_pci_phy *phy, u32 val, u32 reg)
> > > > +{
> > > > +     regmap_write(phy->regmap, reg, val);
> > >
> > > Why not use regmap_ calls directly and avoid the dummy wrappers..?
> >
> > This is because name was the dummy names are a bit shorter :) but if
> > it is also necessary I will use directly regmap_ functions.
>
> At least don't swap the last two arguments around.

You are right. I don't really know why I changed the order there but
those have been deleted in applied series.

>
> regards,
> dan carpenter

Best regards,
    Sergio Paracuellos
>


More information about the devel mailing list