Re: [net-next v5 3/3] net: phy: motorcomm: Add YT8522 100M RMII PHY support

[Minda Chen]

> Add YT8522 100M RMII ethernet PHY base driver support, including PHY ID and
> base config init function.
> 
> Signed-off-by: Minda Chen <minda.chen@xxxxxxxxxxxxxxxx>
> Reviewed-by: Andrew Lunn <andrew@xxxxxxx>
> ---
>  drivers/net/phy/motorcomm.c | 49
> ++++++++++++++++++++++++++++++++++++-
>  1 file changed, 48 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/phy/motorcomm.c b/drivers/net/phy/motorcomm.c
> index dcba50007451..a202015d3869 100644
> --- a/drivers/net/phy/motorcomm.c
> +++ b/drivers/net/phy/motorcomm.c
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0+
>  /*
> - * Motorcomm 8511/8521/8531/8531S/8821 PHY driver.
> + * Motorcomm 8511/8521/8522/8531/8531S/8821 PHY driver.
>   *
>   * Author: Peter Geis <pgwipeout@xxxxxxxxx>
>   * Author: Frank <Frank.Sae@xxxxxxxxxxxxxx> @@ -14,6 +14,7 @@
> 
>  #define PHY_ID_YT8511		0x0000010a
>  #define PHY_ID_YT8521		0x0000011a
> +#define PHY_ID_YT8522		0x4f51e928
>  #define PHY_ID_YT8531		0x4f51e91b
>  #define PHY_ID_YT8531S		0x4f51e91a
>  #define PHY_ID_YT8821		0x4f51ea19
> @@ -227,6 +228,13 @@
>  #define YT8521_LED_100_ON_EN			BIT(5)
>  #define YT8521_LED_10_ON_EN			BIT(4)
> 
> +#define YT8522_EXTREG_SLEEP_CONTROL		0x2027
> +#define YT8522_EN_SLEEP_SW			15
> +

Thanks for the checking script, This should be BIT(15). I am sorry about this.
I will send v6 in next week. 

Hi Andrew
  Do you have any comment to patch 2?