Re: [PATCH 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs

From: Suman Anna <s-anna@ti.com>
To: Nishanth Menon <nm@ti.com>
Cc: Jan Kiszka <jan.kiszka@siemens.com>,
	Grygorii Strashko <grygorii.strashko@ti.com>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Lokesh Vutla <lokeshvutla@ti.com>,
	Grzegorz Jaszczyk <grzegorz.jaszczyk@linaro.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<devicetree@vger.kernel.org>
Subject: Re: [PATCH 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs
Date: Fri, 28 May 2021 13:29:53 -0500	[thread overview]
Message-ID: <e0da118a-077b-2e04-1e75-1550bcfa3540@ti.com> (raw)
In-Reply-To: <20210528182400.gjegxcl3umqqf5gl@filth>

On 5/28/21 1:24 PM, Nishanth Menon wrote:
> On 17:47-20210514, Suman Anna wrote:
>> Hi Nishanth,
>>
>> The following series adds the base MDIO nodes representing the MDIO
>> sub-module present in each of the ICSSG subsystem. These build on
>> top of the foundation ICSSG nodes added in v5.13-rc1. Please see
>> the previous ICSSG foundation series for more details [1]. 
>>
>> The MDIO nodes would only be used on boards wired for supporting
>> the ICSSG Ethernet, so in general are expected to be disabled in
>> all boards where they are not pinned out.
>>
>> I have added these as "enabled" by default in the base dtsi files,
>> and have marked it disabled in all the existing board dts files
>> until we are ready to add in the ICSSG Ethernet functionality.
>>
>> Note that the bus_freq property throws a warning with W=2, but this
>> had been a legacy property defined a long back. The davinci-mdio
>> binding continues to use this.
>>
>> regards
>> Suman
>>
>> [1] https://patchwork.kernel.org/project/linux-arm-kernel/cover/20210114194805.8231-1-s-anna@ti.com/
> 
> Did you mean https://lore.kernel.org/linux-arm-kernel/20210304160712.8452-1-s-anna@ti.com/
> the series we actually did pick up?
> 

v2 is the series we picked up, but I meant to give the v1 cover-letter only for
the details portion. v2 cover-letter didn't have much details.

>>
>> Roger Quadros (1):
>>   arm64: dts: ti: k3-am65-main: Add ICSSG MDIO nodes
>>
>> Suman Anna (1):
>>   arm64: dts: ti: k3-j721e-main: Add ICSSG MDIO nodes
>>
> 
> OK, does'nt apply on my tree as I am picking up patches.. There are
> new cleanups and nodes I have picked up, so could be conflicting with
> those.
> 
> Please rebase and post based on:
> https://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git/log/?h=ti-k3-dts-next
> 
> I'd suggest you do that on monday OR base on monday's linux-next tag for
> the next post.
> 

Yeah ok.

regards
Suman