linux-kernel - Re: [PATCH 1/2] drivers: base: component: add function to query the bound status

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: <2025022705-boat-overhaul-0ed3@gregkh>
Date: Thu, 27 Feb 2025 03:20:15 -0800
From: Greg KH <gregkh@...uxfoundation.org>
To: Heiko Stübner <heiko@...ech.de>
Cc: rafael@...nel.org, dakr@...nel.org, hjc@...k-chips.com,
	andy.yan@...k-chips.com, maarten.lankhorst@...ux.intel.com,
	mripard@...nel.org, tzimmermann@...e.de,
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org,
	linux-rockchip@...ts.infradead.org, quentin.schulz@...rry.de
Subject: Re: [PATCH 1/2] drivers: base: component: add function to query the
 bound status

On Thu, Feb 27, 2025 at 09:44:23AM +0100, Heiko Stübner wrote:
> Hi Greg,
> 
> Am Freitag, 21. Februar 2025, 07:14:07 MEZ schrieb Greg KH:
> > On Fri, Feb 21, 2025 at 12:41:40AM +0100, Heiko Stuebner wrote:
> > > The component helpers already expose the bound status in debugfs, but at
> > > times it might be necessary to also check that state in the kernel and
> > > act differently depending on the result.
> > > 
> > > For example the shutdown handler of a drm-driver might need to stop
> > > a whole output pipeline if the drm device is up and running, but may
> > > run into problems if that drm-device has never been set up before,
> > > for example because the binding deferred.
> > > 
> > > So add a little helper that returns the bound status for a componet
> > > device.
> > > 
> > > Signed-off-by: Heiko Stuebner <heiko@...ech.de>
> > > ---
> > >  drivers/base/component.c  | 14 ++++++++++++++
> > >  include/linux/component.h |  4 +++-
> > >  2 files changed, 17 insertions(+), 1 deletion(-)
> > 
> > Acked-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> 
> just for safety and not stepping on people's toes, does the Acked-by mean
> that this patch can go together with its user through the drm-misc tree?

Yup, merge away!