[U-Boot] [PATCH v2 2/5] net: phy: Add support for Marvell M88E1512
Stefan Roese
sr at denx.de
Fri Dec 9 14:43:24 CET 2016
On 09.12.2016 14:38, Phil Edworthy wrote:
> This device also works with the 88E1518 code, so we just adjust
> the UID mask accordingly.
>
> Signed-off-by: Phil Edworthy <phil.edworthy at renesas.com>
> ---
> v2:
> Don't add a new entry, just adjust the UID mask.
> ---
> drivers/net/phy/marvell.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
> index 95d1492..5adfe7d 100644
> --- a/drivers/net/phy/marvell.c
> +++ b/drivers/net/phy/marvell.c
> @@ -585,10 +585,14 @@ static struct phy_driver M88E1510_driver = {
> .shutdown = &genphy_shutdown,
> };
>
> +/* This supports:
> + * 88E1518, uid 0x1410dd1
> + * 88E1512, uid 0x1410dd4
> + */
Nitpicking comment:
Please use this multiline comment style:
/*
* This supports:
* 88E1518, uid 0x1410dd1
* 88E1512, uid 0x1410dd4
*/
> static struct phy_driver M88E1518_driver = {
> .name = "Marvell 88E1518",
> .uid = 0x1410dd1,
> - .mask = 0xfffffff,
> + .mask = 0xffffffb,
> .features = PHY_GBIT_FEATURES,
> .config = &m88e1518_config,
> .startup = &m88e1011s_startup,
>
Reviewed-by: Stefan Roese <sr at denx.de>
Thanks,
Stefan
More information about the U-Boot
mailing list