linux-kernel - Re: [PATCH 0/4] Add Microchip I3C controller

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]

Message-ID: <d3daaab9-71f5-49e6-b3de-b05339fa7adf@microchip.com>
Date: Wed, 10 Sep 2025 06:14:21 +0000
From: <Durai.ManickamKR@...rochip.com>
To: <conor@...nel.org>
CC: <linux-i3c@...ts.infradead.org>, <devicetree@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <alexandre.belloni@...tlin.com>,
	<Frank.Li@....com>, <robh@...nel.org>, <krzk+dt@...nel.org>,
	<conor+dt@...nel.org>, <Balamanikandan.Gunasundar@...rochip.com>,
	<Nicolas.Ferre@...rochip.com>
Subject: Re: [PATCH 0/4] Add Microchip I3C controller

On 10/09/25 01:16, Conor Dooley wrote:
> On Tue, Sep 09, 2025 at 04:43:29PM +0530, Durai Manickam KR wrote:
>> Add support for microchip SAMA7D65 I3C HCI master only IP. This
>> hardware is an instance of the MIPI I3C HCI Controller implementing
>> version 1.0 specification. This driver adds platform-specific
>> support for SAMA7D65 SoC.
>>
>> I3C in master mode supports up to 12.5MHz, SDR mode data transfer in
>> mixed bus mode (I2C and I3C target devices on same i3c bus). It also
>> supports IBI mechanism.
>>      
>> Features tested and supported :
>> 	Standard CCC commands.
>>         	I3C SDR mode private transfers in PIO mode.
>>         	I2C transfers in PIO mode.
>>         	Pure bus mode and mixed bus mode.
>>
>> Testing done:
>> With this patch we are able to fully configure the lsm6dso I3C slave
>> device. Unlike I2C, I hope there is no linux utility to check the
>> various transactions supported in the host controller. These features
>> will be added later on this driver after testing (probably with I3C
>> analyser KIT).
>>
>> Durai Manickam KR (4):
>>    clk: at91: sama7d65: add peripheral clock for I3C
>>    i3c: master: add Microchip SAMA7D65 I3C HCI master driver
>>    ARM: configs: at91: sama7: Add SAMA7D65 I3C HCI master
>>    ARM: dts: microchip: add I3C controller
>>
>>   arch/arm/boot/dts/microchip/sama7d65.dtsi    |   12 +
>>   arch/arm/configs/sama7_defconfig             |    2 +
>>   drivers/clk/at91/sama7d65.c                  |    5 +-
>>   drivers/i3c/master/Kconfig                   |   14 +
>>   drivers/i3c/master/Makefile                  |    1 +
>>   drivers/i3c/master/sama7d65-i3c-hci-master.c | 2705 ++++++++++++++++++
>>   6 files changed, 2737 insertions(+), 2 deletions(-)
>>   create mode 100644 drivers/i3c/master/sama7d65-i3c-hci-master.c

Hi Conor,

Was planning to sent along with the review comments received for the i3c 
driver. Will add this and send in patch v2 series.

> Where is your dt-binding?