On 6/21/20 12:58 AM, Philippe Mathieu-Daudé wrote:
> Replace the '16' magic value by the PCA9552_PIN_COUNT definition.
Reviewed-by: Cédric Le Goater <c...@kaod.org>
>
> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> include/hw/misc/pca9552.h | 1 +
> hw/misc/pca9552.c | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/hw/misc/pca9552.h b/include/hw/misc/pca9552.h
> index ebb43c63fe..ef6da4988f 100644
> --- a/include/hw/misc/pca9552.h
> +++ b/include/hw/misc/pca9552.h
> @@ -15,6 +15,7 @@
> #define PCA9552(obj) OBJECT_CHECK(PCA9552State, (obj), TYPE_PCA9552)
>
> #define PCA9552_NR_REGS 10
> +#define PCA9552_PIN_COUNT 16
>
> typedef struct PCA9552State {
> /*< private >*/
> diff --git a/hw/misc/pca9552.c b/hw/misc/pca9552.c
> index cac729e35a..cfefb8fce8 100644
> --- a/hw/misc/pca9552.c
> +++ b/hw/misc/pca9552.c
> @@ -291,7 +291,7 @@ static void pca9552_initfn(Object *obj)
> * PCA955X device
> */
> s->max_reg = PCA9552_LS3;
> - s->nr_leds = 16;
> + s->nr_leds = PCA9552_PIN_COUNT;
>
> for (led = 0; led < s->nr_leds; led++) {
> char *name;
>
