| 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
| ||
|
Message-ID: <Ya+mV/zuRVVIGVy1@phenom.ffwll.local>
Date: Tue, 7 Dec 2021 19:22:15 +0100
From: Daniel Vetter <daniel@...ll.ch>
To: Stephen Boyd <swboyd@...omium.org>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org,
Arnd Bergmann <arnd@...db.de>,
Benjamin Gaignard <benjamin.gaignard@...aro.org>,
Chen Feng <puck.chen@...ilicon.com>,
Chen-Yu Tsai <wens@...e.org>,
Christian Gmeiner <christian.gmeiner@...il.com>,
Chun-Kuang Hu <chunkuang.hu@...nel.org>,
Daniel Vetter <daniel.vetter@...ll.ch>,
Emma Anholt <emma@...olt.net>,
Heiko Stübner <heiko@...ech.de>,
Inki Dae <inki.dae@...sung.com>,
James Qian Wang <james.qian.wang@....com>,
Jaroslav Kysela <perex@...ex.cz>,
Joerg Roedel <joro@...tes.org>,
John Stultz <john.stultz@...aro.org>,
Joonyoung Shim <jy0922.shim@...sung.com>,
Jyri Sarha <jyri.sarha@....fi>,
Kai Vehmanen <kai.vehmanen@...ux.intel.com>,
Kyungmin Park <kyungmin.park@...sung.com>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>,
linux-fbdev@...r.kernel.org, linux-omap@...r.kernel.org,
linux-pm@...r.kernel.org, Linus Walleij <linus.walleij@...aro.org>,
Liviu Dudau <liviu.dudau@....com>,
Lucas Stach <l.stach@...gutronix.de>,
Mark Brown <broonie@...nel.org>,
Maxime Ripard <mripard@...nel.org>,
Neil Armstrong <narmstrong@...libre.com>,
Paul Cercueil <paul@...pouillou.net>,
Philipp Zabel <p.zabel@...gutronix.de>,
"Rafael J. Wysocki" <rafael@...nel.org>,
Rob Clark <robdclark@...il.com>,
Russell King <linux@...linux.org.uk>,
Russell King <linux+etnaviv@...linux.org.uk>,
Russell King <rmk+kernel@....linux.org.uk>,
Sandy Huang <hjc@...k-chips.com>,
Saravana Kannan <saravanak@...gle.com>,
Sebastian Reichel <sre@...nel.org>,
Seung-Woo Kim <sw0312.kim@...sung.com>,
Takashi Iwai <tiwai@...e.com>,
Tian Tao <tiantao6@...ilicon.com>,
Tomas Winkler <tomas.winkler@...el.com>,
Tomi Valkeinen <tomba@...nel.org>,
Will Deacon <will@...nel.org>,
Xinliang Liu <xinliang.liu@...aro.org>,
Xinwei Kong <kong.kongxinwei@...ilicon.com>,
Yong Wu <yong.wu@...iatek.com>,
Vitaly Lubart <vitaly.lubart@...el.com>,
Daniele Ceraolo Spurio <daniele.ceraolospurio@...el.com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>
Subject: Re: [PATCH v4 00/34] component: Make into an aggregate bus
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.
Otherwise I fear this might get stuck and I'd really like to avoid that.
Cheers, Daniel
>
> Changes since v3 (https://lore.kernel.org/r/20211026000044.885195-1-swboyd@chromium.org):
> - Picked up tags
> - Rebased to v5.16-rc2
> - Updated component.c for a few new patches there
> - Dropped a conversion patch
> - Added a conversion patch
>
> Changes since v2 (https://lore.kernel.org/r/20211006193819.2654854-1-swboyd@chromium.org):
> - Picked up acks
> - Fixed build warnings/errors
> - Reworked patch series to rename 'master' in a different patch
>
> Changes since v1 (https://lore.kernel.org/r/20210520002519.3538432-1-swboyd@chromium.org):
> - Use devlink to connect components to the aggregate device
> - Don't set the registering device as a parent of the aggregate device
> - New patch for bind_component/unbind_component ops that takes the
> aggregate device
> - Convert all drivers in the tree to use the aggregate driver approach
> - Allow one aggregate driver to be used for multiple aggregate devices
>
> [1] https://lore.kernel.org/r/20210508074118.1621729-1-swboyd@chromium.org
>
>
> Stephen Boyd (34):
> component: Introduce struct aggregate_device
> component: Remove most references to 'master'
> component: Introduce the aggregate bus_type
> component: Move struct aggregate_device out to header file
> component: Add {bind,unbind}_component() ops that take aggregate
> device
> drm/of: Add a drm_of_aggregate_probe() API
> drm/msm: Migrate to aggregate driver
> drm/komeda: Migrate to aggregate driver
> drm/arm/hdlcd: Migrate to aggregate driver
> drm/malidp: Migrate to aggregate driver
> drm/armada: Migrate to aggregate driver
> drm/etnaviv: Migrate to aggregate driver
> drm/kirin: Migrate to aggregate driver
> drm/exynos: Migrate to aggregate driver
> drm/imx: Migrate to aggregate driver
> drm/ingenic: Migrate to aggregate driver
> drm/mcde: Migrate to aggregate driver
> drm/mediatek: Migrate to aggregate driver
> drm/meson: Migrate to aggregate driver
> drm/omap: Migrate to aggregate driver
> drm/rockchip: Migrate to aggregate driver
> drm/sti: Migrate to aggregate driver
> drm/sun4i: Migrate to aggregate driver
> drm/tilcdc: Migrate to aggregate driver
> drm/vc4: Migrate to aggregate driver
> iommu/mtk: Migrate to aggregate driver
> mei: Migrate to aggregate driver
> power: supply: ab8500: Migrate to aggregate driver
> fbdev: omap2: Migrate to aggregate driver
> sound: hdac: Migrate to aggregate driver
> ASoC: codecs: wcd938x: Migrate to aggregate driver
> mei: pxp: Migrate to aggregate driver
> component: Get rid of drm_of_component_probe()
> component: Remove component_master_ops and friends
>
> drivers/base/component.c | 544 ++++++++++--------
> .../gpu/drm/arm/display/komeda/komeda_drv.c | 20 +-
> drivers/gpu/drm/arm/hdlcd_drv.c | 21 +-
> drivers/gpu/drm/arm/malidp_drv.c | 21 +-
> drivers/gpu/drm/armada/armada_drv.c | 23 +-
> drivers/gpu/drm/drm_drv.c | 2 +-
> drivers/gpu/drm/drm_of.c | 18 +-
> drivers/gpu/drm/etnaviv/etnaviv_drv.c | 20 +-
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 21 +-
> .../gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 20 +-
> drivers/gpu/drm/imx/imx-drm-core.c | 20 +-
> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 25 +-
> drivers/gpu/drm/mcde/mcde_drv.c | 23 +-
> drivers/gpu/drm/mediatek/mtk_drm_drv.c | 20 +-
> drivers/gpu/drm/meson/meson_drv.c | 21 +-
> drivers/gpu/drm/msm/msm_drv.c | 46 +-
> drivers/gpu/drm/omapdrm/dss/dss.c | 20 +-
> drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 20 +-
> drivers/gpu/drm/sti/sti_drv.c | 20 +-
> drivers/gpu/drm/sun4i/sun4i_drv.c | 26 +-
> drivers/gpu/drm/tilcdc/tilcdc_drv.c | 28 +-
> drivers/gpu/drm/vc4/vc4_drv.c | 20 +-
> drivers/iommu/mtk_iommu.c | 14 +-
> drivers/iommu/mtk_iommu.h | 6 +-
> drivers/iommu/mtk_iommu_v1.c | 14 +-
> drivers/misc/mei/hdcp/mei_hdcp.c | 22 +-
> drivers/misc/mei/pxp/mei_pxp.c | 22 +-
> drivers/power/supply/ab8500_charger.c | 22 +-
> drivers/video/fbdev/omap2/omapfb/dss/dss.c | 20 +-
> include/drm/drm_of.h | 10 +-
> include/linux/component.h | 92 ++-
> sound/hda/hdac_component.c | 21 +-
> sound/soc/codecs/wcd938x.c | 20 +-
> 33 files changed, 772 insertions(+), 490 deletions(-)
>
> Cc: Arnd Bergmann <arnd@...db.de>
> Cc: Benjamin Gaignard <benjamin.gaignard@...aro.org>
> Cc: Chen Feng <puck.chen@...ilicon.com>
> Cc: Chen-Yu Tsai <wens@...e.org>
> Cc: Christian Gmeiner <christian.gmeiner@...il.com>
> Cc: Chun-Kuang Hu <chunkuang.hu@...nel.org>
> Cc: Daniel Vetter <daniel.vetter@...ll.ch>
> Cc: Emma Anholt <emma@...olt.net>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Cc: "Heiko Stübner" <heiko@...ech.de>
> Cc: Inki Dae <inki.dae@...sung.com>
> Cc: James Qian Wang (Arm Technology China) <james.qian.wang@....com>
> Cc: Jaroslav Kysela <perex@...ex.cz>
> Cc: Joerg Roedel <joro@...tes.org>
> Cc: John Stultz <john.stultz@...aro.org>
> Cc: Joonyoung Shim <jy0922.shim@...sung.com>
> Cc: Jyri Sarha <jyri.sarha@....fi>
> Cc: Kai Vehmanen <kai.vehmanen@...ux.intel.com>
> Cc: Kyungmin Park <kyungmin.park@...sung.com>
> Cc: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> Cc: <linux-fbdev@...r.kernel.org>
> Cc: <linux-omap@...r.kernel.org>
> Cc: <linux-pm@...r.kernel.org>
> Cc: Linus Walleij <linus.walleij@...aro.org>
> Cc: Liviu Dudau <liviu.dudau@....com>
> Cc: Lucas Stach <l.stach@...gutronix.de>
> Cc: Mark Brown <broonie@...nel.org>
> Cc: Maxime Ripard <mripard@...nel.org>
> Cc: Neil Armstrong <narmstrong@...libre.com>
> Cc: Paul Cercueil <paul@...pouillou.net>
> Cc: Philipp Zabel <p.zabel@...gutronix.de>
> Cc: "Rafael J. Wysocki" <rafael@...nel.org>
> Cc: Rob Clark <robdclark@...il.com>
> Cc: Russell King <linux@...linux.org.uk>
> Cc: Russell King <linux+etnaviv@...linux.org.uk>
> Cc: Russell King <rmk+kernel@....linux.org.uk>
> Cc: Sandy Huang <hjc@...k-chips.com>
> Cc: Saravana Kannan <saravanak@...gle.com>
> Cc: Sebastian Reichel <sre@...nel.org>
> Cc: Seung-Woo Kim <sw0312.kim@...sung.com>
> Cc: Takashi Iwai <tiwai@...e.com>
> Cc: Tian Tao <tiantao6@...ilicon.com>
> Cc: Tomas Winkler <tomas.winkler@...el.com>
> Cc: Tomi Valkeinen <tomba@...nel.org>
> Cc: Will Deacon <will@...nel.org>
> Cc: Xinliang Liu <xinliang.liu@...aro.org>
> Cc: Xinwei Kong <kong.kongxinwei@...ilicon.com>
> Cc: Yong Wu <yong.wu@...iatek.com>
> Cc: Vitaly Lubart <vitaly.lubart@...el.com>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@...el.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@...el.com>
> Cc: Kai Vehmanen <kai.vehmanen@...ux.intel.com>
>
> base-commit: 136057256686de39cc3a07c2e39ef6bc43003ff6
> --
> https://chromeos.dev
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
Powered by blists - more mailing lists