Received: by 2002:a05:6a10:6744:0:0:0:0 with SMTP id w4csp4500461pxu; Mon, 12 Oct 2020 22:58:15 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwQZsO3VIKHcZFVqwO2T5PcTpY59TCgMUmOR8nBCCxHIZIYJEKyjFb2WZwNlBxIlSmefTwf X-Received: by 2002:aa7:c2d8:: with SMTP id m24mr1105663edp.90.1602568695156; Mon, 12 Oct 2020 22:58:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1602568695; cv=none; d=google.com; s=arc-20160816; b=XNkG5kZ9AXmIyiUeKJmx3zviaRIleNLNKXUXjvPGT0XZiaw6fQS4LQhGNZ3GtD0rZy sFJ8ekvyrZ0rYQnP9ViBCWiGSOg+JWwXnQCDGx8n7ouWeGkeIq9Rzg3fQIW3r6qG4hI6 3hfQ+yTlKcdvapRwPjlU0uPzD/vBq4Iv/eO5UQCltR2xSiVG57wg1EIUBrULWh9pXdL/ KpniJLmnm92npSn0rkUhOcnkwlTFO2FSwZ58LF6BlRyxVc722T20muArqM3SQDpCqE19 M6TbnfgTNhis9OsO8EstjptFn+l177/oAGF5ABiNbIMvHMBWLvBaqYSrl55DW4m0ScIx H6gw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version; bh=beueufrXvWkZ/KQcrxBvRIlUQpnpq8jxDfg8o7R6yww=; b=ZZpMlMDryW/2/xlzn+m5fGcu72gKbDTV0F+rSba5m4Yq1fkaJzyRQoU8/eO51f5BLE D5YP7Kz2all6CrD9HjruQxY+1mSplM1c7gRi8zJTxx1S99B7HeZXjTg3XdKpJB2JeONv bWS++dgAFb3g9rYrk10v6KRZpj+KEogHnBuB3bOGv1vNrkwHYiEyLFa8sM97zQC73sRW HRwIuI13PW9xyw8P1yt3DiyOE0PIhz4DeGH2WfHs2zPyz+pf9urB/Jv1iQBOpRpFNy1z mVnDiKExIX6jTqNOlQ9p8GQG7xqlXbnthOovhB/wSyfZbRsyvhx6a+zMohS5Sx1mn9Md Ea3w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q9si14347923edn.539.2020.10.12.22.57.53; Mon, 12 Oct 2020 22:58:15 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389517AbgJLPbJ (ORCPT + 99 others); Mon, 12 Oct 2020 11:31:09 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:42081 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388527AbgJLPbJ (ORCPT ); Mon, 12 Oct 2020 11:31:09 -0400 Received: from mail-qk1-f182.google.com ([209.85.222.182]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.145]) with ESMTPSA (Nemesis) id 1M5PRT-1kRARb1wLF-001QUy; Mon, 12 Oct 2020 17:31:06 +0200 Received: by mail-qk1-f182.google.com with SMTP id q63so18080040qkf.3; Mon, 12 Oct 2020 08:31:06 -0700 (PDT) X-Gm-Message-State: AOAM530XcP7wb8zAcjMY7SPnyaM1gM3y1r1kgv3/mAG9zNvp2uYk3VgI ZUWvozBqOR+GpDY02cOfMoEU0Cdpq6sCrfikNZo= X-Received: by 2002:a05:620a:2ee:: with SMTP id a14mr5630220qko.286.1602516665130; Mon, 12 Oct 2020 08:31:05 -0700 (PDT) MIME-Version: 1.0 References: <20201008154651.1901126-1-arnd@arndb.de> <20201008154651.1901126-9-arnd@arndb.de> In-Reply-To: From: Arnd Bergmann Date: Mon, 12 Oct 2020 17:30:49 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 08/13] m68k: m68328: use legacy_timer_tick() To: Geert Uytterhoeven Cc: Linux Kernel Mailing List , Russell King , Tony Luck , Fenghua Yu , Greg Ungerer , Finn Thain , Philip Blundell , Joshua Thompson , Sam Creasey , "James E.J. Bottomley" , Helge Deller , Thomas Gleixner , Daniel Lezcano , John Stultz , Stephen Boyd , Linus Walleij , "linux-ia64@vger.kernel.org" , Parisc List , linux-m68k , Linux ARM Content-Type: text/plain; charset="UTF-8" X-Provags-ID: V03:K1:DJJD98gQfMORp0800O7Ew01mrOIiBaWtMDU5TgDNGrJZmMB1AqZ NqPQk/mwzxVPcHmEjj6zegLLQNVhCmGchJktO77E/kRqQwwAQc7nVdyPFjLExbBQtfO21nv sL8xXmEEuCOKG6g967og41FVYdyYLmlP8qsBNcIJyepNG+zBALDzlOb73dbyrDBXnLvPa0v 56lM8ztGjzBoTBYeF30Ow== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:ggoW9tp1yI4=:Rjwwz7JmODT6i2Ujg+riPF EDSaovwE46D4knVSyExkaQiy7me2/AZD89AKQlZGgv1b/07YvCkvVAzzM2S2/0qhNCnd3wg+E Gwe8FyelBilvrjEDDJxpOoUwKqhfGK/WWyy3NtR+qAVO23UWiCTP5VXFW/IconwKYAiuVjPiu gugS7X6MSJ/e/VVR0KIVIhUh0rPeljjjT2pBC3+AxPEkifadT7jvR7IhrZaJcMHu0w5/MClA8 XRCH0mYZiPJ+54QIYwY3pHTzUozak4HooOrdMkXB9q/mTpmgfCfw8Ys3mk4UTsLkARDx8mZgX MJiSLF3jALBa8I3fc1sF+P9LXSAQVmTFtZKX5iCDXaI5JGPMHZox4FrfFissjjUX3PDujrvfX m/pCbu7SYJfk1BbnDFhI+qY94v6jvlFDsm8zNrRIMh+76gMekTdaC0442i3WWQ4ZUvzztu0AP uJq+PK6yCOma555JDOewlGWmclTp/S+LTDfRLazR0r+ey4YWa8wNUmXoN9HsTNPT1PkBvd91L 3ZU+VzcMRBkb2NKP1PDpSOywicC0FeA1ftd4tlg3F1/1HAiize86c7OJJevXTouaeICmTktHS S708f3Eb0NNHVfrjXYaCsNLiNUFaYq47vP5GF34nyjjecb6vjuS+PuDE/OK6ISSqTTlypajzd Cfqv198bKjP9frkk2mH6e/ICk8c7zH+htH6YXE3mj2JK+6gw6/MmXBkF0O0poyXcsgigiK2+2 pe/pnqbreffCQMIy+8ov/Ns47zTYrmfW4MR0lf+Aksh903IAABIcB8hpqEZ2AwXvXKF+yg1KY dXaKVgLHvtc/uQp/v7/w9RcFZXrnfk/LtZDV8BTyAa3ohsnJnMChpxXzc+GEhW36ms3q+XdJi 0qjk7GfEinH+0ug6vdbOPM7ssfLa31kAXskhiyCNui6Z1OZnQKow5162GleqtG3VDMqp8q64o ls6bi2zpB3ds4N6pXoMRgSMZIjdybd7Ovd8cCJLYNyrpa7mwYJEFa Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Oct 12, 2020 at 3:15 PM Geert Uytterhoeven wrote: > > Given this feature is SoC-specific, not platform-specific, perhaps > it makes sense to move the selects to the M68{,EZ,VZ}328 symbols? > > Regardless: > Reviewed-by: Geert Uytterhoeven Ok, folded in the change blow, using one less line. I couldn't figure out whether it should just be part of the CONFIG_M68000 instead, which doesn't appear to have any machine support by itself. The dragonball CPU configuration looks really odd, because you have to build exactly one of M68{,EZ,VZ}328 into the kernel to get a successful compilation, while Kconfig allows many other combinations. Arnd diff --git a/arch/m68k/Kconfig.cpu b/arch/m68k/Kconfig.cpu index 322a35ef14c6..648054d4f860 100644 --- a/arch/m68k/Kconfig.cpu +++ b/arch/m68k/Kconfig.cpu @@ -104,6 +104,7 @@ config M68060 config M68328 bool "MC68328" depends on !MMU + select LEGACY_TIMER_TICK select M68000 help Motorola 68328 processor support. @@ -111,6 +112,7 @@ config M68328 config M68EZ328 bool "MC68EZ328" depends on !MMU + select LEGACY_TIMER_TICK select M68000 help Motorola 68EX328 processor support. @@ -118,6 +120,7 @@ config M68EZ328 config M68VZ328 bool "MC68VZ328" depends on !MMU + select LEGACY_TIMER_TICK select M68000 help Motorola 68VZ328 processor support. diff --git a/arch/m68k/Kconfig.machine b/arch/m68k/Kconfig.machine index 0ff9338b958e..e3c835440d9a 100644 --- a/arch/m68k/Kconfig.machine +++ b/arch/m68k/Kconfig.machine @@ -146,7 +146,6 @@ config PILOT config PILOT3 bool "Pilot 1000/5000, PalmPilot Personal/Pro, or PalmIII support" depends on M68328 - select LEGACY_TIMER_TICK select PILOT help Support for the Palm Pilot 1000/5000, Personal/Pro and PalmIII. @@ -160,21 +159,18 @@ config XCOPILOT_BUGS config UCSIMM bool "uCsimm module support" depends on M68EZ328 - select LEGACY_TIMER_TICK help Support for the Arcturus Networks uCsimm module. config UCDIMM bool "uDsimm module support" depends on M68VZ328 - select LEGACY_TIMER_TICK help Support for the Arcturus Networks uDsimm module. config DRAGEN2 bool "DragenEngine II board support" depends on M68VZ328 - select LEGACY_TIMER_TICK help Support for the DragenEngine II board.