From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Alexander Usyskin <alexander.usyskin@intel.com>
Cc: "Miquel Raynal" <miquel.raynal@bootlin.com>,
"Richard Weinberger" <richard@nod.at>,
"Vignesh Raghavendra" <vigneshr@ti.com>,
"Lucas De Marchi" <lucas.demarchi@intel.com>,
"Thomas Hellström" <thomas.hellstrom@linux.intel.com>,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
"David Airlie" <airlied@gmail.com>,
"Simona Vetter" <simona@ffwll.ch>,
"Jani Nikula" <jani.nikula@linux.intel.com>,
"Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
"Tvrtko Ursulin" <tursulin@ursulin.net>,
"Karthik Poosa" <karthik.poosa@intel.com>,
"Reuven Abliyev" <reuven.abliyev@intel.com>,
"Oren Weil" <oren.jer.weil@intel.com>,
linux-mtd@lists.infradead.org, dri-devel@lists.freedesktop.org,
intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7 00/12] mtd: add driver for Intel discrete graphics
Date: Wed, 26 Mar 2025 15:22:57 -0400 [thread overview]
Message-ID: <Z-RUEVrst6WWzDUa@intel.com> (raw)
In-Reply-To: <20250326152623.3897204-1-alexander.usyskin@intel.com>
On Wed, Mar 26, 2025 at 05:26:11PM +0200, Alexander Usyskin wrote:
> Add driver for access to Intel discrete graphics card
> internal NVM device.
> Expose device on auxiliary bus by i915 and Xe drivers and
> provide mtd driver to register this device with MTD framework.
>
> This is a rewrite of "drm/i915/spi: spi access for discrete graphics"
> and "spi: add driver for Intel discrete graphics"
> series with connection to the Xe driver and splitting
> the spi driver part to separate module in mtd subsystem.
>
> This series intended to be pushed through drm-xe-next.
>
> V2: Replace dev_* prints with drm_* prints in drm (xe and i915) patches.
> Enable NVM device on Battlemage HW (xe driver patch)
> Fix overwrite register address (xe driver patch)
> Add Rodrigo's r-b
>
> V3: Use devm_pm_runtime_enable to simplify flow.
> Drop print in i915 unload that was accidentally set as error.
> Drop HAS_GSC_NVM macro in line with latest Xe changes.
> Add more Rodrigo's r-b and Miquel's ack.
>
> V4: Add patch that always creates mtd master device
> and adjust mtd-intel-dg power management to use this device.
>
> V5: Fix master device creation to accomodate for devices without
> partitions (create partitoned master in this case)
> Rebase over latest drm-xe-next
> Add ack's
> V6: Fix master device release (use rigth idr in release)
> Rebase over latest drm-xe-next
> Grammar and style fixes
>
> V7: Add patch with non-posted erase support (fix hang on BMG)
> Rebase over latest drm-xe-next
There are 2 things that I should had noticed earlier and mentioned to you.
I'm sorry for forgetting that.
1. Rebase on drm-tip branch of this repo: https://gitlab.freedesktop.org/drm/tip
We merge to drm-xe-next, but development and CI happens on drm-tip.
2. Please also send to intel-xe@lists.freedesktop.org Xe CI only runs on that
mailing list. intel-gfx is for i915.
No need to rebase or resend right now. Let's wait for a few reviews,
then we rebase and resend.
Thanks,
Rodrigo.
>
> Abliyev, Reuven (1):
> drm/xe/nvm: add support for non-posted erase
>
> Alexander Usyskin (11):
> mtd: core: always create master device
> mtd: add driver for intel graphics non-volatile memory device
> mtd: intel-dg: implement region enumeration
> mtd: intel-dg: implement access functions
> mtd: intel-dg: register with mtd
> mtd: intel-dg: align 64bit read and write
> mtd: intel-dg: wake card on operations
> drm/i915/nvm: add nvm device for discrete graphics
> drm/i915/nvm: add support for access mode
> drm/xe/nvm: add on-die non-volatile memory device
> drm/xe/nvm: add support for access mode
>
> MAINTAINERS | 7 +
> drivers/gpu/drm/i915/Makefile | 4 +
> drivers/gpu/drm/i915/i915_driver.c | 6 +
> drivers/gpu/drm/i915/i915_drv.h | 3 +
> drivers/gpu/drm/i915/i915_reg.h | 1 +
> drivers/gpu/drm/i915/intel_nvm.c | 115 ++++
> drivers/gpu/drm/i915/intel_nvm.h | 15 +
> drivers/gpu/drm/xe/Makefile | 1 +
> drivers/gpu/drm/xe/regs/xe_gsc_regs.h | 4 +
> drivers/gpu/drm/xe/xe_device.c | 5 +
> drivers/gpu/drm/xe/xe_device_types.h | 6 +
> drivers/gpu/drm/xe/xe_heci_gsc.c | 5 +-
> drivers/gpu/drm/xe/xe_nvm.c | 161 +++++
> drivers/gpu/drm/xe/xe_nvm.h | 15 +
> drivers/gpu/drm/xe/xe_pci.c | 6 +
> drivers/mtd/devices/Kconfig | 11 +
> drivers/mtd/devices/Makefile | 1 +
> drivers/mtd/devices/mtd_intel_dg.c | 884 ++++++++++++++++++++++++++
> drivers/mtd/mtdcore.c | 141 ++--
> drivers/mtd/mtdcore.h | 2 +-
> drivers/mtd/mtdpart.c | 17 +-
> include/linux/intel_dg_nvm_aux.h | 29 +
> 22 files changed, 1385 insertions(+), 54 deletions(-)
> create mode 100644 drivers/gpu/drm/i915/intel_nvm.c
> create mode 100644 drivers/gpu/drm/i915/intel_nvm.h
> create mode 100644 drivers/gpu/drm/xe/xe_nvm.c
> create mode 100644 drivers/gpu/drm/xe/xe_nvm.h
> create mode 100644 drivers/mtd/devices/mtd_intel_dg.c
> create mode 100644 include/linux/intel_dg_nvm_aux.h
>
> --
> 2.43.0
>
WARNING: multiple messages have this Message-ID (diff)
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Alexander Usyskin <alexander.usyskin@intel.com>
Cc: "Miquel Raynal" <miquel.raynal@bootlin.com>,
"Richard Weinberger" <richard@nod.at>,
"Vignesh Raghavendra" <vigneshr@ti.com>,
"Lucas De Marchi" <lucas.demarchi@intel.com>,
"Thomas Hellström" <thomas.hellstrom@linux.intel.com>,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
"David Airlie" <airlied@gmail.com>,
"Simona Vetter" <simona@ffwll.ch>,
"Jani Nikula" <jani.nikula@linux.intel.com>,
"Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
"Tvrtko Ursulin" <tursulin@ursulin.net>,
"Karthik Poosa" <karthik.poosa@intel.com>,
"Reuven Abliyev" <reuven.abliyev@intel.com>,
"Oren Weil" <oren.jer.weil@intel.com>,
linux-mtd@lists.infradead.org, dri-devel@lists.freedesktop.org,
intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7 00/12] mtd: add driver for Intel discrete graphics
Date: Wed, 26 Mar 2025 15:22:57 -0400 [thread overview]
Message-ID: <Z-RUEVrst6WWzDUa@intel.com> (raw)
In-Reply-To: <20250326152623.3897204-1-alexander.usyskin@intel.com>
On Wed, Mar 26, 2025 at 05:26:11PM +0200, Alexander Usyskin wrote:
> Add driver for access to Intel discrete graphics card
> internal NVM device.
> Expose device on auxiliary bus by i915 and Xe drivers and
> provide mtd driver to register this device with MTD framework.
>
> This is a rewrite of "drm/i915/spi: spi access for discrete graphics"
> and "spi: add driver for Intel discrete graphics"
> series with connection to the Xe driver and splitting
> the spi driver part to separate module in mtd subsystem.
>
> This series intended to be pushed through drm-xe-next.
>
> V2: Replace dev_* prints with drm_* prints in drm (xe and i915) patches.
> Enable NVM device on Battlemage HW (xe driver patch)
> Fix overwrite register address (xe driver patch)
> Add Rodrigo's r-b
>
> V3: Use devm_pm_runtime_enable to simplify flow.
> Drop print in i915 unload that was accidentally set as error.
> Drop HAS_GSC_NVM macro in line with latest Xe changes.
> Add more Rodrigo's r-b and Miquel's ack.
>
> V4: Add patch that always creates mtd master device
> and adjust mtd-intel-dg power management to use this device.
>
> V5: Fix master device creation to accomodate for devices without
> partitions (create partitoned master in this case)
> Rebase over latest drm-xe-next
> Add ack's
> V6: Fix master device release (use rigth idr in release)
> Rebase over latest drm-xe-next
> Grammar and style fixes
>
> V7: Add patch with non-posted erase support (fix hang on BMG)
> Rebase over latest drm-xe-next
There are 2 things that I should had noticed earlier and mentioned to you.
I'm sorry for forgetting that.
1. Rebase on drm-tip branch of this repo: https://gitlab.freedesktop.org/drm/tip
We merge to drm-xe-next, but development and CI happens on drm-tip.
2. Please also send to intel-xe@lists.freedesktop.org Xe CI only runs on that
mailing list. intel-gfx is for i915.
No need to rebase or resend right now. Let's wait for a few reviews,
then we rebase and resend.
Thanks,
Rodrigo.
>
> Abliyev, Reuven (1):
> drm/xe/nvm: add support for non-posted erase
>
> Alexander Usyskin (11):
> mtd: core: always create master device
> mtd: add driver for intel graphics non-volatile memory device
> mtd: intel-dg: implement region enumeration
> mtd: intel-dg: implement access functions
> mtd: intel-dg: register with mtd
> mtd: intel-dg: align 64bit read and write
> mtd: intel-dg: wake card on operations
> drm/i915/nvm: add nvm device for discrete graphics
> drm/i915/nvm: add support for access mode
> drm/xe/nvm: add on-die non-volatile memory device
> drm/xe/nvm: add support for access mode
>
> MAINTAINERS | 7 +
> drivers/gpu/drm/i915/Makefile | 4 +
> drivers/gpu/drm/i915/i915_driver.c | 6 +
> drivers/gpu/drm/i915/i915_drv.h | 3 +
> drivers/gpu/drm/i915/i915_reg.h | 1 +
> drivers/gpu/drm/i915/intel_nvm.c | 115 ++++
> drivers/gpu/drm/i915/intel_nvm.h | 15 +
> drivers/gpu/drm/xe/Makefile | 1 +
> drivers/gpu/drm/xe/regs/xe_gsc_regs.h | 4 +
> drivers/gpu/drm/xe/xe_device.c | 5 +
> drivers/gpu/drm/xe/xe_device_types.h | 6 +
> drivers/gpu/drm/xe/xe_heci_gsc.c | 5 +-
> drivers/gpu/drm/xe/xe_nvm.c | 161 +++++
> drivers/gpu/drm/xe/xe_nvm.h | 15 +
> drivers/gpu/drm/xe/xe_pci.c | 6 +
> drivers/mtd/devices/Kconfig | 11 +
> drivers/mtd/devices/Makefile | 1 +
> drivers/mtd/devices/mtd_intel_dg.c | 884 ++++++++++++++++++++++++++
> drivers/mtd/mtdcore.c | 141 ++--
> drivers/mtd/mtdcore.h | 2 +-
> drivers/mtd/mtdpart.c | 17 +-
> include/linux/intel_dg_nvm_aux.h | 29 +
> 22 files changed, 1385 insertions(+), 54 deletions(-)
> create mode 100644 drivers/gpu/drm/i915/intel_nvm.c
> create mode 100644 drivers/gpu/drm/i915/intel_nvm.h
> create mode 100644 drivers/gpu/drm/xe/xe_nvm.c
> create mode 100644 drivers/gpu/drm/xe/xe_nvm.h
> create mode 100644 drivers/mtd/devices/mtd_intel_dg.c
> create mode 100644 include/linux/intel_dg_nvm_aux.h
>
> --
> 2.43.0
>
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2025-03-26 19:23 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-26 15:26 [PATCH v7 00/12] mtd: add driver for Intel discrete graphics Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 01/12] mtd: core: always create master device Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-04-01 15:00 ` Miquel Raynal
2025-04-01 15:00 ` Miquel Raynal
2025-04-07 12:47 ` Usyskin, Alexander
2025-04-07 12:47 ` Usyskin, Alexander
2025-04-09 9:59 ` Miquel Raynal
2025-04-09 9:59 ` Miquel Raynal
2025-04-10 9:44 ` Usyskin, Alexander
2025-04-10 9:44 ` Usyskin, Alexander
2025-04-10 14:12 ` Usyskin, Alexander
2025-04-10 14:12 ` Usyskin, Alexander
2025-04-29 9:23 ` Miquel Raynal
2025-04-29 9:23 ` Miquel Raynal
2025-05-06 11:07 ` Usyskin, Alexander
2025-05-06 11:07 ` Usyskin, Alexander
2025-05-12 8:25 ` Miquel Raynal
2025-05-12 8:25 ` Miquel Raynal
2025-03-26 15:26 ` [PATCH v7 02/12] mtd: add driver for intel graphics non-volatile memory device Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 03/12] mtd: intel-dg: implement region enumeration Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 04/12] mtd: intel-dg: implement access functions Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 05/12] mtd: intel-dg: register with mtd Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 06/12] mtd: intel-dg: align 64bit read and write Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 07/12] mtd: intel-dg: wake card on operations Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 08/12] drm/i915/nvm: add nvm device for discrete graphics Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 09/12] drm/i915/nvm: add support for access mode Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 10/12] drm/xe/nvm: add on-die non-volatile memory device Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 11/12] drm/xe/nvm: add support for access mode Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 15:26 ` [PATCH v7 12/12] drm/xe/nvm: add support for non-posted erase Alexander Usyskin
2025-03-26 15:26 ` Alexander Usyskin
2025-03-26 16:09 ` ✗ Fi.CI.CHECKPATCH: warning for mtd: add driver for Intel discrete graphics (rev7) Patchwork
2025-03-26 16:09 ` ✗ Fi.CI.SPARSE: " Patchwork
2025-03-26 16:32 ` ✓ i915.CI.BAT: success " Patchwork
2025-03-26 18:55 ` ✗ i915.CI.Full: failure " Patchwork
2025-03-26 19:22 ` Rodrigo Vivi [this message]
2025-03-26 19:22 ` [PATCH v7 00/12] mtd: add driver for Intel discrete graphics Rodrigo Vivi
2025-04-09 9:33 ` Usyskin, Alexander
2025-04-09 9:33 ` Usyskin, Alexander
2025-04-10 14:30 ` ✗ Fi.CI.BUILD: failure for mtd: add driver for Intel discrete graphics (rev8) Patchwork
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=Z-RUEVrst6WWzDUa@intel.com \
--to=rodrigo.vivi@intel.com \
--cc=airlied@gmail.com \
--cc=alexander.usyskin@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@linux.intel.com \
--cc=joonas.lahtinen@linux.intel.com \
--cc=karthik.poosa@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=lucas.demarchi@intel.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=miquel.raynal@bootlin.com \
--cc=mripard@kernel.org \
--cc=oren.jer.weil@intel.com \
--cc=reuven.abliyev@intel.com \
--cc=richard@nod.at \
--cc=simona@ffwll.ch \
--cc=thomas.hellstrom@linux.intel.com \
--cc=tursulin@ursulin.net \
--cc=tzimmermann@suse.de \
--cc=vigneshr@ti.com \
/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.