Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp612038pxk; Fri, 11 Sep 2020 16:09:06 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxpRSAruegrB2c10A1DlsU12cU4XXdViMy4+8KqC6of5j75u6o1wH2as6WSMEUBF+dghO// X-Received: by 2002:aa7:de82:: with SMTP id j2mr5310758edv.3.1599865746470; Fri, 11 Sep 2020 16:09:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1599865746; cv=none; d=google.com; s=arc-20160816; b=qSNJqfUb1+bETO+8N1+VjjSzFE78WaOBx5wqI//QAfETISEyrA6HCXke8tXLK7u/0L a+8wFitKKUFXaZODQ4IlAFo2lIp8BJVENuvu2lCCPxjspDrxCgXJ1PsH8vCCUqcwHplI gzK82sMK6foGHTd0Ztwc/JvF+4kQc+9tZYetUTmNU8WJbDNZ0r42o8j4Li7Mj1XgRfs0 PiZiK6c3fUDTCBOF2lH/CQRlOUtwjjtPIn402i1IwtK0+KbEeC9qUL0PtsbBSZUZ5mxJ h6zHd9mwnSqOIp8oKnp8aqtKEhku/vC19g6lotYAQn0cSrYjSEww9isdBu4iqBLp3vhh f/sg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:references:message-id :in-reply-to:subject:cc:to:from:date; bh=tspkX+5dYu3rO8RxSA4GXyeWoLWjnOLC7u8mtbvnSwY=; b=o2N++wTQs0Qia9XoElAQL7J8KwqT35S26192OXddyFCIeXQQsYkOnjGEvO/hl1bqMv kFDsDmDRGVvtnvdSqYJjQqeBcvt5uvZ8Nd8XapDtwXz7Se6srEMmKZ1z3pCceRdr9P30 cTZYCilOPusSZVFNa9eB8z5ZJFI6WalIPK1xT1xalaZLbELyq5kP/85MfwWRj41KevXS Mba94f7aMwprJ1xmuwOMNNr5seP+aExsl1e7KbJNhbS8+eOZZ6+KNfolT5kXFkLdrMEx kp9nOMCz7EH5Y5DyJ2p0sPCbOVLoUhI71YzIZHvXW9ItG/uI3LxVC6QQEKuicIyx2Xld pTRg== 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 mm8si2023843ejb.53.2020.09.11.16.08.43; Fri, 11 Sep 2020 16:09:06 -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 S1725915AbgIKXHb (ORCPT + 99 others); Fri, 11 Sep 2020 19:07:31 -0400 Received: from kvm5.telegraphics.com.au ([98.124.60.144]:40108 "EHLO kvm5.telegraphics.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725907AbgIKXHW (ORCPT ); Fri, 11 Sep 2020 19:07:22 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by kvm5.telegraphics.com.au (Postfix) with ESMTP id B58472ABF7; Fri, 11 Sep 2020 19:07:17 -0400 (EDT) Date: Sat, 12 Sep 2020 09:07:22 +1000 (AEST) From: Finn Thain To: Geert Uytterhoeven cc: "David S. Miller" , Bartlomiej Zolnierkiewicz , Joshua Thompson , linux-m68k , Linux Kernel Mailing List , linux-ide@vger.kernel.org Subject: Re: [PATCH] ide/macide: Convert Mac IDE driver to platform driver In-Reply-To: Message-ID: References: <00ee44fe6ecdce1c783c3cc3b1b9a62b498dcdb2.1597736545.git.fthain@telegraphics.com.au> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 11 Sep 2020, Geert Uytterhoeven wrote: > > Sorry, I completely missed that the Baboon case registers a > "pata_platform" device instead of a "macide" device. Hence please > ignore my comments related to that. Sorry for the noise. > No problem. That misunderstanding got me thinking about implications stemming from my patch that I may have overlooked. Here's what I found. 1) Your presumption that the old macide driver would keep supporting all variants does make sense, as that would delay disruption for as long as possible (i.e. for as long as the IDE subsystem remains). However, if my patch does not get merged until 2021, that disruption would not arrive earlier than promised by commit 7ad19a99ad431 ("ide: officially deprecated the legacy IDE driver"). 2) My patch omitted a mac_defconfig update that would enable an alternative driver for the Baboon case. I will remedy this in v2. 3) It turns out that the Debian/m68k kernel config has CONFIG_BLK_DEV_PLATFORM=m. This will work fine with this patch. (I assume that Debian developers will replace CONFIG_BLK_DEV_PLATFORM with CONFIG_PATA_PLATFORM prior to the removal of the IDE subsystem next year.)