IronPort-SDR: fUkcPZPrJJZn8G8UKtM5Ng0a7pjArBarDEblODBTaZaGzVU+HpnZuodRu7jcH2GvZV8seeU5kv
 H/T6euq0LrhJO+vxJcbM6Xf3WBTSC8pw2HAhKyqhJwcxy9EV8UpB/sHjD+gL2PTCracEGW+/Sm
 At3iY4G22+znlUPY0emOpvB7hzCi57Sco12iK+RQFbw+LOwaVaRkd78noReFfwRQr2cArdYzs5
 U7jBWNUWm3sRgWcA0fPWPxkVHY8EOj7GujSWZ4wYbqMivhDoz4LJ+MYrCKfviK+Ld9qMi1Z0dM
 9ws3oExXxIgD35L4IZebY6Jj
Date:   Tue, 16 Nov 2021 23:05:54 +0100
From:   Horatiu Vultur <horatiu.vultur@microchip.com>
To:     Alexandre Belloni <alexandre.belloni@bootlin.com>
CC:     Vinod Koul <vkoul@kernel.org>, <kishon@ti.com>,
        <robh+dt@kernel.org>, <andrew@lunn.ch>, <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: <20211116220554.d3amfihujwua2sff@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>
 <20211116115228.5qn4o3gkh26674rw@soft-dev3-1.localhost>
 <YZP0lPN+TCtBN8ic@piout.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
In-Reply-To: <YZP0lPN+TCtBN8ic@piout.net>
Precedence: bulk

The 11/16/2021 19:12, Alexandre Belloni wrote:
> 
> Hello,
> 
> On 16/11/2021 12:52:28+0100, Horatiu Vultur wrote:
> > The 11/16/2021 17:13, Vinod Koul wrote:
> > >
> > > 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.
> >
> 
> It is fine if it doesn't work at runtime with only one branch, both will
> get merged in linux-next and you can test with that if necessary. I'll
> let both series go through separate trees.

Ah.. sorry for all the confusion. I though they need to go through the
same tree.

So Vinod, if you think that the patch series is OK please take it in
your tree, otherwise let me know what comments you have.

> 
> 
> --
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com

-- 
/Horatiu