Hello Dinesh,
On 04.08.25 06:20, Maniyam, Dinesh wrote:
Hi @Heiko Schocher
-----Original Message-----
From: Maniyam, Dinesh
Sent: Monday, 28 July 2025 3:01 pm
To: u-boot@lists.denx.de; Heiko Schocher <h...@denx.de>
Cc: Marek <ma...@denx.de>; Simon <simon.k.r.goldschm...@gmail.com>;
Simon Glass <s...@chromium.org>; Tom Rini <tr...@konsulko.com>; Dario
Binacchi <dario.binac...@amarulasolutions.com>; Ilias Apalodimas
<ilias.apalodi...@linaro.org>; Heinrich Schuchardt <xypron.g...@gmx.de>;
Jerome Forissier <jerome.foriss...@linaro.org>; Mattijs Korpershoek
<mkorpersh...@baylibre.com>; Ibai Erkiaga <ibai.erkiaga-elo...@amd.com>;
Michal Simek <michal.si...@amd.com>; Dmitry Rokosov
<ddroko...@salutedevices.com>; Jonas Karlman <jo...@kwiboo.se>; Sebastian
Reichel <sebastian.reic...@collabora.com>; Chee, Tien Fong; Hea, Kok Kiang; Ng,
Boon Khai; Yuslaimi, Alif Zakuan; Lim, Jit Loon; Maniyam, Dinesh
<dinesh.mani...@intel.com>
Subject: RE: [PATCH v7 00/12] Add Synopsys MIPI I3C Driver support
Hi @Heiko Schocher
-----Original Message-----
From: Maniyam, Dinesh <dinesh.mani...@altera.com>
Sent: Monday, 28 July 2025 2:56 pm
To: u-boot@lists.denx.de
Cc: Marek <ma...@denx.de>; Simon <simon.k.r.goldschm...@gmail.com>;
Simon Glass <s...@chromium.org>; Tom Rini <tr...@konsulko.com>; Heiko
Schocher <h...@denx.de>; Dario Binacchi
<dario.binac...@amarulasolutions.com>; Ilias Apalodimas
<ilias.apalodi...@linaro.org>; Heinrich Schuchardt
<xypron.g...@gmx.de>; Jerome Forissier <jerome.foriss...@linaro.org>;
Mattijs Korpershoek <mkorpersh...@baylibre.com>; Ibai Erkiaga
<ibai.erkiaga-elo...@amd.com>; Michal Simek <michal.si...@amd.com>;
Dmitry Rokosov <ddroko...@salutedevices.com>; Jonas Karlman
<jo...@kwiboo.se>; Sebastian Reichel
<sebastian.reic...@collabora.com>; Chee, Tien Fong
<tien.fong.c...@altera.com>; Hea, Kok Kiang
<kok.kiang....@altera.com>; Maniyam, Dinesh
<dinesh.mani...@altera.com>; Ng, Boon Khai <boon.khai...@altera.com>;
Yuslaimi, Alif Zakuan <alif.zakuan.yusla...@altera.com>; Lim, Jit Loon
<jit.loon....@altera.com>; Dinesh Maniyam <dinesh.mani...@intel.com>
Subject: [PATCH v7 00/12] Add Synopsys MIPI I3C Driver support
From: Dinesh Maniyam <dinesh.mani...@intel.com>
This patchset add Synopsys MIPI I3C Driver support for Intel Agilex5 devices.
The i3c driver is leveraged from the master/dw-i3c-master.c,
i3c/device.c and i3c/master.c Linux version 6.6.37 LTS And few header
files included to be part of the migration; i3c/internals.h,
include/linux/i3c/ccc.h, include/linux/i3c/device.h and
include/linux/i3c/master.h.
Additional i3c uclass driver and command files added to support i3c
read and write in U-Boot.
Patch status:
Detail changelog can find in commit message.
v7->v6:
--------
- Remove extra variable in struct udevice
- reuse the existing dev_ofnode to get the node in master.c
v6->v5:
--------
- Resolved documentation warning in include/i3c.h
- Move declarations to the top in cmd/i3c.c functions
v5->v4:
--------
- Add header file to doc/api/i3c
- Rephrase function description for i3c.h
- Separated subcommand as a function for cmd/i3c.c
- Used memory address to transfer msgs from i3c
- Rephrase description in cmd/i3c.rst file
- Replaced int with unsigned for positive variables.
v4->v3:
--------
Resolve ci test:
- Add i3c node in index.rst
- Formating error in i3c.rst
- Resolve cpu_relax undefined for sandbox
v3->v2:
--------
- Squash commit 10 to commit 9
- Resolve index for new files to pass the checkpatch
Commit: sandbox_defconfig: Enable configs for sandbox i3c
Commit: agilex5_defconfig: Enable i3c configs for agilex5
- use savedefconfig
drivers: Enabled Kconfig and Makefile for i3c support
- Add maintainer for i3c
drivers: i3c: Enabled Kconfig and Makefile for DWI3C
- Add empty lines for if/endif
drivers: i3c: Add i3c sandbox simple test.
- Removed DECLARE_GLOBAL_DATA_PTR
v1->v2:
--------
Reorder commits.
Added commits:
- drivers: i3c: Add i3c sandbox simple test.
- test: cmd: Add simple test for i3c
- configs: sandbox_defconfig: Enable configs for sandbox i3c
- configs: agilex5_defconfig: Enable i3c configs for agilex5
History:
--------
[v1]:
https://patchwork.ozlabs.org/project/uboot/cover/20250218025705.50051-
1-
dinesh.mani...@intel.com/
[v2]:
https://patchwork.ozlabs.org/project/uboot/cover/20250314040902.43621-
1-
dinesh.mani...@altera.com/
[v3]:
https://patchwork.ozlabs.org/project/uboot/cover/20250324083704.26259-
1-
dinesh.mani...@altera.com/
[v4]:
https://patchwork.ozlabs.org/project/uboot/cover/20250417021840.6212-
1-dinesh.mani...@altera.com/
[v5]:
https://patchwork.ozlabs.org/project/uboot/cover/20250513101913.30880-
1-
dinesh.mani...@altera.com/
[v6]:
https://patchwork.ozlabs.org/project/uboot/cover/20250513101913.30880-
1-
dinesh.mani...@altera.com/
Dinesh Maniyam (12):
drivers: i3c: Add new i3c uclass id
drivers: i3c: Add driver for MIPI DWI3C
drivers: i3c: Add i3c uclass driver.
drivers: Enabled Kconfig and Makefile for i3c support
drivers: i3c: Enabled Kconfig and Makefile for DWI3C
drivers: i3c: Add i3c sandbox simple test.
drivers: i3c: master: Enable probe i3c without slave device
i3c: master: dw-i3c-master: Fix OD_TIMING for spike filter
cmd: Add i3c command support.
test: cmd: Add simple test for i3c
configs: sandbox_defconfig: Enable configs for sandbox i3c
configs: agilex5_defconfig: Enable i3c configs for agilex5
MAINTAINERS | 7 +
arch/sandbox/dts/test.dts | 8 +
cmd/Kconfig | 6 +
cmd/Makefile | 1 +
cmd/i3c.c | 271 ++++
configs/sandbox_defconfig | 4 +
configs/socfpga_agilex5_defconfig | 3 +
doc/api/i3c.rst | 8 +
doc/api/index.rst | 1 +
doc/usage/cmd/i3c.rst | 146 ++
doc/usage/index.rst | 1 +
drivers/Kconfig | 2 +
drivers/Makefile | 1 +
drivers/i3c/Kconfig | 27 +
drivers/i3c/Makefile | 5 +
drivers/i3c/device.c | 262 ++++
drivers/i3c/i3c-uclass.c | 38 +
drivers/i3c/internals.h | 26 +
drivers/i3c/master.c | 2066 ++++++++++++++++++++++++++++
drivers/i3c/master/Kconfig | 11 +
drivers/i3c/master/Makefile | 3 +
drivers/i3c/master/dw-i3c-master.c | 1062 ++++++++++++++
drivers/i3c/sandbox_i3c.c | 56 +
include/dm/uclass-id.h | 1 +
include/dw-i3c.h | 252 ++++
include/i3c.h | 93 ++
include/linux/i3c/ccc.h | 385 ++++++
include/linux/i3c/device.h | 286 ++++
include/linux/i3c/master.h | 698 ++++++++++
test/cmd/Makefile | 1 +
test/cmd/i3c.c | 50 +
test/dm/Makefile | 1 +
test/dm/i3c.c | 34 +
33 files changed, 5816 insertions(+)
create mode 100644 cmd/i3c.c
create mode 100644 doc/api/i3c.rst
create mode 100644 doc/usage/cmd/i3c.rst create mode 100644
drivers/i3c/Kconfig create mode 100644 drivers/i3c/Makefile create
mode
100644 drivers/i3c/device.c create mode 100644
drivers/i3c/i3c-uclass.c create mode 100644 drivers/i3c/internals.h
create mode 100644 drivers/i3c/master.c create mode 100644
drivers/i3c/master/Kconfig create mode 100644
drivers/i3c/master/Makefile create mode 100644
drivers/i3c/master/dw-i3c- master.c create mode 100644
drivers/i3c/sandbox_i3c.c create mode 100644 include/dw-i3c.h create
mode 100644 include/i3c.h create mode 100644 include/linux/i3c/ccc.h
create mode 100644 include/linux/i3c/device.h create mode 100644
include/linux/i3c/master.h create mode 100644 test/cmd/i3c.c create
mode 100644 test/dm/i3c.c
--
2.35.3
As discussed earlier, I have made changes to pass the ci.
CI is passing now for me.
https://github.com/u-boot/u-boot/pull/802/checks?check_run_id=46661813275
Thanks
Dinesh
Did you manage to pull in my patchseries, please let me know the status
Please fix some small nitpicks from me and askar... than I think, it
can go in from my side
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Johanna Denk, Tabea Lutz
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: h...@denx.de
