Messages in this thread |  | | From | "Lad, Prabhakar" <> | Date | Mon, 19 Dec 2022 11:21:01 +0000 | Subject | Re: [PATCH v5 4/6] riscv: mm: dma-noncoherent: Pass direction and operation to ALT_CMO_OP() |
| |
Hi Conor,
Thank you for the review.
On Sat, Dec 17, 2022 at 8:52 PM Conor Dooley <conor@kernel.org> wrote: > > Hey Prabhakar, > > On Mon, Dec 12, 2022 at 11:55:03AM +0000, Prabhakar wrote: > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > Pass direction and operation to ALT_CMO_OP() macro. > > > > Vendors might want to perform different operations based on the direction > > and callbacks (arch_sync_dma_for_device/arch_sync_dma_for_cpu/ > > arch_dma_prep_coherent) so to handle such cases pass the direction and > > operation to ALT_CMO_OP() macro. This is in preparation for adding errata > > for the Andes CPU core. > > This patch seems to break the build on top of the most recent > linux-next: > ......./stuff/linux/arch/riscv/mm/pmem.c:13:53: error: too few arguments provided to function-like macro invocation > ALT_CMO_OP(clean, addr, size, riscv_cbom_block_size); > ^ > /stuff/linux/arch/riscv/include/asm/errata_list.h:127:9: note: macro 'ALT_CMO_OP' defined here > #define ALT_CMO_OP(_op, _start, _size, _cachesize, _dir, _ops) \ > ^ > .. CC block/partitions/sgi.o > .+...+/stuff/linux/arch/riscv/mm/pmem.c:13:2: error: use of undeclared identifier 'ALT_CMO_OP' > ALT_CMO_OP(clean, addr, size, riscv_cbom_block_size); > ^ > /stuff/linux/arch/riscv/mm/pmem.c:19:53: error: too few arguments provided to function-like macro invocation > ALT_CMO_OP(inval, addr, size, riscv_cbom_block_size); > ^ > /stuff/linux/arch/riscv/include/asm/errata_list.h:127:9: note: macro 'ALT_CMO_OP' defined here > #define ALT_CMO_OP(_op, _start, _size, _cachesize, _dir, _ops) \ > ^ > ........... AR lib/math/built-in.a > ./stuff/linux/arch/riscv/mm/pmem.c:19:2: .error: use of undeclared identifier 'ALT_CMO_OP' > ALT_CMO_OP(inval, addr, size, riscv_cbom_block_size); > ^ > ..4 errors generated. > > The pmem stuff is new so that'd be why it has not come up before. > > (FWIW, clang allmodconfig) > Yes I need to rebase my changes on Palmer's branch.
Cheers, Prabhakar
|  |