IronPort-SDR: 6BLFG/kzWU+9TdFx51b+S79rC0vLPNmp0s7BN1WwaQMcfGPweK240iWbIWoGEHUtcZ8NUHtDdS
 LZJIKZ+qbb0wwbeJ0JG8iSDSiE326I48pPShCIlEh4RK34sJuGa+X6kfO9OLKRyrt2d18S1Qhp
 FjnJZ1PJE8SQbamGwfd/fR4enYHjf13fK2BpmRscTe0YMnb4T/f1aR7DpR24aLLjPmbY37BSTV
 gmK+hZ3Sunq/q6ZhAc3PzKz8RNCDQnKT53ArVXpLSwzAYgj6TlKRHi5I8CHGnpRWXGKt4Sj4rB
 bejErPEcLAsEW7rfsyZwQkKM
Date:   Tue, 16 Nov 2021 12:52:28 +0100
From:   Horatiu Vultur <horatiu.vultur@microchip.com>
To:     Vinod Koul <vkoul@kernel.org>
CC:     <kishon@ti.com>, <robh+dt@kernel.org>, <andrew@lunn.ch>,
        <alexandre.belloni@bootlin.com>, <kuba@kernel.org>,
        <linux-phy@lists.infradead.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH resend v4 0/3] phy: Add driver for lan966x Serdes driver
Message-ID: <20211116115228.5qn4o3gkh26674rw@soft-dev3-1.localhost>
References: <20211116100818.1615762-1-horatiu.vultur@microchip.com>
 <20211116102151.k3evnn4f2rwsi5nk@soft-dev3-1.localhost>
 <YZOLyQPsgoOYJZch@matsya>
 <20211116105601.fauvfv2wzlmcf6jc@soft-dev3-1.localhost>
 <YZOZZv3NPq2C9NXu@matsya>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
In-Reply-To: <YZOZZv3NPq2C9NXu@matsya>
Precedence: bulk

The 11/16/2021 17:13, Vinod Koul wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 16-11-21, 11:56, Horatiu Vultur wrote:
> > The 11/16/2021 16:15, Vinod Koul wrote:
> > >
> > > On 16-11-21, 11:21, Horatiu Vultur wrote:
> > > > The 11/16/2021 11:08, Horatiu Vultur wrote:
> > > >
> > > > Hi Kison, Vinod,
> > > >
> > > > Can you let me know if you have more comments to this patch series?
> > > > Otherwise can you ack on it? Because I would like to have the patches
> > > > merged via netdev if that is OK for you.
> > >
> > > Any reason for merge thru netdev, there is no dependency and now with
> > > merge window closed, I can pick this up..
> >
> > Because I would like to send some patches for the lan966x network driver.
> > And these network patches depend on the serdes driver.
> 
> There cant be a compile time dependency... the network driver can use
> the phy apis.. I dont think it is required here... Did I miss something
> obvious?

You are right, there is no compile time dependency and the network
driver can use the PHY APIs.

But at runtime the network driver will need to configure the serdes
using the PHY API to be able to work properly.

> 
> --
> ~Vinod

-- 
/Horatiu