Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp54271pxk; Thu, 10 Sep 2020 23:32:01 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwq7gzrfrCUSuXkIF+1+YKOiLjoUNm71472n3/iToKplXOsMMQxqr8OcASL7dqh6GtT/F0P X-Received: by 2002:a17:906:6d92:: with SMTP id h18mr565161ejt.405.1599805921749; Thu, 10 Sep 2020 23:32:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1599805921; cv=none; d=google.com; s=arc-20160816; b=aWJ6pVAA+/FE8PzxVvF/H6wFQzICGpKw4Ouqea1q0vUialUwzfJylsdw2yvJHFJYDv LV28DeksKrgLWgHPXTpgmu3K2nwjKz42oti9MR+ikCk+VIYT2WUYdtkJFv5P18WJD/va uQY6mmS/X8lMEGFwjZ9ttgUCZr04oEdfsfcOk/TVIgbgtqvqlxiJG29W8sV7HEMImE9r 9Vx3qphho7HtJ7IProMPx+XwAtaNFlmfyQ2NM/+KBVlCstptnZ+UkVg6AU26HeQHQSaS GrKwvd8XVkSO0BJ2/pZG4UrmmBNd0Gv1naEfUufVowkvxx2aPCH78X6H/E8weCVC6XBQ taig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version; bh=XFz8rKiULsWlS4B3yTs1ahls2aSZLYJA1vMCjs+Zp4o=; b=CcKOKfuSI+wTzDUisQ2ij8q3ObFamibzqo5h7dcaI5lBqNL7pY4PW09xzmZWdFeczm lDhyVdhrFH+7fsT3nCM5nV2WmmJ3E61R/gEmCwR6Fdqfk2+Ac5cNNyirPH1K5w7T26Ik DbLh9sjjTjfskCCb8rj++jGiVt1YWjpHFJ2Na4ILjs04VDOJsgjlwSrSCi/UiKCSfmWP wRaVLScyTQ6c6KYGvDDpjCdqAwrBm7NjBUFFbagRF3jtw1EeFzE9KZHdLg140exvfWNd gqZpol2/XA1o5UbG/ttRJlp5J54vRM54ks6de4/ovb70UzuDpC050huXYmlQ9NjsfMex INWA== 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 lt3si667132ejb.687.2020.09.10.23.31.38; Thu, 10 Sep 2020 23:32:01 -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 S1725785AbgIKG3D (ORCPT + 99 others); Fri, 11 Sep 2020 02:29:03 -0400 Received: from mail-oi1-f194.google.com ([209.85.167.194]:33962 "EHLO mail-oi1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725782AbgIKG25 (ORCPT ); Fri, 11 Sep 2020 02:28:57 -0400 Received: by mail-oi1-f194.google.com with SMTP id n2so8473430oij.1; Thu, 10 Sep 2020 23:28:56 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=XFz8rKiULsWlS4B3yTs1ahls2aSZLYJA1vMCjs+Zp4o=; b=CDlnVg9im/AKCsNJZhVz6oRLyvmBDANp4/nZUYJXznMx9EAHOW12z5GeO7cTfF7QNs 3Z4tCCDzuTL2Wil1JNezI/wwPWdR5eLrlmP9qTjAVgO0+R8YQVHdZoFhQtNihFFp7VHY E499kWnRmrG4t130VE/tpkw7JzVcr7/Z77N1OCc4cK1omhF5khzbrVRx2qHGjyJiooY+ 8xtycnZUqL0mnVCoX74jiG70HuL2kvUYBNdwR1PAmegIBjL87P2gkUTFHbqihigDvKHi KuHQFENlk7QvXsVaKDnIMRfrJ6uzOWlMNpGe9umCsza0sMYIiowq6S+bSvkl7dZ+MlYS m9pg== X-Gm-Message-State: AOAM532fgqLkB/sUaSK3NJXdyxO282vw3oWibMPWvBXymSb2LC1YO9Cw AUwuzkR7qQH7pahsFR58bSPIJqzyk9q0aVKxmJM= X-Received: by 2002:aca:b742:: with SMTP id h63mr398999oif.148.1599805736227; Thu, 10 Sep 2020 23:28:56 -0700 (PDT) MIME-Version: 1.0 References: <00ee44fe6ecdce1c783c3cc3b1b9a62b498dcdb2.1597736545.git.fthain@telegraphics.com.au> In-Reply-To: From: Geert Uytterhoeven Date: Fri, 11 Sep 2020 08:28:45 +0200 Message-ID: Subject: Re: [PATCH] ide/macide: Convert Mac IDE driver to platform driver To: Finn Thain Cc: "David S. Miller" , Bartlomiej Zolnierkiewicz , Joshua Thompson , linux-m68k , Linux Kernel Mailing List , linux-ide@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Finn, On Fri, Sep 11, 2020 at 1:05 AM Finn Thain wrote: > On Thu, 10 Sep 2020, Geert Uytterhoeven wrote: > > On Thu, Sep 10, 2020 at 2:23 AM Finn Thain wrote: > > > > How does the driver know not to use the special port_ops after > > > > this change? > > > > > > > > > > The driver always uses the special port_ops after this change because it > > > no longer handles the MAC_IDE_BABOON case. That case is handled by either > > > > non-MAC_IDE_BABOON case? > > > > > drivers/ata/pata_platform.c or drivers/ide/ide_platform.c, depending on > > > .config. > > > > Ideally, we do need to differentiate, right? > > > > Sorry, I'm lost. > > The commit log explains that the macide driver is only intended to support > two of the three variants, because the generic drivers are already able to > support the third variant (Baboon). Stan confirmed this on his PB 190. > > But, IIUC, you seem to want macide to continue to support all three > variants (?) The existing code to implement that has an old bug that > reassigns d.port_ops when it is const. IMO, the const is correct because > macide should concern itself with mac hardware quirks and should not try > to mimic a generic driver by setting d.port_ops = NULL. Does that make > sense? 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. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds