[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Y3wnDOnMPHuFlCaP@lunn.ch>
Date: Tue, 22 Nov 2022 02:34:04 +0100
From: Andrew Lunn <andrew@...n.ch>
To: Frank <Frank.Sae@...or-comm.com>
Cc: Peter Geis <pgwipeout@...il.com>,
Heiner Kallweit <hkallweit1@...il.com>,
Russell King <linux@...linux.org.uk>,
"David S . Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>, yinghong.zhang@...or-comm.com,
fei.zhang@...or-comm.com, hua.sun@...or-comm.com,
netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net-next] net: phy: add Motorcomm YT8531S phy id.
> #define PHY_ID_YT8511 0x0000010a
> #define PHY_ID_YT8521 0x0000011A
> +#define PHY_ID_YT8531S 0x4F51E91A
The indentation looks odd here.
>
> -/* 0xA000, 0xA001, 0xA003 ,and 0xA006 ~ 0xA00A are common ext registers
> +/* 0xA000, 0xA001, 0xA003 , 0xA006 ~ 0xA00A and 0xA012 are common ext registers
Please remove the space before the ,
> 2.31.0.windows.1
Does that mean you are hosting your Linux builds on Windows machine?
Andrew
Powered by blists - more mailing lists