Re: [PATCH v4 00/34] component: Make into an aggregate bus

From: Stephen Boyd <swboyd@chromium.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Daniel Vetter <daniel@ffwll.ch>
Cc: linux-fbdev@vger.kernel.org, Emma Anholt <emma@anholt.net>,
	Neil Armstrong <narmstrong@baylibre.com>,
	Xinliang Liu <xinliang.liu@linaro.org>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	Liviu Dudau <liviu.dudau@arm.com>,
	dri-devel@lists.freedesktop.org, Jaroslav Kysela <perex@perex.cz>,
	Paul Cercueil <paul@crapouillou.net>,
	Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Benjamin Gaignard <benjamin.gaignard@linaro.org>,
	Russell King <linux@armlinux.org.uk>,
	Will Deacon <will@kernel.org>,
	Vitaly Lubart <vitaly.lubart@intel.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Saravana Kannan <saravanak@google.com>,
	Joerg Roedel <joro@8bytes.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Chen-Yu Tsai <wens@csie.org>, Sandy Huang <hjc@rock-chips.com>,
	James Qian Wang <james.qian.wang@arm.com>,
	Chen Feng <puck.chen@hisilicon.com>,
	Russell King <rmk+kernel@arm.linux.org.uk>,
	Tomas Winkler <tomas.winkler@intel.com>,
	Xinwei Kong <kong.kongxinwei@hisilicon.com>,
	Chun-Kuang Hu <chunkuang.hu@kernel.org>,
	Arnd Bergmann <arnd@arndb.de>,
	linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	Russell King <linux+etnaviv@armlinux.org.uk>,
	Mark Brown <broonie@kernel.org>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>,
	Takashi Iwai <tiwai@suse.com>,
	linux-omap@vger.kernel.org, Yong Wu <yong.wu@mediatek.com>,
	Tomi Valkeinen <tomba@kernel.org>,
	Kai Vehmanen <kai.vehmanen@linux.intel.com>,
	Jyri Sarha <jyri.sarha@iki.fi>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	linux-kernel@vger.kernel.org,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Sebastian Reichel <sre@kernel.org>,
	Tian Tao <tiantao6@hisilicon.com>,
	freedreno@lists.freedesktop.org
Subject: Re: [PATCH v4 00/34] component: Make into an aggregate bus
Date: Mon, 13 Dec 2021 13:43:19 -0800	[thread overview]
Message-ID: <CAE-0n53dnsAdjBUVmDMbcvrSJEE=RPu7PGX_zaWCZ_ZBUAythA@mail.gmail.com> (raw)
In-Reply-To: <Ya+mV/zuRVVIGVy1@phenom.ffwll.local>

Quoting Daniel Vetter (2021-12-07 10:22:15)
> On Thu, Dec 02, 2021 at 02:26:58PM -0800, Stephen Boyd wrote:
> > This series is from discussion we had on reordering the device lists for
> > drm shutdown paths[1]. I've introduced an 'aggregate' bus that we put
> > the aggregate device onto and then we probe the aggregate device once
> > all the components are probed and call component_add(). The probe/remove
> > hooks are where the bind/unbind calls go, and then a shutdown hook is
> > added that can be used to shutdown the drm display pipeline at the right
> > time.
> >
> > This works for me on my sc7180 board. I no longer get a warning from i2c
> > at shutdown that we're trying to make an i2c transaction after the i2c
> > bus has been shutdown. There's more work to do on the msm drm driver to
> > extract component device resources like clks, regulators, etc. out of
> > the component bind function into the driver probe but I wanted to move
> > everything over now in other component drivers before tackling that
> > problem.
> >
> > Tested-by tags would be appreciated, and Acked-by/Reviewed-by tags too.
>
> Thanks for pushing this forward. Unfortunately I'm completely burried and
> it's just not improving, so merge plan:
>
> - please get Greg KH to ack the bus/driver core stuff
>
> - please get one of the drm-misc committers we have from Google's Chromeos
>   team (there should be a few by now) to review&push this.

Sounds like a plan. Thanks!

>
> Otherwise I fear this might get stuck and I'd really like to avoid that.
>