Commit 6f5976c0 authored by Michael Roth's avatar Michael Roth Committed by Bartosz Golaszewski
Browse files

gpio: pca953x: clarify log messages about auto increment feature 



The probe messages currently print "using AI" and "using no AI",
which can be confusing to users unfamiliar with the datasheet term.

Clarify these by spelling out "auto increment", which is the meaning
of the AI bit described in the register map.

No functional change, only clearer log wording and matching comment
update.

Signed-off-by: default avatarMichael Roth <mail@mroth.net>
Link: https://lore.kernel.org/r/20251018101404.3630905-1-mail@mroth.net


Signed-off-by: default avatarBartosz Golaszewski <bartosz.golaszewski@linaro.org>
parent eba11116

drivers/gpio/gpio-pca953x.c

+3 −3
Original line number Diff line number Diff line
@@ -306,7 +306,7 @@ static inline u8 pca953x_get_bit_mask(struct pca953x_chip *chip, unsigned int of 
 *     Interrupt mask register		0x40 + 5 * bank_size	RW

 *     Interrupt status register	0x40 + 6 * bank_size	R

 *

 * - Registers with bit 0x80 set, the AI bit

 * - Registers with bit 0x80 set, the AI bit (auto increment)

 *   The bit is cleared and the registers fall into one of the

 *   categories above.

 */
@@ -1206,10 +1206,10 @@ static int pca953x_probe(struct i2c_client *client) 
	pca953x_setup_gpio(chip, chip->driver_data & PCA_GPIO_MASK);



	if (NBANK(chip) > 2 || PCA_CHIP_TYPE(chip->driver_data) == PCA957X_TYPE) {

		dev_info(dev, "using AI\n");

		dev_info(dev, "using auto increment\n");

		regmap_config = &pca953x_ai_i2c_regmap;

	} else {

		dev_info(dev, "using no AI\n");

		dev_info(dev, "using no auto increment\n");

		regmap_config = &pca953x_i2c_regmap;

	}