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>,
"Raag Jadav" <raag.jadav@intel.com>,
"Reuven Abliyev" <reuven.abliyev@intel.com>,
linux-mtd@lists.infradead.org, intel-xe@lists.freedesktop.org,
dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v13 00/10] mtd: add driver for Intel discrete graphics
Date: Mon, 23 Jun 2025 13:16:46 -0400 [thread overview]
Message-ID: <aFmL_g4LSPLEfxu-@intel.com> (raw)
In-Reply-To: <20250617134532.3768283-1-alexander.usyskin@intel.com>
On Tue, Jun 17, 2025 at 04:45:22PM +0300, 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
>
> V8: Create separate partition device under master device, if requested
> and configure parent of usual partitions to this partition.
> Rebase over drm-tip.
>
> V9: Fix checkpatch warning on non-posted erase patch.
> Add Rodrigo's review and ack.
>
> V10: Drop master device creation patch as it now in mtd-next.
> Drop power-management patch, it will be merged lately after
> master device patch is propagated.
> Rebase over drm-tip.
>
> V11: Fix review comments.
> Add reviewed-by.
> Add cleanup in error path.
> Add PADDING region that exists on some BMG devices.
>
> V12: Add Raag's r-b.
> Rebase over drm-tip.
>
> V13: Rebase over drm-tip again to make it mergable.
I pushed this version to drm-xe-next, including i915 patches.
Thanks,
Rodrigo.
>
>
> Alexander Usyskin (9):
> 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
> 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
>
> Reuven Abliyev (1):
> drm/xe/nvm: add support for non-posted erase
>
> 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 | 121 ++++
> 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 | 167 ++++++
> 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 | 830 ++++++++++++++++++++++++++
> include/linux/intel_dg_nvm_aux.h | 32 +
> 19 files changed, 1236 insertions(+), 4 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>,
"Raag Jadav" <raag.jadav@intel.com>,
"Reuven Abliyev" <reuven.abliyev@intel.com>,
linux-mtd@lists.infradead.org, intel-xe@lists.freedesktop.org,
dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v13 00/10] mtd: add driver for Intel discrete graphics
Date: Mon, 23 Jun 2025 13:16:46 -0400 [thread overview]
Message-ID: <aFmL_g4LSPLEfxu-@intel.com> (raw)
In-Reply-To: <20250617134532.3768283-1-alexander.usyskin@intel.com>
On Tue, Jun 17, 2025 at 04:45:22PM +0300, 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
>
> V8: Create separate partition device under master device, if requested
> and configure parent of usual partitions to this partition.
> Rebase over drm-tip.
>
> V9: Fix checkpatch warning on non-posted erase patch.
> Add Rodrigo's review and ack.
>
> V10: Drop master device creation patch as it now in mtd-next.
> Drop power-management patch, it will be merged lately after
> master device patch is propagated.
> Rebase over drm-tip.
>
> V11: Fix review comments.
> Add reviewed-by.
> Add cleanup in error path.
> Add PADDING region that exists on some BMG devices.
>
> V12: Add Raag's r-b.
> Rebase over drm-tip.
>
> V13: Rebase over drm-tip again to make it mergable.
I pushed this version to drm-xe-next, including i915 patches.
Thanks,
Rodrigo.
>
>
> Alexander Usyskin (9):
> 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
> 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
>
> Reuven Abliyev (1):
> drm/xe/nvm: add support for non-posted erase
>
> 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 | 121 ++++
> 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 | 167 ++++++
> 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 | 830 ++++++++++++++++++++++++++
> include/linux/intel_dg_nvm_aux.h | 32 +
> 19 files changed, 1236 insertions(+), 4 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-06-23 17:17 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-17 13:45 [PATCH v13 00/10] mtd: add driver for Intel discrete graphics Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 01/10] mtd: add driver for intel graphics non-volatile memory device Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 02/10] mtd: intel-dg: implement region enumeration Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 03/10] mtd: intel-dg: implement access functions Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 04/10] mtd: intel-dg: register with mtd Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 05/10] mtd: intel-dg: align 64bit read and write Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 06/10] drm/i915/nvm: add nvm device for discrete graphics Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 07/10] drm/i915/nvm: add support for access mode Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 08/10] drm/xe/nvm: add on-die non-volatile memory device Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 09/10] drm/xe/nvm: add support for access mode Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 13:45 ` [PATCH v13 10/10] drm/xe/nvm: add support for non-posted erase Alexander Usyskin
2025-06-17 13:45 ` Alexander Usyskin
2025-06-17 14:06 ` ✗ CI.checkpatch: warning for mtd: add driver for Intel discrete graphics (rev6) Patchwork
2025-06-17 14:07 ` ✓ CI.KUnit: success " Patchwork
2025-06-17 14:22 ` ✗ CI.checksparse: warning " Patchwork
2025-06-17 14:54 ` ✓ Xe.CI.BAT: success " Patchwork
2025-06-17 16:08 ` ✓ i915.CI.BAT: success for mtd: add driver for Intel discrete graphics (rev14) Patchwork
2025-06-17 22:01 ` ✗ Xe.CI.Full: failure for mtd: add driver for Intel discrete graphics (rev6) Patchwork
2025-06-23 17:16 ` Rodrigo Vivi [this message]
2025-06-23 17:16 ` [PATCH v13 00/10] mtd: add driver for Intel discrete graphics Rodrigo Vivi
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=aFmL_g4LSPLEfxu-@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=intel-xe@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=raag.jadav@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.