linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Raag Jadav <raag.jadav@intel.com>
Cc: <lucas.demarchi@intel.com>, <thomas.hellstrom@linux.intel.com>,
	<jarkko.nikula@linux.intel.com>, <airlied@gmail.com>,
	<simona@ffwll.ch>, <andriy.shevchenko@linux.intel.com>,
	<mika.westerberg@linux.intel.com>, <jsd@semihalf.com>,
	<andi.shyti@kernel.org>, <riana.tauro@intel.com>,
	<srinivasa.adatrao@intel.com>, <michael.j.ruhl@intel.com>,
	<intel-xe@lists.freedesktop.org>, <linux-i2c@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v7 0/5] drm/xe: i2c support
Date: Thu, 10 Jul 2025 11:39:25 -0400	[thread overview]
Message-ID: <aG_erdEFxtBPg_oh@intel.com> (raw)
In-Reply-To: <20250710071612.2714990-1-raag.jadav@intel.com>

On Thu, Jul 10, 2025 at 12:46:07PM +0530, Raag Jadav wrote:
> Hi,
> 
> On behalf of Heikki, since he's on vacation.

ops, I'm really sorry that I had missed this.
The first think I did this morning was to merge his series,
changing while merging, once I heard he was out on vacation.

But thank you so much for sending this as well.

> 
> Changed since v6:
> - Fix SPDX checkpatch warning.
> 
> Changed since v5:
> - The modification to the dw_i2c_plat_probe() is split into its own
>   patch as proposed by Andi.
> - I removed completely the comment from the last patch in the series
>   ("drm/xe/xe_i2c: Add support for i2c in survivability mode").
> 
> Changed since v4:
> - Cleanups requested by Andy.
> - Casting the PCI power modes to make sparse happy - Raag.
> - Limiting the use of this thing to Battlemage again. But I'm not sure
>   if this is the correct thing to do.
> 
> Changed since v3:
> - Cleanups as requested by Rodrigo.
> - The licence is now changed, but still need confirmation for it!
> 
> Changed since v2:
> - Added dependency on regmap when i2c is enabled.
> 
> Changed since v1:
> - Now rebased on top of drm-tip.
> - No longer ignoring errors from xe_i2c_probe().
> - Cleanups pointed out by Lucas.
> 
> I've also included followup patches from Raag and Riana to this
> series.
> 
> Original cover letter:
> 
> Some of the future GPUs will provide access to the on-board Synopsys
> DesignWare I2C host adapter. The i2c is used to connect various
> microcontrollers. The initially supported microcontroller unit is
> called Add-In Management Controller (AMC).
> 
> Thanks,
> 
> Heikki Krogerus (3):
>   i2c: designware: Use polling by default when there is no irq resource
>   i2c: designware: Add quirk for Intel Xe
>   drm/xe: Support for I2C attached MCUs
> 
> Raag Jadav (1):
>   drm/xe/pm: Wire up suspend/resume for I2C controller
> 
> Riana Tauro (1):
>   drm/xe/xe_i2c: Add support for i2c in survivability mode
> 
>  drivers/gpu/drm/xe/Kconfig                  |   1 +
>  drivers/gpu/drm/xe/Makefile                 |   1 +
>  drivers/gpu/drm/xe/regs/xe_i2c_regs.h       |  20 ++
>  drivers/gpu/drm/xe/regs/xe_irq_regs.h       |   1 +
>  drivers/gpu/drm/xe/regs/xe_pmt.h            |   2 +-
>  drivers/gpu/drm/xe/regs/xe_regs.h           |   2 +
>  drivers/gpu/drm/xe/xe_device.c              |   5 +
>  drivers/gpu/drm/xe/xe_device_types.h        |   4 +
>  drivers/gpu/drm/xe/xe_i2c.c                 | 329 ++++++++++++++++++++
>  drivers/gpu/drm/xe/xe_i2c.h                 |  62 ++++
>  drivers/gpu/drm/xe/xe_irq.c                 |   2 +
>  drivers/gpu/drm/xe/xe_pm.c                  |   9 +
>  drivers/gpu/drm/xe/xe_survivability_mode.c  |  19 +-
>  drivers/i2c/busses/i2c-designware-platdrv.c |  18 +-
>  14 files changed, 460 insertions(+), 15 deletions(-)
>  create mode 100644 drivers/gpu/drm/xe/regs/xe_i2c_regs.h
>  create mode 100644 drivers/gpu/drm/xe/xe_i2c.c
>  create mode 100644 drivers/gpu/drm/xe/xe_i2c.h
> 
> -- 
> 2.34.1
> 

      parent reply	other threads:[~2025-07-10 15:39 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-10  7:16 [PATCH v7 0/5] drm/xe: i2c support Raag Jadav
2025-07-10  7:16 ` [PATCH v7 1/5] i2c: designware: Use polling by default when there is no irq resource Raag Jadav
2025-07-28 13:43   ` Jarkko Nikula
2025-07-10  7:16 ` [PATCH v7 2/5] i2c: designware: Add quirk for Intel Xe Raag Jadav
2025-07-28 13:44   ` Jarkko Nikula
2025-07-10  7:16 ` [PATCH v7 3/5] drm/xe: Support for I2C attached MCUs Raag Jadav
2025-07-10  7:16 ` [PATCH v7 4/5] drm/xe/pm: Wire up suspend/resume for I2C controller Raag Jadav
2025-07-10  7:16 ` [PATCH v7 5/5] drm/xe/xe_i2c: Add support for i2c in survivability mode Raag Jadav
2025-07-10 15:39 ` Rodrigo Vivi [this message]

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=aG_erdEFxtBPg_oh@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=airlied@gmail.com \
    --cc=andi.shyti@kernel.org \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=jarkko.nikula@linux.intel.com \
    --cc=jsd@semihalf.com \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lucas.demarchi@intel.com \
    --cc=michael.j.ruhl@intel.com \
    --cc=mika.westerberg@linux.intel.com \
    --cc=raag.jadav@intel.com \
    --cc=riana.tauro@intel.com \
    --cc=simona@ffwll.ch \
    --cc=srinivasa.adatrao@intel.com \
    --cc=thomas.hellstrom@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).