[PATCH v2 09/11] m68k: stmark2: use ioport.h macros for resources
Andy Shevchenko
andriy.shevchenko at intel.com
Wed May 13 13:18:41 PDT 2026
On Wed, May 13, 2026 at 11:14:33AM +0200, Angelo Dureghello wrote:
> Align also other resource declaration using DEFINE_RES_.
DEFINE_RES_*()
...
> static struct resource dspi_spi0_resource[] = {
> - [0] = {
> - .start = MCFDSPI_BASE0,
> - .end = MCFDSPI_BASE0 + 0xFF,
> - .flags = IORESOURCE_MEM,
> - },
> - [1] = {
> - .start = 12,
> - .end = 13,
> - .flags = IORESOURCE_DMA,
> - },
> - [2] = {
> - .start = MCF_IRQ_DSPI0,
> - .end = MCF_IRQ_DSPI0,
> - .flags = IORESOURCE_IRQ,
> - },
> + DEFINE_RES_MEM(MCFDSPI_BASE0, 0x100),
> + DEFINE_RES_DMA(12),
> + DEFINE_RES_DMA(13),
> + DEFINE_RES_IRQ(MCF_IRQ_DSPI0),
> };
This conversion drops hard indices, would it be a problem (for example,
some code modifies the fields based on the [hard coded] index...)?
If not, perhaps good to mention in the commit message and move IRQ to be the
second one?
--
With Best Regards,
Andy Shevchenko
More information about the linux-arm-kernel
mailing list