On Fri, Aug 05, 2016 at 05:54:21PM +0530, Nagaraju Lakkaraju wrote:
> Hello,
>
> I added all review comments and re-sending for review.
You should also take a look at the output of scripts/checkpatch.pl:
total: 9 errors, 82 warnings, 1 checks, 179 lines checked
Andrew
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
> Date: Wed, 3 Aug 2016 18:28:24 +0530
> Subject: [PATCH
Hello,
I added all review comments and re-sending for review.
>From a5017f5878a92d2acec86a6a29b1498c457cb73a Mon Sep 17 00:00:00 2001
From: Nagaraju Lakkaraju
Date: Wed, 3 Aug 2016 18:28:24 +0530
Subject: [PATCH v2] net: phy: Add drivers for Microsemi PHYs
Signed-off-by: Nagaraju Lakkaraju
---
On Fri, Jul 29, 2016 at 10:17:36AM +0200, Andrew Lunn wrote:
> EXTERNAL EMAIL
>
>
> > > +/* RGMII Rx Clock delay value change with board lay-out */ static u8
> > > +rgmii_rx_clk_delay = RGMII_RX_CLK_DELAY_1_1_NS;
> >
> > Doesn't this stop you from having a board with two PHYs with different
> >
; Thanks,
> > Raju
> >
> > -Original Message-
> > From: Andrew Lunn [mailto:and...@lunn.ch]
> > Sent: Tuesday, July 26, 2016 6:14 PM
> > To: Raju Lakkaraju
> > Cc: netdev@vger.kernel.org; f.faine...@gmail.com; Allan Nielsen
> > Subject: Re: Mic
> > +/* RGMII Rx Clock delay value change with board lay-out */ static u8
> > +rgmii_rx_clk_delay = RGMII_RX_CLK_DELAY_1_1_NS;
>
> Doesn't this stop you from having a board with two PHYs with different
> layouts? You should be getting this value from the device tree.
>
> Raju: As of now, RGMII
t: Tuesday, July 26, 2016 6:14 PM
> To: Raju Lakkaraju
> Cc: netdev@vger.kernel.org; f.faine...@gmail.com; Allan Nielsen
> Subject: Re: Microsemi VSC 8531/41 PHY Driver
>
> EXTERNAL EMAIL
>
>
> > +/* RGMII Rx Clock delay value change with board lay-out */ st
July 26, 2016 6:14 PM
> To: Raju Lakkaraju
> Cc: netdev@vger.kernel.org; f.faine...@gmail.com; Allan Nielsen
> Subject: Re: Microsemi VSC 8531/41 PHY Driver
>
> EXTERNAL EMAIL
>
>
>> +/* RGMII Rx Clock delay value change with board lay-out */ static u8
>> +rgmii_rx_clk_
Subject: Re: Microsemi VSC 8531/41 PHY Driver
EXTERNAL EMAIL
> +/* RGMII Rx Clock delay value change with board lay-out */ static u8
> +rgmii_rx_clk_delay = RGMII_RX_CLK_DELAY_1_1_NS;
Doesn't this stop you from having a board with two PHYs with different layouts?
You should be getting
> +/* RGMII Rx Clock delay value change with board lay-out */
> +static u8 rgmii_rx_clk_delay = RGMII_RX_CLK_DELAY_1_1_NS;
Doesn't this stop you from having a board with two PHYs with different
layouts? You should be getting this value from the device tree.
> + phydev->supported = (SUPPORTED_
040 6686 0132
-Original Message-
From: Andrew Lunn [mailto:and...@lunn.ch]
Sent: Tuesday, July 26, 2016 5:26 PM
To: Raju Lakkaraju
Cc: netdev@vger.kernel.org; f.faine...@gmail.com; Allan Nielsen
Subject: Re: Microsemi VSC 8531/41 PHY Driver
EXTERNAL EMAIL
On Tue, Jul 26, 2016 at 09:4
On Tue, Jul 26, 2016 at 09:49:53AM +, Raju Lakkaraju wrote:
> Hello All,
>
> I would like to introduce myself as Nagaraju Lakkaraju (Raju), is working in
> Microsemi Communications India Pvt. Ltd (Formerly known as Vitesse
> Semiconductors Limited) - Hyderabad as Sr. Staff Engineer.
> I do w
12 matches
Mail list logo
