diff options
| author | ChiYuan Huang <cy_huang@richtek.com> | 2025-05-08 11:01:23 +0800 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2025-05-08 23:41:14 +0900 |
| commit | 07cb9b51076f452c286fb69f3af4c65e54affaa3 (patch) | |
| tree | beaefbca63c8e918091ac6647ac0e316c981ac38 | |
| parent | cece89eb2bd2634e7faf65f2ecfccfc4809261c8 (diff) | |
ASoC: codecs: rt9123: Fix sparse cast warning
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>
| -rw-r--r-- | sound/soc/codecs/rt9123.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/soc/codecs/rt9123.c b/sound/soc/codecs/rt9123.c index d9a22fda3d26..242e8c975a62 100644 --- a/sound/soc/codecs/rt9123.c +++ b/sound/soc/codecs/rt9123.c @@ -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"); |