On Fri, 23 Feb 2024, Sam Ravnborg via B4 Relay wrote: > include/asm/parport.h is sparc64 specific. > Rename it to parport_64.h and use the generic version for sparc32. > > This fixed all{mod,yes}config build errors like: > > parport_pc.c:(.text):undefined-reference-to-ebus_dma_enable > parport_pc.c:(.text):undefined-reference-to-ebus_dma_irq_enable > parport_pc.c:(.text):undefined-reference-to-ebus_dma_register > > The errors occur as the sparc32 build references sparc64 symbols. > > Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx> > Cc: Andreas Larsson <andreas@xxxxxxxxxxx> > Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> > Cc: Maciej W. Rozycki <macro@xxxxxxxxxxx> > Closes: https://lore.kernel.org/r/20230406160548.25721-1-rdunlap@xxxxxxxxxxxxx/ > Fixes: 66bcd06099bb ("parport_pc: Also enable driver for PCI systems") > Cc: stable@xxxxxxxxxxxxxxx # v5.18+ > --- LGTM, it relies on SPARC never to enable ISA. Reviewed-by: Maciej W. Rozycki <macro@xxxxxxxxxxx> Tested-by: Maciej W. Rozycki <macro@xxxxxxxxxxx> # build-tested The other changes in this patch series address issues that do not appear with my ad-hoc SPARC test configuration, so I have no immediate way to verify them. Maciej