Re: [PATCH v2 4/8] PCI/AER: Rename struct aer_stats to aer_report

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 17, 2025 at 3:29 AM Karolina Stolarek
<karolina.stolarek@xxxxxxxxxx> wrote:
>
> On 14/02/2025 03:35, Jon Pan-Doh wrote:
> > -     dev->aer_stats = kzalloc(sizeof(struct aer_stats), GFP_KERNEL);
> > +     dev->aer_report = kzalloc(sizeof(struct aer_report), GFP_KERNEL);
>
> The rename brings back a checkpatch warning (to use
> sizeof(*dev->aer_report)). If you feel like it, you can fix it.

Huh. Similar to my other reply[1], checkpatch isn't showing any warnings.

Maybe a divergence in checkpatch version? Mine is 0.32 (Pulled from
pci/aer branch commit: 28d3871db7ef8ad0112f195c48a72d8638af89d1).

[1] https://lore.kernel.org/linux-pci/CAMC_AXVgYegnfc-vyKuxZS-Ck=aCJ95=HqdYNraVv99kXxw1QA@xxxxxxxxxxxxxx/

Thanks,
Jon





[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux