linux-kernel - Re: [PATCH v4 2/4] drm/panel: Add refcount support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250527-origami-mayfly-of-superiority-2a1cf1@houat>
Date: Tue, 27 May 2025 17:04:00 +0200
From: Maxime Ripard <mripard@...nel.org>
To: Jani Nikula <jani.nikula@...ux.intel.com>
Cc: Anusha Srivatsa <asrivats@...hat.com>, 
	Neil Armstrong <neil.armstrong@...aro.org>, Jessica Zhang <quic_jesszhan@...cinc.com>, 
	Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Thomas Zimmermann <tzimmermann@...e.de>, 
	David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>, dri-devel@...ts.freedesktop.org, 
	linux-kernel@...r.kernel.org, Luca Ceresoli <luca.ceresoli@...tlin.com>
Subject: Re: [PATCH v4 2/4] drm/panel: Add refcount support

Hi,

On Fri, May 23, 2025 at 02:34:05PM +0300, Jani Nikula wrote:
> On Tue, 20 May 2025, Jani Nikula <jani.nikula@...ux.intel.com> wrote:
> > Maxime -
> >
> > I'm cutting a lot of context here. Not because I don't think it deserves
> > an answer, but because I seem to be failing at communication.
> >
> > On Mon, 19 May 2025, Maxime Ripard <mripard@...nel.org> wrote:
> >> You still haven't explained why it would take anything more than
> >> registering a dumb device at probe time though.
> >
> > With that, do you mean a dumb struct device, or any struct device with a
> > suitable lifetime, that we'd pass to devm_drm_panel_alloc()?
> 
> I'm no expert in ACPI, but I think it needs to be a struct device
> embedded inside struct acpi_device to implement the
> drm_panel_add_follower() lookup for ACPI.

What data do you have in the ACPI tables to associate the HID
touchscreen to the panel?

drm_panel_add_follower use the DT to lookup the panel, so the panel must
have a device->of_node pointer, so that obviously won't work, but it
might with ACPI, or we might need to split that function into several
parts to accomodate ACPI.

Maxime

Download attachment "signature.asc" of type "application/pgp-signature" (274 bytes)