[PATCH v5 3/5] gpio: spacemit: add support for K1 SoC

Chen Wang unicorn_wang at outlook.com
Thu Feb 20 15:35:51 PST 2025


On 2025/2/20 21:34, Bartosz Golaszewski wrote:
> On Mon, Feb 17, 2025 at 1:58 PM Yixun Lan <dlan at gentoo.org> wrote:
[......]
>> +#define to_spacemit_gpio_bank(x) container_of((x), struct spacemit_gpio_bank, gc)
>> +
>> +struct spacemit_gpio;
>> +
>> +struct spacemit_gpio_bank {
>> +       struct gpio_chip                gc;
>> +       struct spacemit_gpio            *sg;
>> +       void __iomem                    *base;
>> +       u32                             index;
>> +       u32                             irq_mask;
>> +       u32                             irq_rising_edge;
>> +       u32                             irq_falling_edge;
>> +};
>> +
>> +struct spacemit_gpio {
>> +       struct  device                  *dev;
>> +       struct  spacemit_gpio_bank      sgb[NR_BANKS];
>> +};
> Please don't use tabs in struct definitions.

Why not?I see 
https://www.kernel.org/doc/html/latest/process/maintainer-tip.html#struct-declarations-and-initializers

[......]


> _______________________________________________
> linux-riscv mailing list
> linux-riscv at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv



More information about the linux-riscv mailing list