Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp239248pxb; Wed, 18 Nov 2020 03:19:54 -0800 (PST) X-Google-Smtp-Source: ABdhPJxTS1KhHmHsdyhRWrJkritJh42e+voRTzmFKAy4MKeKaHBgmAXE/DlsWZYoP230NiLGlX5k X-Received: by 2002:a17:906:c82c:: with SMTP id dd12mr5969740ejb.321.1605698394231; Wed, 18 Nov 2020 03:19:54 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1605698394; cv=none; d=google.com; s=arc-20160816; b=OXU5qYis4Rdq9h7IgDOcoaW7zrNUK/1qQJZL5Hs4UhrOn8j+hEPEDQD+w0+/a4k4F+ 48jFp6j2w5anjSa63K3OrGmpCbx2e07ejEwCJ3TyYG3qQZK5c+jluN0FSPZW2CrwFs5b 5AZVtt2eJFB3XXZLwuaJ0kSuJ3fu9jeEUGCZaNz1bintPIGZiZHIs5iOfTCtT05xmWJx A89mqx/hgf9tqJR9h5RFgmo/zEgAfoEE5Jzh5zgyMp952zumtSCQGF4j2AVO+YCtS/xr 7UDzge1TlLDyGfcJ7mTAc9uMwFz33FhL+we4lnL7bwqD5ZgDYXb9JDIrWdEMnQYZtjKx zfWQ== 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=D8wT37MkPFkyxY6qYH5+6Fkhw2mpanFwYZDumxMMKKk=; b=kQBme80vQmpZ96XHtmL1icIOIHTZy2s6JzWwGRz9oFyK1KfE+fxX4Et6XMh2PLQD+P fzWugCBt74t6tWhFM73t1o1pTdyh/LQ1+n4X1jXuXvWO4Qt1Zq9OtQ1R2ifDhfJtO9bQ 7vLIyKrKsK4I0eIjno3W24Tp4Zz9IG28lLhRWDirSsZP+QW2hnJkkRBOYx6wq3FnCLtL VrS7a/Vnp6O3x4P0ynUPP65kYws/grmCi/0XYMo+M8KuUvHcACx2mW/JPUFmQSi6ZzS5 LbqN6NTIwl1GByynCS67/Lyx0KnHGuaEBAbCCrr2dCSW6eWNqYlhXja+ElEm7Y155J1J kHBQ== 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id fx4si15033526ejb.449.2020.11.18.03.19.31; Wed, 18 Nov 2020 03:19:54 -0800 (PST) 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727736AbgKRLPK (ORCPT + 99 others); Wed, 18 Nov 2020 06:15:10 -0500 Received: from mail-oi1-f193.google.com ([209.85.167.193]:37552 "EHLO mail-oi1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726156AbgKRLPJ (ORCPT ); Wed, 18 Nov 2020 06:15:09 -0500 Received: by mail-oi1-f193.google.com with SMTP id m17so1791637oie.4; Wed, 18 Nov 2020 03:15:09 -0800 (PST) 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=D8wT37MkPFkyxY6qYH5+6Fkhw2mpanFwYZDumxMMKKk=; b=HyHYGweC6n65p+vjQ8/8miLZYweHigIBz4YeORsAnKGEDjb452OSKvcYNxHXy1irjb hXFSzv1aZ8usyQsRJzDyuhWX9Vb9hA8ttiY0EX89XSYccfCS3TsK+Ub1KJL8mnPeHUaz cSVp8AfSzbq1bKTxx2CRAvNqFMde6JcfVy20IYZO4iWDvsUrL+pOcnppd+4tu3bXDEdo B97MI/Xf2LF1AvSzWL2Kr3XFRcw5cbaiHHy7sYFa5qJXwqGHzqghI+WimeUcyfoBpVQp uHDdE1gAQSXPYC6yc/N0l1RP1v8SCI+0f0gOQ5nwvMyhaHTcNKwhrCyTkG2lz/obF5O5 TGVw== X-Gm-Message-State: AOAM530BapPIJyxlarGunAZBc1jK9QuAlpvCaP/HKkA84OnSf0ZvHS2d gxEyJBuLRpQdVCgc3n1FW9D9gTUSmkx9a5Y3t8g= X-Received: by 2002:aca:c4c9:: with SMTP id u192mr2294643oif.69.1605698108951; Wed, 18 Nov 2020 03:15:08 -0800 (PST) MIME-Version: 1.0 References: <20201111054356.793390-1-ben.widawsky@intel.com> <20201111054356.793390-2-ben.widawsky@intel.com> In-Reply-To: From: "Rafael J. Wysocki" Date: Wed, 18 Nov 2020 12:14:56 +0100 Message-ID: Subject: Re: [RFC PATCH 1/9] cxl/acpi: Add an acpi_cxl module for the CXL interconnect To: Dan Williams Cc: "Rafael J. Wysocki" , Ben Widawsky , linux-cxl@vger.kernel.org, Linux Kernel Mailing List , Linux PCI , ACPI Devel Maling List , Ira Weiny , Vishal Verma , "Kelley, Sean V" , Bjorn Helgaas , "Rafael J . Wysocki" Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 17, 2020 at 10:45 PM Dan Williams wrote: > > On Tue, Nov 17, 2020 at 6:33 AM Rafael J. Wysocki wrote: > [..] > > > +static struct acpi_driver acpi_cxl_driver = { > > > > First of all, no new acpi_driver instances, pretty please! > > > > acpi_default_enumeration() should create a platform device with the > > ACPI0017 ID for you. Can't you provide a driver for this one? > > > > Ah, yes, I recall we had this discussion around the time the ACPI0012 > NFIT driver was developed. IIRC the reason ACPI0012 remaining an > acpi_driver was due to a need to handle ACPI notifications, is that > the deciding factor? Sort of. In fact, a platform device driver can still handle ACPI notifications just fine, it just needs to install a notify handler for that. The cases when an acpi_driver is really needed are basically when creating the platform device during the enumeration is not desirable, like in the PCI or PNP cases (because they both create device objects of a different type to represent the "physical" device). It doesn't look like it is really needed for ACPI0012, but since it is there already, well ... > ACPI0017 does not have any notifications so it seems like platform driver is the way to go. Indeed.