<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, May 9, 2025 at 9:56 AM Luca Ceresoli <<a href="mailto:luca.ceresoli@bootlin.com">luca.ceresoli@bootlin.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This is the new API for allocating DRM bridges.<br>
<br>
Signed-off-by: Luca Ceresoli <<a href="mailto:luca.ceresoli@bootlin.com" target="_blank">luca.ceresoli@bootlin.com</a>><br>
<br></blockquote><div>Looks good!</div><div><br></div><div>Reviewed-by: Anusha Srivatsa <<a href="mailto:asrivats@redhat.com">asrivats@redhat.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
<br>
Cc: "Rob Herring (Arm)" <<a href="mailto:robh@kernel.org" target="_blank">robh@kernel.org</a>><br>
Cc: Hsin-Te Yuan <<a href="mailto:yuanhsinte@chromium.org" target="_blank">yuanhsinte@chromium.org</a>><br>
Cc: Jani Nikula <<a href="mailto:jani.nikula@intel.com" target="_blank">jani.nikula@intel.com</a>><br>
Cc: Pin-yen Lin <<a href="mailto:treapking@chromium.org" target="_blank">treapking@chromium.org</a>><br>
Cc: Sui Jingfeng <<a href="mailto:sui.jingfeng@linux.dev" target="_blank">sui.jingfeng@linux.dev</a>><br>
Cc: Xin Ji <<a href="mailto:xji@analogixsemi.com" target="_blank">xji@analogixsemi.com</a>><br>
---<br>
 drivers/gpu/drm/bridge/analogix/anx7625.c | 7 +++----<br>
 1 file changed, 3 insertions(+), 4 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/bridge/analogix/anx7625.c<br>
index 8a9079c2ed5c22565d0733ef899119e347947d21..0ac4a82c5a6e5e665d913674b664905a96c0d8e6 100644<br>
--- a/drivers/gpu/drm/bridge/analogix/anx7625.c<br>
+++ b/drivers/gpu/drm/bridge/analogix/anx7625.c<br>
@@ -2596,7 +2596,6 @@ static int anx7625_link_bridge(struct drm_dp_aux *aux)<br>
                return ret;<br>
        }<br>
<br>
-       platform->bridge.funcs = &anx7625_bridge_funcs;<br>
        platform->bridge.of_node = dev->of_node;<br>
        if (!anx7625_of_panel_on_aux_bus(dev))<br>
                platform->bridge.ops |= DRM_BRIDGE_OP_EDID;<br>
@@ -2630,10 +2629,10 @@ static int anx7625_i2c_probe(struct i2c_client *client)<br>
                return -ENODEV;<br>
        }<br>
<br>
-       platform = devm_kzalloc(dev, sizeof(*platform), GFP_KERNEL);<br>
-       if (!platform) {<br>
+       platform = devm_drm_bridge_alloc(dev, struct anx7625_data, bridge, &anx7625_bridge_funcs);<br>
+       if (IS_ERR(platform)) {<br>
                DRM_DEV_ERROR(dev, "fail to allocate driver data\n");<br>
-               return -ENOMEM;<br>
+               return PTR_ERR(platform);<br>
        }<br>
<br>
        pdata = &platform->pdata;<br>
<br>
-- <br>
2.49.0<br>
<br>
</blockquote></div></div>