linux-kernel - Re: [PATCH 2/2] m68k: m68328: remove duplicate code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <ab3a33d0-7a3a-f12b-244d-24195c06f3fc@linux-m68k.org>
Date:   Mon, 19 Oct 2020 09:44:53 +1000
From:   Greg Ungerer <gerg@...ux-m68k.org>
To:     Arnd Bergmann <arnd@...db.de>,
        Geert Uytterhoeven <geert@...ux-m68k.org>
Cc:     linux-m68k@...ts.linux-m68k.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] m68k: m68328: remove duplicate code

Hi Arnd,

Overall looks good.


On 15/10/20 10:32 pm, Arnd Bergmann wrote:
[snip]
> diff --git a/arch/m68k/Kconfig.cpu b/arch/m68k/Kconfig.cpu
> index 694c4fca9f5d..a65ce7618232 100644
> --- a/arch/m68k/Kconfig.cpu
> +++ b/arch/m68k/Kconfig.cpu
> @@ -35,7 +35,7 @@ endchoice
>   if M68KCLASSIC
>   
>   config M68000
> -	bool "MC68000"
> +	bool
>   	depends on !MMU
>   	select CPU_HAS_NO_BITFIELDS
>   	select CPU_HAS_NO_MULDIV64
> @@ -102,21 +102,21 @@ config M68060
>   	  processor, say Y. Otherwise, say N.
>   
>   config M68328
> -	bool "MC68328"
> +	bool
>   	depends on !MMU
>   	select M68000
>   	help
>   	  Motorola 68328 processor support.
>   
>   config M68EZ328
> -	bool "MC68EZ328"
> +	bool
>   	depends on !MMU
>   	select M68000
>   	help
>   	  Motorola 68EX328 processor support.
>   
>   config M68VZ328
> -	bool "MC68VZ328"
> +	bool
>   	depends on !MMU
>   	select M68000
>   	help
> diff --git a/arch/m68k/Kconfig.machine b/arch/m68k/Kconfig.machine
> index 17e8c3a292d7..1851c66e8667 100644
> --- a/arch/m68k/Kconfig.machine
> +++ b/arch/m68k/Kconfig.machine
> @@ -136,14 +136,13 @@ config SUN3
>   
>   	  If you don't want to compile a kernel exclusively for a Sun 3, say N.
>   
> -endif # M68KCLASSIC
> -
>   config PILOT
>   	bool
>   
>   config PILOT3
>   	bool "Pilot 1000/5000, PalmPilot Personal/Pro, or PalmIII support"
> -	depends on M68328
> +	depends on !MMU
> +	select M68328

Given that M68328 depends on !MMU do you also need or want that here?

Regards
Greg