From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: Simon Glass <sjg@chromium.org>
Cc: kettenis@openbsd.org, u-boot@lists.denx.de,
jh80.chung@samsung.com, trini@konsulko.com, sven@svenpeter.dev,
marcan@marcan.st, bmeng.cn@gmail.com
Subject: Re: [PATCH 1/8] nvme: Split out PCI support
Date: Sat, 22 Jan 2022 13:47:53 +0100 (CET) [thread overview]
Message-ID: <d3cbcce6c097bf26@bloch.sibelius.xs4all.nl> (raw)
In-Reply-To: <CAPnjgZ1JvYh9yrHcMjBhbmN13v8v2aSdMSivqV8rzt9MfAXERQ@mail.gmail.com> (message from Simon Glass on Fri, 21 Jan 2022 18:40:11 -0700)
> From: Simon Glass <sjg@chromium.org>
> Date: Fri, 21 Jan 2022 18:40:11 -0700
>
> Hi Mark,
>
> On Fri, 14 Jan 2022 at 04:05, Mark Kettenis <kettenis@openbsd.org> wrote:
> >
> > Apple SoCs have an integrated NVMe controller that isn't connected
> > over a PCIe bus. In preparation for adding support for this NVMe
> > controller, split out the PCI support into its own file. This file
> > is selected through a new CONFIG_NVME_PCI Kconfig option, so do
> > a wholesale replacement of CONFIG_NVME with CONFIG_NVME_PCI.
> >
> > Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
> > ---
> > configs/clearfog_gt_8k_defconfig | 2 +-
> > configs/firefly-rk3399_defconfig | 2 +-
> > configs/khadas-vim3_android_ab_defconfig | 2 +-
> > configs/khadas-vim3_android_defconfig | 2 +-
> > configs/khadas-vim3_defconfig | 2 +-
> > configs/khadas-vim3l_android_ab_defconfig | 2 +-
> > configs/khadas-vim3l_android_defconfig | 2 +-
> > configs/khadas-vim3l_defconfig | 2 +-
> > configs/kontron_sl28_defconfig | 2 +-
> > configs/ls1012afrdm_qspi_defconfig | 2 +-
> > configs/ls1012afrdm_tfa_defconfig | 2 +-
> > .../ls1012afrwy_qspi_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1012afrwy_qspi_defconfig | 2 +-
> > configs/ls1012afrwy_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1012afrwy_tfa_defconfig | 2 +-
> > configs/ls1012aqds_qspi_defconfig | 2 +-
> > configs/ls1012aqds_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1012aqds_tfa_defconfig | 2 +-
> > configs/ls1012ardb_qspi_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1012ardb_qspi_defconfig | 2 +-
> > configs/ls1012ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1012ardb_tfa_defconfig | 2 +-
> > configs/ls1021aiot_qspi_defconfig | 2 +-
> > configs/ls1021aiot_sdcard_defconfig | 2 +-
> > configs/ls1021aqds_ddr4_nor_defconfig | 2 +-
> > configs/ls1021aqds_ddr4_nor_lpuart_defconfig | 2 +-
> > configs/ls1021aqds_nand_defconfig | 2 +-
> > configs/ls1021aqds_nor_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1021aqds_nor_defconfig | 2 +-
> > configs/ls1021aqds_nor_lpuart_defconfig | 2 +-
> > configs/ls1021aqds_qspi_defconfig | 2 +-
> > configs/ls1021aqds_sdcard_ifc_defconfig | 2 +-
> > configs/ls1021aqds_sdcard_qspi_defconfig | 2 +-
> > configs/ls1021atsn_qspi_defconfig | 2 +-
> > configs/ls1021atsn_sdcard_defconfig | 2 +-
> > configs/ls1021atwr_nor_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1021atwr_nor_defconfig | 2 +-
> > configs/ls1021atwr_nor_lpuart_defconfig | 2 +-
> > configs/ls1021atwr_qspi_defconfig | 2 +-
> > configs/ls1021atwr_sdcard_ifc_defconfig | 2 +-
> > configs/ls1021atwr_sdcard_qspi_defconfig | 2 +-
> > configs/ls1028aqds_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1028aqds_tfa_defconfig | 2 +-
> > configs/ls1028aqds_tfa_lpuart_defconfig | 2 +-
> > configs/ls1028ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1028ardb_tfa_defconfig | 2 +-
> > configs/ls1043aqds_defconfig | 2 +-
> > configs/ls1043aqds_lpuart_defconfig | 2 +-
> > configs/ls1043aqds_nand_defconfig | 2 +-
> > configs/ls1043aqds_nor_ddr3_defconfig | 2 +-
> > configs/ls1043aqds_qspi_defconfig | 2 +-
> > configs/ls1043aqds_sdcard_ifc_defconfig | 2 +-
> > configs/ls1043aqds_sdcard_qspi_defconfig | 2 +-
> > configs/ls1043aqds_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1043aqds_tfa_defconfig | 2 +-
> > configs/ls1043ardb_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1043ardb_defconfig | 2 +-
> > configs/ls1043ardb_nand_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1043ardb_nand_defconfig | 2 +-
> > configs/ls1043ardb_sdcard_defconfig | 2 +-
> > configs/ls1043ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1043ardb_tfa_defconfig | 2 +-
> > configs/ls1046afrwy_tfa_defconfig | 2 +-
> > configs/ls1046aqds_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1046aqds_defconfig | 2 +-
> > configs/ls1046aqds_lpuart_defconfig | 2 +-
> > configs/ls1046aqds_nand_defconfig | 2 +-
> > configs/ls1046aqds_qspi_defconfig | 2 +-
> > configs/ls1046aqds_sdcard_ifc_defconfig | 2 +-
> > configs/ls1046aqds_sdcard_qspi_defconfig | 2 +-
> > configs/ls1046aqds_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1046aqds_tfa_defconfig | 2 +-
> > configs/ls1046ardb_emmc_defconfig | 2 +-
> > configs/ls1046ardb_qspi_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1046ardb_qspi_defconfig | 2 +-
> > configs/ls1046ardb_qspi_spl_defconfig | 2 +-
> > configs/ls1046ardb_sdcard_defconfig | 2 +-
> > configs/ls1046ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1046ardb_tfa_defconfig | 2 +-
> > configs/ls1088aqds_defconfig | 2 +-
> > configs/ls1088aqds_qspi_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1088aqds_qspi_defconfig | 2 +-
> > configs/ls1088aqds_sdcard_ifc_defconfig | 2 +-
> > configs/ls1088aqds_sdcard_qspi_defconfig | 2 +-
> > configs/ls1088aqds_tfa_defconfig | 2 +-
> > configs/ls1088ardb_qspi_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1088ardb_qspi_defconfig | 2 +-
> > configs/ls1088ardb_sdcard_qspi_defconfig | 2 +-
> > configs/ls1088ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls1088ardb_tfa_defconfig | 2 +-
> > configs/ls2080aqds_SECURE_BOOT_defconfig | 2 +-
> > configs/ls2080aqds_defconfig | 2 +-
> > configs/ls2080aqds_nand_defconfig | 2 +-
> > configs/ls2080aqds_qspi_defconfig | 2 +-
> > configs/ls2080aqds_sdcard_defconfig | 2 +-
> > configs/ls2080ardb_SECURE_BOOT_defconfig | 2 +-
> > configs/ls2080ardb_defconfig | 2 +-
> > configs/ls2080ardb_nand_defconfig | 2 +-
> > configs/ls2081ardb_defconfig | 2 +-
> > configs/ls2088aqds_tfa_defconfig | 2 +-
> > configs/ls2088ardb_qspi_SECURE_BOOT_defconfig | 2 +-
> > configs/ls2088ardb_qspi_defconfig | 2 +-
> > configs/ls2088ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/ls2088ardb_tfa_defconfig | 2 +-
> > configs/lx2160aqds_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/lx2160aqds_tfa_defconfig | 2 +-
> > configs/lx2160ardb_tfa_SECURE_BOOT_defconfig | 2 +-
> > configs/lx2160ardb_tfa_defconfig | 2 +-
> > configs/lx2160ardb_tfa_stmm_defconfig | 2 +-
> > configs/mvebu_crb_cn9130_defconfig | 2 +-
> > configs/mvebu_db_armada8k_defconfig | 2 +-
> > configs/mvebu_db_cn9130_defconfig | 2 +-
> > configs/mvebu_espressobin-88f3720_defconfig | 2 +-
> > configs/mvebu_mcbin-88f8040_defconfig | 2 +-
> > configs/mvebu_puzzle-m801-88f8040_defconfig | 2 +-
> > configs/nanopc-t4-rk3399_defconfig | 2 +-
> > configs/octeontx2_96xx_defconfig | 2 +-
> > configs/octeontx_81xx_defconfig | 2 +-
> > configs/octeontx_83xx_defconfig | 2 +-
> > configs/p3450-0000_defconfig | 2 +-
> > configs/pinebook-pro-rk3399_defconfig | 2 +-
> > configs/qemu-x86_64_defconfig | 2 +-
> > configs/qemu-x86_defconfig | 2 +-
> > configs/qemu_arm64_defconfig | 2 +-
> > configs/qemu_arm_defconfig | 2 +-
> > configs/rcar3_salvator-x_defconfig | 2 +-
> > configs/roc-pc-mezzanine-rk3399_defconfig | 2 +-
> > configs/rock-pi-4-rk3399_defconfig | 2 +-
> > configs/rock-pi-4c-rk3399_defconfig | 2 +-
> > configs/rock-pi-n10-rk3399pro_defconfig | 2 +-
> > configs/rock960-rk3399_defconfig | 2 +-
> > configs/rockpro64-rk3399_defconfig | 2 +-
> > configs/sandbox64_defconfig | 2 +-
> > configs/sandbox_defconfig | 2 +-
> > configs/sandbox_flattree_defconfig | 2 +-
> > configs/sandbox_noinst_defconfig | 2 +-
> > configs/sandbox_spl_defconfig | 2 +-
> > configs/sifive_unmatched_defconfig | 2 +-
> > configs/synquacer_developerbox_defconfig | 2 +-
> > configs/turris_mox_defconfig | 2 +-
> > configs/turris_omnia_defconfig | 2 +-
> > doc/develop/driver-model/nvme.rst | 1 +
> > drivers/nvme/Kconfig | 10 +++-
> > drivers/nvme/Makefile | 1 +
> > drivers/nvme/nvme.c | 38 ++------------
> > drivers/nvme/nvme.h | 3 ++
> > drivers/nvme/nvme_pci.c | 49 +++++++++++++++++++
> > 147 files changed, 207 insertions(+), 177 deletions(-)
> > create mode 100644 drivers/nvme/nvme_pci.c
> >
>
> Shouldn't the vendor addition go in another patch?
I don't think that would make a ton of sense. It is intricately
linked to splitting out the PCI support, as the current code grovels
into pci_child_plat struct in a place that needs to be bus-agnostic.
> Otherwise:
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Tested on: Macbook Air M1
> Tested-by: Simon Glass <sjg@chromium.org>
next prev parent reply other threads:[~2022-01-22 12:48 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-14 11:04 [PATCH 0/8] Apple M1 NVMe storage support Mark Kettenis
2022-01-14 11:04 ` [PATCH 1/8] nvme: Split out PCI support Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-22 12:47 ` Mark Kettenis [this message]
2022-01-22 13:18 ` Simon Glass
2022-01-22 14:57 ` Mark Kettenis
2022-01-14 11:04 ` [PATCH 2/8] mailbox: apple: Add driver for Apple IOP mailbox Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-22 13:54 ` Mark Kettenis
2022-01-22 17:17 ` Simon Glass
2022-01-14 11:04 ` [PATCH 3/8] arm: apple: Add RTKit support Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-22 13:59 ` Mark Kettenis
2022-01-22 17:17 ` Simon Glass
2022-01-22 19:31 ` Mark Kettenis
2022-01-14 11:04 ` [PATCH 4/8] nvme: Introduce driver ops Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-22 13:33 ` Mark Kettenis
2022-01-22 17:17 ` Simon Glass
2022-01-14 11:04 ` [PATCH 5/8] nvme: Add shutdown function Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-14 11:04 ` [PATCH 6/8] power: domain: apple: Add reset support Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-22 14:11 ` Mark Kettenis
2022-01-22 17:17 ` Simon Glass
2022-01-22 19:35 ` Mark Kettenis
2022-01-14 11:04 ` [PATCH 7/8] nvme: apple: Add driver for Apple NVMe storage controller Mark Kettenis
2022-01-22 1:40 ` Simon Glass
2022-01-22 14:45 ` Mark Kettenis
2022-01-22 17:17 ` Simon Glass
2022-01-22 17:41 ` Mark Kettenis
2022-01-22 18:28 ` Simon Glass
2022-01-14 11:04 ` [PATCH 8/8] configs: apple: Add NVMe boot target Mark Kettenis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d3cbcce6c097bf26@bloch.sibelius.xs4all.nl \
--to=mark.kettenis@xs4all.nl \
--cc=bmeng.cn@gmail.com \
--cc=jh80.chung@samsung.com \
--cc=kettenis@openbsd.org \
--cc=marcan@marcan.st \
--cc=sjg@chromium.org \
--cc=sven@svenpeter.dev \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.