On Fri, Aug 05, 2016 at 05:54:21PM +0530, Nagaraju Lakkaraju wrote:
> Hello,
>
> I added all review comments and re-sending for review.
>
> >From a5017f5878a92d2acec86a6a29b1498c457cb73a Mon Sep 17 00:00:00 2001
> From: Nagaraju Lakkaraju <raju.lakkar...@microsemi.com>
> Date: Wed, 3 Aug 2016 18:28:24 +0530
> Subject: [PATCH v2] net: phy: Add drivers for Microsemi PHYs
Please use git send-email to send patches.
All comments which should not be committed to the change log should
appear after the ---. It is normal to briefly list changes between
this version and the previous version.
> +static int vsc85xx_phy_page_set(struct phy_device *phydev, u8 page)
> +{
> + int rc;
> +
> + rc = phy_write(phydev, MSCC_EXT_PAGE_ACCESS, page);
> + return rc;
Please remove rc and just do
return phy_write(phydev, MSCC_EXT_PAGE_ACCESS, page);
> +}
> +
> +static int vsc85xx_default_config(struct phy_device *phydev)
This function is setting the RGMII delay. So vsc85xx_default_config()
is not a particularly good name.
Andrew
