From: Pramod Kumar <pramod.kumar@xxxxxxxxxxxx>
Date: Tue, 31 May 2016 19:06:39 +0530
> +static int iproc_mdio_wait_for_idle(void __iomem *base, bool result)
> +{
> + u32 val;
> + unsigned int timeout = 1000; /* loop for 1s */
Please order your local variable declarations from longest to shortest line.
> +static int start_miim_ops(void __iomem *base,
> + u16 phyid, u32 reg, u16 val, u32 op)
> +{
> + int ret;
> + u32 param;
Likewise.
> +static int iproc_mdiomux_read(struct mii_bus *bus, int phyid, int reg)
> +{
> + int ret;
> + struct iproc_mdiomux_desc *md = bus->priv;
Likewise.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
