On Sun, Mar 26, 2023 at 12:25:57PM -0400, William Breathitt Gray wrote:
> Changes in v5:
> - Wrap lines to 100 characters rather than 80
It's good to keep things under 80 where we can.
> int (*set_type_config)(unsigned int **buf, unsigned int type,
> - const struct regmap_irq *irq_data, int idx);
> + const struct regmap_irq *irq_data, int idx, void *irq_drv_data);
> unsigned int (*get_irq_reg)(struct regmap_irq_chip_data *data,
> unsigned int base, int index);
> void *irq_drv_data;
There's no benefit from overflowing here and...
> @@ -1660,7 +1660,8 @@ struct regmap_irq_chip {
> unsigned int regmap_irq_get_irq_reg_linear(struct regmap_irq_chip_data *data,
> unsigned int base, int index);
> int regmap_irq_set_type_config_simple(unsigned int **buf, unsigned int type,
> - const struct regmap_irq *irq_data, int idx);
> + const struct regmap_irq *irq_data, int idx,
> + void *irq_drv_data);
...it's not even consistent.
Attachment:
signature.asc
Description: PGP signature
