| Message ID | 60cbfd566777a6d6e3664d7f29fa91424716c873.1693535514.git.tanyuan@tinylab.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add dependencies of POWER_RESET for MIPS Malta, x86, and PowerMac | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 8 jobs. |
| snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 8 jobs. |
| snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
| snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 6 jobs. |
| snowpatch_ozlabs/github-powerpc_kernel_qemu | success | Successfully ran 23 jobs. |
Le 01/09/2023 à 04:43, Yuan Tan a écrit : > PowerMac's power off depends on ADB_CUDA to work. Enable it when > POWER_RESET is set for convenience. > > Suggested-by: Zhangjin Wu <falcon@tinylab.org> > Signed-off-by: Yuan Tan <tanyuan@tinylab.org> > --- > arch/powerpc/platforms/powermac/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/powerpc/platforms/powermac/Kconfig b/arch/powerpc/platforms/powermac/Kconfig > index 130707ec9f99..9e633d7e8367 100644 > --- a/arch/powerpc/platforms/powermac/Kconfig > +++ b/arch/powerpc/platforms/powermac/Kconfig > @@ -2,6 +2,7 @@ > config PPC_PMAC > bool "Apple PowerMac based machines" > depends on PPC_BOOK3S && CPU_BIG_ENDIAN > + select ADB_CUDA if POWER_RESET ADB_CUDA depends on !PPC_PMAC64. What will happen if PPC_PMAC64 is selected ? > select MPIC > select FORCE_PCI > select PPC_INDIRECT_PCI if PPC32
Hi, On 9/1/2023 2:10 PM, Christophe Leroy wrote: > > Le 01/09/2023 à 04:43, Yuan Tan a écrit : >> PowerMac's power off depends on ADB_CUDA to work. Enable it when >> POWER_RESET is set for convenience. >> >> Suggested-by: Zhangjin Wu <falcon@tinylab.org> >> Signed-off-by: Yuan Tan <tanyuan@tinylab.org> >> --- >> arch/powerpc/platforms/powermac/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/powerpc/platforms/powermac/Kconfig b/arch/powerpc/platforms/powermac/Kconfig >> index 130707ec9f99..9e633d7e8367 100644 >> --- a/arch/powerpc/platforms/powermac/Kconfig >> +++ b/arch/powerpc/platforms/powermac/Kconfig >> @@ -2,6 +2,7 @@ >> config PPC_PMAC >> bool "Apple PowerMac based machines" >> depends on PPC_BOOK3S && CPU_BIG_ENDIAN >> + select ADB_CUDA if POWER_RESET > ADB_CUDA depends on !PPC_PMAC64. > > What will happen if PPC_PMAC64 is selected ? > >> select MPIC >> select FORCE_PCI >> select PPC_INDIRECT_PCI if PPC32 I didn't notice that ADB_CUDA depends on !PPC_PMAC64. In the case where PPC_PMAC64 is set, ADB_CUDA indeed should not be enabled. Thank you for pointing that out. I will fix it in v3 and check again in x86 and mips :)
diff --git a/arch/powerpc/platforms/powermac/Kconfig b/arch/powerpc/platforms/powermac/Kconfig index 130707ec9f99..9e633d7e8367 100644 --- a/arch/powerpc/platforms/powermac/Kconfig +++ b/arch/powerpc/platforms/powermac/Kconfig @@ -2,6 +2,7 @@ config PPC_PMAC bool "Apple PowerMac based machines" depends on PPC_BOOK3S && CPU_BIG_ENDIAN + select ADB_CUDA if POWER_RESET select MPIC select FORCE_PCI select PPC_INDIRECT_PCI if PPC32
PowerMac's power off depends on ADB_CUDA to work. Enable it when POWER_RESET is set for convenience. Suggested-by: Zhangjin Wu <falcon@tinylab.org> Signed-off-by: Yuan Tan <tanyuan@tinylab.org> --- arch/powerpc/platforms/powermac/Kconfig | 1 + 1 file changed, 1 insertion(+)