leds: aw200xx: Add delay after software reset

geo-stark · lag-linaro · commit d883a5ab2f34 · 2023-12-13T11:28:58.000Z
According to the datasheets of AW200xx devices, the software reset takes at least 1ms. Therefore, it is required to add a delay after the reset before issuing commands to the device. Signed-off-by: George Stark <gnstark@salutedevices.com> Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com> Link: https://lore.kernel.org/r/20231125200519.1750-7-ddrokosov@salutedevices.com Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/leds/leds-aw200xx.c b/drivers/leds/leds-aw200xx.c
@@ -321,6 +321,9 @@ static int aw200xx_chip_reset(const struct aw200xx *const chip)
 	if (ret)
 		return ret;
 
+	/* According to the datasheet software reset takes at least 1ms */
+	fsleep(1000);
+
 	regcache_mark_dirty(chip->regmap);
 	return regmap_write(chip->regmap, AW200XX_REG_FCD, AW200XX_FCD_CLEAR);
 }

Original file line number	Diff line number	Diff line change
`@@ -321,6 +321,9 @@ static int aw200xx_chip_reset(const struct aw200xx *const chip)`
`321`	`321`	`if (ret)`
`322`	`322`	`return ret;`
`323`	`323`
	`324`	`+ /* According to the datasheet software reset takes at least 1ms */`
	`325`	`+ fsleep(1000);`
	`326`	`+`
`324`	`327`	`regcache_mark_dirty(chip->regmap);`
`325`	`328`	`return regmap_write(chip->regmap, AW200XX_REG_FCD, AW200XX_FCD_CLEAR);`
`326`	`329`	`}`