[PATCH v4 1/5] net: phy: aquantia: refresh format

[Tom Rini]

On Fri, Oct 31, 2025 at 07:33:13PM +0100, Marek Vasut wrote:
> On 10/31/25 6:21 PM, Beiyan Yun wrote:
> > 
> > 
> > > On 31 Oct 2025, at 11:51 PM, Marek Vasut <marek.vasut at mailbox.org> wrote:
> > > 
> > > On 10/31/25 4:21 PM, Beiyan Yun wrote:
> > > > Refresh format using clang-format.
> > > > Signed-off-by: Beiyan Yun <root at infi.wang>
> > > > ---
> > > > Changes in v4:
> > > > - New
> > > >   drivers/net/phy/aquantia.c | 159 +++++++++++++++++--------------------
> > > >   1 file changed, 75 insertions(+), 84 deletions(-)
> > > > diff --git a/drivers/net/phy/aquantia.c b/drivers/net/phy/aquantia.c
> > > > index 903fcd667f6..439c4c48bdc 100644
> > > > --- a/drivers/net/phy/aquantia.c
> > > > +++ b/drivers/net/phy/aquantia.c
> > > > @@ -18,28 +18,28 @@
> > > >   #include <asm/byteorder.h>
> > > >   #include <fs.h>
> > > >   -#define AQUNTIA_10G_CTL		0x20
> > > > -#define AQUNTIA_VENDOR_P1	0xc400
> > > > +#define AQUNTIA_10G_CTL 0x20
> > > > +#define AQUNTIA_VENDOR_P1 0xc400
> > > >   -#define AQUNTIA_SPEED_LSB_MASK	0x2000
> > > > -#define AQUNTIA_SPEED_MSB_MASK	0x40
> > > > +#define AQUNTIA_SPEED_LSB_MASK 0x2000
> > > > +#define AQUNTIA_SPEED_MSB_MASK 0x40
> > > #define<space>MACRO<tab>0xvalue
> > > 
> > > is perfectly fine format.
> > 
> > Agree, until clang-format has become the default.
> This change makes readability worse, no matter what formatting tool is the
> current recommendation.

Generally speaking, one should not just reformat the code, for
reformatting sake. If someone wants to tweak .clang-format and push the
changes to the kernel (whom I would assume are as picky as anyone else
here), that's fine with me.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20251031/299521e8/attachment.sig>