mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-09-18 22:14:16 +00:00
mfd: sec: Don't compare against NULL / 0 for errors, use !
Follow general style and use if (!arg) instead of comparing against NULL. While at it, drop a useless init in sec-irq.c. Signed-off-by: André Draszik <andre.draszik@linaro.org> Link: https://lore.kernel.org/r/20250409-s2mpg10-v4-19-d66d5f39b6bf@linaro.org Signed-off-by: Lee Jones <lee@kernel.org>
This commit is contained in:
parent
adf91d9e19
commit
856c6514d5
2 changed files with 3 additions and 3 deletions
|
@ -164,7 +164,7 @@ int sec_pmic_probe(struct device *dev, int device_type, unsigned int irq,
|
|||
int ret, num_sec_devs;
|
||||
|
||||
sec_pmic = devm_kzalloc(dev, sizeof(struct sec_pmic_dev), GFP_KERNEL);
|
||||
if (sec_pmic == NULL)
|
||||
if (!sec_pmic)
|
||||
return -ENOMEM;
|
||||
|
||||
dev_set_drvdata(dev, sec_pmic);
|
||||
|
|
|
@ -448,8 +448,8 @@ static const struct regmap_irq_chip s5m8767_irq_chip = {
|
|||
|
||||
int sec_irq_init(struct sec_pmic_dev *sec_pmic)
|
||||
{
|
||||
int ret = 0;
|
||||
const struct regmap_irq_chip *sec_irq_chip;
|
||||
int ret;
|
||||
|
||||
switch (sec_pmic->device_type) {
|
||||
case S5M8767X:
|
||||
|
@ -496,7 +496,7 @@ int sec_irq_init(struct sec_pmic_dev *sec_pmic)
|
|||
ret = devm_regmap_add_irq_chip(sec_pmic->dev, sec_pmic->regmap_pmic,
|
||||
sec_pmic->irq, IRQF_ONESHOT,
|
||||
0, sec_irq_chip, &sec_pmic->irq_data);
|
||||
if (ret != 0)
|
||||
if (ret)
|
||||
return dev_err_probe(sec_pmic->dev, ret,
|
||||
"Failed to add %s IRQ chip\n",
|
||||
sec_irq_chip->name);
|
||||
|
|
Loading…
Add table
Reference in a new issue