All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lukas Wunner <lukas@wunner.de>
To: Atharva Tiwari <atharvatiwarilinuxdev@gmail.com>
Cc: airlied@gmail.com, ardb@kernel.org, bp@alien8.de,
	dri-devel@lists.freedesktop.org, francescopompo2@gmail.com,
	intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org,
	jani.nikula@linux.intel.com, javierm@redhat.com,
	joonas.lahtinen@linux.intel.com, linux-efi@vger.kernel.org,
	linux-kernel@vger.kernel.org, lszubowi@redhat.com,
	rodrigo.vivi@intel.com, simona@ffwll.ch, tursulin@ursulin.net,
	tzimmermann@suse.de
Subject: Re: [PATCH v2 1/2] efi/libstub: enable apple-set-os for all apple devices
Date: Thu, 29 Jan 2026 11:33:00 +0100	[thread overview]
Message-ID: <aXs3XOYucVna8eGH@wunner.de> (raw)
In-Reply-To: <20260129043759.1220-1-atharvatiwarilinuxdev@gmail.com>

On Thu, Jan 29, 2026 at 04:37:59AM +0000, Atharva Tiwari wrote:
> > As discussed previously, your patch will regress older MacBook Airs
> > which use SPI to access keyboard + trackpad if the set_os protocol
> > is invoked:
> > 
> > https://lore.kernel.org/all/ZoJPgSlZJ3ZlU2zL@wunner.de/
> > 
> > The last time this topic came up, I suggested counting the number of
> > GPUs instead of using the DMI quirk.  I even provided a patch but
> > nobody with an eGPU bothered to test it, so the thread fizzled out:
> > 
> > https://lore.kernel.org/all/Z6paeFrjdv7L3mtv@wunner.de/
> > 
> > Maybe you could give it a spin and verify if it fixes the issue for you?
> 
> I tested your patch, and well it works (you still need the 2nd patch to 
> avoid a blank screen), but i can see how this will fail
> like for example, with the iMacPro (only has dGPU) you would need 2 eGPUs
> to turn on apple-set-os.

I don't quite follow.  The patch counts the number of GPUs and
uses the set_os protocol if there are at least 2 of them.
One dGPU + one eGPU = 2, so the protocol is used.

Am I missing something?

Thanks,

Lukas

  reply	other threads:[~2026-01-29 10:33 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-28 14:58 [PATCH v2 0/2] Add support for eGPUs on Apple Products Atharva Tiwari
2026-01-28 14:58 ` [PATCH v2 1/2] efi/libstub: enable apple-set-os for all apple devices Atharva Tiwari
2026-01-28 15:45   ` Lukas Wunner
2026-01-29  4:37     ` Atharva Tiwari
2026-01-29 10:33       ` Lukas Wunner [this message]
2026-01-29 17:53         ` Atharva Tiwari
2026-02-02  8:01     ` Atharva Tiwari
2026-01-28 14:58 ` [PATCH v2 2/2] drm/i915/display: Disable display for iMac's Atharva Tiwari
2026-01-28 15:23   ` Jani Nikula
2026-01-28 15:37     ` Ville Syrjälä
2026-01-28 16:03       ` Lukas Wunner
2026-01-28 15:49   ` Lukas Wunner
2026-01-28 15:56 ` [PATCH v2 0/2] Add support for eGPUs on Apple Products Lukas Wunner
2026-01-28 17:12   ` Atharva Tiwari
2026-01-29 21:00 ` ✗ LGCI.VerificationFailed: failure for Add support for eGPUs on Apple Products (rev2) Patchwork
2026-01-29 21:02 ` 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=aXs3XOYucVna8eGH@wunner.de \
    --to=lukas@wunner.de \
    --cc=airlied@gmail.com \
    --cc=ardb@kernel.org \
    --cc=atharvatiwarilinuxdev@gmail.com \
    --cc=bp@alien8.de \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=francescopompo2@gmail.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    --cc=javierm@redhat.com \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=linux-efi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lszubowi@redhat.com \
    --cc=rodrigo.vivi@intel.com \
    --cc=simona@ffwll.ch \
    --cc=tursulin@ursulin.net \
    --cc=tzimmermann@suse.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.