Re: [PATCH v2 3/3] kconfig: add dependencies of POWER_RESET for PowerMac

2023-09-01 Thread Yuan Tan

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 
Signed-off-by: Yuan Tan 
---
   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 :)




Re: [PATCH v2 3/3] kconfig: add dependencies of POWER_RESET for PowerMac

2023-09-01 Thread Christophe Leroy


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 
> Signed-off-by: Yuan Tan 
> ---
>   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


[PATCH v2 3/3] kconfig: add dependencies of POWER_RESET for PowerMac

2023-08-31 Thread Yuan Tan
PowerMac's power off depends on ADB_CUDA to work. Enable it when
POWER_RESET is set for convenience.

Suggested-by: Zhangjin Wu 
Signed-off-by: Yuan Tan 
---
 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
select MPIC
select FORCE_PCI
select PPC_INDIRECT_PCI if PPC32
-- 
2.34.1