netdev - Re: [RFC PATCH net-next 0/6] Add support for OPEN Alliance 10BASE-T1x MACPHY Serial Interface

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Mon, 18 Sep 2023 06:16:58 +0000
From: <Parthiban.Veerasooran@...rochip.com>
To: <andrew@...n.ch>, <alexandru.tachici@...log.com>, <ada@...rsis.com>
CC: <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>,
	<pabeni@...hat.com>, <robh+dt@...nel.org>,
	<krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>, <corbet@....net>,
	<Steen.Hegelund@...rochip.com>, <rdunlap@...radead.org>, <horms@...nel.org>,
	<casper.casan@...il.com>, <netdev@...r.kernel.org>,
	<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
	<linux-doc@...r.kernel.org>, <Horatiu.Vultur@...rochip.com>,
	<Woojung.Huh@...rochip.com>, <Nicolas.Ferre@...rochip.com>,
	<UNGLinuxDriver@...rochip.com>, <Thorsten.Kummermehr@...rochip.com>,
	<alexandru.tachici@...log.com>
Subject: Re: [RFC PATCH net-next 0/6] Add support for OPEN Alliance 10BASE-T1x
 MACPHY Serial Interface

Hi Andrew,

Sure. Thanks for the suggestion. I hope my previous reply for 
Alexander's email clarifies the below comment as well.

It would be nice if Alexandru Tachici gives us more clarity on their 
device and driver.

Best Regards,
Parthiban V

On 15/09/23 7:52 pm, Andrew Lunn wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On Fri, Sep 15, 2023 at 03:56:59PM +0200, Alexander Dahl wrote:
>> Hello,
>>
>> this is interesting, by chance I just looked at a chip claiming
>> similar features today, which already has a driver in kernel: Analog
>> Devices ADIN1110.
> 
> Ah, interesting. I had no idea this driver/device is an OA TC6 device.
> 
> So ideally, we want the adin1110.c to also use the new framework, and
> remove the duplicate code. Parthiban, please look at the driver and
> make sure the APIs are such that Alexandru Tachici can swap his driver
> to using it. Having two implementations should help make it clear what
> is the same and what is different.
> 
>     Andrew
>