[PATCH v2] drm/bridge: tda998x: Use __be32 for audio port OF property pointer

Kory Maincent posted 1 patch 1 month, 2 weeks ago
drivers/gpu/drm/bridge/tda998x_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH v2] drm/bridge: tda998x: Use __be32 for audio port OF property pointer
Posted by Kory Maincent 1 month, 2 weeks ago
From: "Kory Maincent (TI)" <kory.maincent@bootlin.com>

of_get_property() returns a pointer to big-endian (__be32) data, but
port_data in tda998x_get_audio_ports() was declared as const u32 *,
causing a sparse endianness type mismatch warning. Fix the declaration
to use const __be32 *.

Fixes: 7e567624dc5a4 ("drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding")
Cc: stable@vger.kernel.org
Signed-off-by: Kory Maincent (TI) <kory.maincent@bootlin.com>
Reviewed-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
---
 drivers/gpu/drm/bridge/tda998x_drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/tda998x_drv.c b/drivers/gpu/drm/bridge/tda998x_drv.c
index 367a46c057e41..6c427bc75896b 100644
--- a/drivers/gpu/drm/bridge/tda998x_drv.c
+++ b/drivers/gpu/drm/bridge/tda998x_drv.c
@@ -1762,7 +1762,7 @@ static const struct drm_bridge_funcs tda998x_bridge_funcs = {
 static int tda998x_get_audio_ports(struct tda998x_priv *priv,
 				   struct device_node *np)
 {
-	const u32 *port_data;
+	const __be32 *port_data;
 	u32 size;
 	int i;
 
-- 
2.43.0
Re: [PATCH v2] drm/bridge: tda998x: Use __be32 for audio port OF property pointer
Posted by Luca Ceresoli 1 month, 1 week ago
On Tue, 28 Apr 2026 11:04:56 +0200, Kory Maincent wrote:
> of_get_property() returns a pointer to big-endian (__be32) data, but
> port_data in tda998x_get_audio_ports() was declared as const u32 *,
> causing a sparse endianness type mismatch warning. Fix the declaration
> to use const __be32 *.

Applied, thanks!

[1/1] drm/bridge: tda998x: Use __be32 for audio port OF property pointer
      commit: 2a46a9356ba7b1bdd741c8b41e5374edcd960557

Best regards,
-- 
Luca Ceresoli <luca.ceresoli@bootlin.com>