[PATCH 19/20] PCI/sysfs: Convert legacy I/O and memory attributes to static definitions

Krzysztof Wilczyński kwilczynski at kernel.org
Fri Apr 10 22:04:58 AEST 2026


Hello,

> > +static const struct bin_attribute pci_legacy_mem_sparse_attr = {
> > +	.attr = { .name = "legacy_mem_sparse", .mode = 0600 },
> > +	.size = 0x100000 << 5,
> 
> I suggest naming these legacy literals with defines. At least the mem one 
> could also use SZ_1M:
> 
> #define PCI_LEGACY_MEM_SIZE	SZ_1M

Sounds good.  Will do.

Thank you!

	Krzysztof


More information about the Linuxppc-dev mailing list