ASoC: codecs: rt9123: Fix sparse cast warning
authorChiYuan Huang <cy_huang@richtek.com>
Thu, 8 May 2025 03:01:23 +0000 (11:01 +0800)
committerMark Brown <broonie@kernel.org>
Thu, 8 May 2025 14:41:14 +0000 (23:41 +0900)
Use i2c block read/write API to fix casting warning.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202505072140.iHyLlDN6-lkp@intel.com/
Signed-off-by: ChiYuan Huang <cy_huang@richtek.com>
Link: https://patch.msgid.link/185a5ffea22ebd20725fdc7739db6b6addfae3ad.1746672687.git.cy_huang@richtek.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/rt9123.c

index d9a22fda3d2624a0c661173a1a7dcab858fcd16d..242e8c975a62168085397dec025be70839c51d72 100644 (file)
@@ -399,17 +399,17 @@ static int rt9123_i2c_probe(struct i2c_client *i2c)
                dev_dbg(dev, "No 'enable' GPIO specified, treat it as default on\n");
 
        /* Check vendor id information */
-       ret = i2c_smbus_read_word_data(i2c, RT9123_REG_COMBOID);
+       ret = i2c_smbus_read_i2c_block_data(i2c, RT9123_REG_COMBOID, sizeof(value), (u8 *)&value);
        if (ret < 0)
                return dev_err_probe(dev, ret, "Failed to read vendor-id\n");
 
-       venid = be16_to_cpu(ret);
+       venid = be16_to_cpu(value);
        if ((venid & RT9123_MASK_VENID) != RT9123_FIXED_VENID)
                return dev_err_probe(dev, -ENODEV, "Incorrect vendor-id 0x%04x\n", venid);
 
        /* Trigger RG reset before regmap init cache */
        value = cpu_to_be16(RT9123_MASK_SWRST);
-       ret = i2c_smbus_write_word_data(i2c, RT9123_REG_AMPCTRL, value);
+       ret = i2c_smbus_write_i2c_block_data(i2c, RT9123_REG_AMPCTRL, sizeof(value), (u8 *)&value);
        if (ret)
                return dev_err_probe(dev, ret, "Failed to trigger RG reset\n");