linux-kernel - Re: [PATCH 02/34] platform: arm64: acer-aspire1-ec: convert to devm_drm_bridge

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 for Android: free password hash cracker in your pocket

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

Message-ID: <a9000632-a6d1-d369-c317-9ee73aa645dc@linux.intel.com>
Date: Mon, 7 Apr 2025 19:46:59 +0300 (EEST)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Luca Ceresoli <luca.ceresoli@...tlin.com>
cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, 
    Maxime Ripard <mripard@...nel.org>, 
    Thomas Zimmermann <tzimmermann@...e.de>, David Airlie <airlied@...il.com>, 
    Simona Vetter <simona@...ll.ch>, Andrzej Hajda <andrzej.hajda@...el.com>, 
    Neil Armstrong <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>, 
    Laurent Pinchart <Laurent.pinchart@...asonboard.com>, 
    Jonas Karlman <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>, 
    Jagan Teki <jagan@...rulasolutions.com>, Shawn Guo <shawnguo@...nel.org>, 
    Sascha Hauer <s.hauer@...gutronix.de>, 
    Pengutronix Kernel Team <kernel@...gutronix.de>, 
    Fabio Estevam <festevam@...il.com>, 
    Douglas Anderson <dianders@...omium.org>, 
    Chun-Kuang Hu <chunkuang.hu@...nel.org>, 
    Krzysztof Kozlowski <krzk@...nel.org>, 
    Dmitry Baryshkov <dmitry.baryshkov@...aro.org>, 
    Anusha Srivatsa <asrivats@...hat.com>, 
    Paul Kocialkowski <paulk@...-base.io>, Dmitry Baryshkov <lumag@...nel.org>, 
    Hervé Codina <herve.codina@...tlin.com>, 
    Hui Pu <Hui.Pu@...ealthcare.com>, 
    Thomas Petazzoni <thomas.petazzoni@...tlin.com>, 
    dri-devel@...ts.freedesktop.org, asahi@...ts.linux.dev, 
    LKML <linux-kernel@...r.kernel.org>, chrome-platform@...ts.linux.dev, 
    imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org, 
    linux-mediatek@...ts.infradead.org, linux-amlogic@...ts.infradead.org, 
    linux-renesas-soc@...r.kernel.org, platform-driver-x86@...r.kernel.org, 
    linux-samsung-soc@...r.kernel.org, linux-arm-msm@...r.kernel.org, 
    freedreno@...ts.freedesktop.org, linux-stm32@...md-mailman.stormreply.com, 
    Bryan O'Donoghue <bryan.odonoghue@...aro.org>, 
    Hans de Goede <hdegoede@...hat.com>
Subject: Re: [PATCH 02/34] platform: arm64: acer-aspire1-ec: convert to
 devm_drm_bridge_alloc() API

On Mon, 7 Apr 2025, Luca Ceresoli wrote:

> This is the new API for allocating DRM bridges.
> 
> Signed-off-by: Luca Ceresoli <luca.ceresoli@...tlin.com>
> 
> ---
> 
> Cc: "Bryan O'Donoghue" <bryan.odonoghue@...aro.org>
> Cc: "Ilpo Järvinen" <ilpo.jarvinen@...ux.intel.com>
> Cc: Hans de Goede <hdegoede@...hat.com>
> ---
>  drivers/platform/arm64/acer-aspire1-ec.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/platform/arm64/acer-aspire1-ec.c b/drivers/platform/arm64/acer-aspire1-ec.c
> index 958fe1bf5f85bb69ac7962f217de9f0b40cde9a1..438532a047e68799ac53a16a4c813fc16be997b9 100644
> --- a/drivers/platform/arm64/acer-aspire1-ec.c
> +++ b/drivers/platform/arm64/acer-aspire1-ec.c
> @@ -452,9 +452,9 @@ static int aspire_ec_probe(struct i2c_client *client)
>  	int ret;
>  	u8 tmp;
>  
> -	ec = devm_kzalloc(dev, sizeof(*ec), GFP_KERNEL);
> -	if (!ec)
> -		return -ENOMEM;
> +	ec = devm_drm_bridge_alloc(dev, struct aspire_ec, bridge, &aspire_ec_bridge_funcs);
> +	if (IS_ERR(ec))
> +		return PTR_ERR(ec);
>  
>  	ec->client = client;
>  	i2c_set_clientdata(client, ec);
> @@ -497,7 +497,6 @@ static int aspire_ec_probe(struct i2c_client *client)
>  	fwnode = device_get_named_child_node(dev, "connector");
>  	if (fwnode) {
>  		INIT_WORK(&ec->work, aspire_ec_bridge_update_hpd_work);
> -		ec->bridge.funcs = &aspire_ec_bridge_funcs;
>  		ec->bridge.of_node = to_of_node(fwnode);
>  		ec->bridge.ops = DRM_BRIDGE_OP_HPD;
>  		ec->bridge.type = DRM_MODE_CONNECTOR_USB;

Hi Luca,

It took a while to locate where the code for the new helper is. I suggest 
if you need send another version of the series directly linking to the 
commit in the cover letter so that it won't take multiple hoops to find it 
if one wants to review the code and is not having all drm trees easily at 
hand. Here it is for the benefit of other pdx86 people:

https://gitlab.freedesktop.org/drm/misc/kernel/-/commit/0cc6aadd7fc1e629b715ea3d1ba537ef2da95eec


Acked-by: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>

I assume you want this to go through the drm tree where the helper already 
is?

-- 
 i.