| Message ID | 961d125558137f7cb960de65e5f71da5d299d3bc.1689792825.git.tjeznach@rivosinc.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream, archived |
| Headers | show |
| Series | Linux RISC-V IOMMU Support | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | success | Series has a cover letter |
| conchuod/tree_selection | success | Guessed tree name to be for-next at HEAD 471aba2e4760 |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 4 and now 4 |
| conchuod/verify_signedoff | fail | author Signed-off-by missing |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/build_rv64_clang_allmodconfig | success | Errors and warnings before: 9 this patch: 9 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 9 this patch: 9 |
| conchuod/build_rv32_defconfig | fail | Build failed |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 3 this patch: 3 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | warning | WARNING: Missing commit description - Add an appropriate one |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On Wed, Jul 19, 2023 at 12:33:46PM -0700, Tomasz Jeznach wrote: $subject: RISC-V: arch/riscv/config: enable RISC-V IOMMU support Please look at any other commits to the files you are touching and use a subject line that emulates them. In this case, try git log --oneline --no-merges -- arch/riscv/configs/ Same goes for the odd pattern in your driver patches. Also, the patch may be trivial, but you still need to sign off on it and provide a commit message. Thanks, Conor. > --- > arch/riscv/configs/defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig > index 0a0107460a5c..1a0c3b24329f 100644 > --- a/arch/riscv/configs/defconfig > +++ b/arch/riscv/configs/defconfig > @@ -178,6 +178,7 @@ CONFIG_VIRTIO_PCI=y > CONFIG_VIRTIO_BALLOON=y > CONFIG_VIRTIO_INPUT=y > CONFIG_VIRTIO_MMIO=y > +CONFIG_RISCV_IOMMU=y > CONFIG_SUN8I_DE2_CCU=m > CONFIG_SUN50I_IOMMU=y > CONFIG_RPMSG_CHAR=y > -- > 2.34.1 > > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv
On Wed, Jul 19, 2023 at 1:22 PM Conor Dooley <conor@kernel.org> wrote: > > On Wed, Jul 19, 2023 at 12:33:46PM -0700, Tomasz Jeznach wrote: > > $subject: RISC-V: arch/riscv/config: enable RISC-V IOMMU support > > Please look at any other commits to the files you are touching > and use a subject line that emulates them. In this case, try > git log --oneline --no-merges -- arch/riscv/configs/ > Same goes for the odd pattern in your driver patches. > > Also, the patch may be trivial, but you still need to sign off on it > and provide a commit message. > ack. added to-do the list for v2. Thank you, - Tomasz > Thanks, > Conor. > > > --- > > arch/riscv/configs/defconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig > > index 0a0107460a5c..1a0c3b24329f 100644 > > --- a/arch/riscv/configs/defconfig > > +++ b/arch/riscv/configs/defconfig > > @@ -178,6 +178,7 @@ CONFIG_VIRTIO_PCI=y > > CONFIG_VIRTIO_BALLOON=y > > CONFIG_VIRTIO_INPUT=y > > CONFIG_VIRTIO_MMIO=y > > +CONFIG_RISCV_IOMMU=y > > CONFIG_SUN8I_DE2_CCU=m > > CONFIG_SUN50I_IOMMU=y > > CONFIG_RPMSG_CHAR=y > > -- > > 2.34.1 > > > > > > _______________________________________________ > > linux-riscv mailing list > > linux-riscv@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-riscv
On 19/07/2023 23:07, Tomasz Jeznach wrote: > On Wed, Jul 19, 2023 at 1:22 PM Conor Dooley <conor@kernel.org> wrote: >> >> On Wed, Jul 19, 2023 at 12:33:46PM -0700, Tomasz Jeznach wrote: >> >> $subject: RISC-V: arch/riscv/config: enable RISC-V IOMMU support >> >> Please look at any other commits to the files you are touching >> and use a subject line that emulates them. In this case, try >> git log --oneline --no-merges -- arch/riscv/configs/ >> Same goes for the odd pattern in your driver patches. >> >> Also, the patch may be trivial, but you still need to sign off on it >> and provide a commit message. >> > > ack. added to-do the list for v2. Please run checkpatch before sending. Best regards, Krzysztof
diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig index 0a0107460a5c..1a0c3b24329f 100644 --- a/arch/riscv/configs/defconfig +++ b/arch/riscv/configs/defconfig @@ -178,6 +178,7 @@ CONFIG_VIRTIO_PCI=y CONFIG_VIRTIO_BALLOON=y CONFIG_VIRTIO_INPUT=y CONFIG_VIRTIO_MMIO=y +CONFIG_RISCV_IOMMU=y CONFIG_SUN8I_DE2_CCU=m CONFIG_SUN50I_IOMMU=y CONFIG_RPMSG_CHAR=y