Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp3147142pxb; Sun, 31 Jan 2021 05:58:23 -0800 (PST) X-Google-Smtp-Source: ABdhPJyyYf03bxq0b0qNXshO8QT/5R5G6Au/oPfFujTBtib6QxGkGzXmX88zSDcsyS0y97CsduA7 X-Received: by 2002:a05:6402:b27:: with SMTP id bo7mr6263840edb.372.1612101502911; Sun, 31 Jan 2021 05:58:22 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1612101502; cv=none; d=google.com; s=arc-20160816; b=P1i6ZW/SbTdUixKraVG3ucesTvdFClORbIimycPRNM+yy3GZV1rQx6NxkaAtIbjNNa JwhUit3x7bNqEuoZ9/Ro26zp8axuL+vpgqFf26SBxNfEMx7+DLhcyPrkLhx5UWZgycfc G8wUQse5zzzJM/5NQtHhcA+Sn77FWQLUcpmrmyiCLSixHZWncXZAiXoLuP1ZqHSUuI/f 5O9er9ZGKcwrEGidJ62wiFxWWgD2tZksrDgeJiVvNA29TAz4AFsg+Ik4IjB7sYOOTslH 9lq0NgwBI8/+s8zLFkNsGFYEKQkZTNz6pod8m7lHA4LAq/+TMtDKtoHL9i1zi7rpcJBU UvUw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date; bh=G+wqDdrTXwLi5/EVMn/8Quh0whWAFQV/3wb8+CCRLvw=; b=tWufxGOJr7fItEwCWJELkxv6Zy4YDKpqErX07k1l5tprxNKtaLrezLlCf6D/G9oPVf vgIQ7dMyMs/W8YGzPZtd8eaEHHb+b90S09W3yZDyIDrzH1h2vRuPkxE2Xt4DCg97UUKw 2QslSW92SipoDNl0YlcOB8tQvvl117SKj4l82uo6h/IvOtpYVPXNRKtW71qtLC2isFlq 0aLZA5AeiWKOCdNw80eyQ1i3bQApaAa8n0kkwl+WM5fPaSrg1Sz2AZ465mqa5t33l0rU Nva1TKqVoV16T1CEkQoecNEjPorMTS9WZebKK3X35p5huWzIyiHE8iuYcmFTxaVikZbi Md3A== 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 bu13si7892208ejb.588.2021.01.31.05.57.59; Sun, 31 Jan 2021 05:58:22 -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 S231923AbhAaNyD (ORCPT + 99 others); Sun, 31 Jan 2021 08:54:03 -0500 Received: from mail.kernel.org ([198.145.29.99]:50510 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231137AbhAaLDA (ORCPT ); Sun, 31 Jan 2021 06:03:00 -0500 Received: from archlinux (cpc108967-cmbg20-2-0-cust86.5-4.cable.virginm.net [81.101.6.87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 9791B64E21; Sun, 31 Jan 2021 11:00:25 +0000 (UTC) Date: Sun, 31 Jan 2021 11:00:21 +0000 From: Jonathan Cameron To: Stephen Boyd Cc: Gwendal Grignou , linux-kernel , linux-iio , Dmitry Torokhov , Benson Leung , Guenter Roeck , Douglas Anderson Subject: Re: [PATCH 3/3] iio: proximity: Add a ChromeOS EC MKBP proximity driver Message-ID: <20210131110021.46a700e4@archlinux> In-Reply-To: <161161826068.76967.15170332425672601158@swboyd.mtv.corp.google.com> References: <20210122225443.186184-1-swboyd@chromium.org> <20210122225443.186184-4-swboyd@chromium.org> <20210124173820.4528b9c9@archlinux> <161160076017.76967.4467861058817044169@swboyd.mtv.corp.google.com> <161161826068.76967.15170332425672601158@swboyd.mtv.corp.google.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ... > > > > > > +static const struct of_device_id cros_ec_proximity_of_match[] = { > > > > > > + { .compatible = "google,cros-ec-proximity" }, > > > > > > + {} > > > > > > +}; > > > > > > +MODULE_DEVICE_TABLE(of, cros_ec_proximity_of_match); > > > > > > +#endif > > > > > > + > > > > > > +static struct platform_driver cros_ec_proximity_driver = { > > > > > > + .driver = { > > > > > > + .name = "cros-ec-proximity", > > > > > > + .of_match_table = of_match_ptr(cros_ec_proximity_of_match), > > > > Add a ACPI match table to match. > > > > > > I don't have an ACPI system in hand. What should the ACPI table look > > > like? Can ACPI use the of_match_table logic? > > AFAIK, ACPI uses .acpi_match_table, see > > drivers/iio/magnetometer/ak8975.c for a simple example. > > Ok. I'm leaning towards punting on this. I don't have an ACPI system to > test and I don't know what the ACPI match table should have in it. If > you can tell me what to put in the acpi_match_table then I can add it. Unless we have a known ACPI ID don't add support. We let a few of those in in the past, (mostly because I didn't know better) and they are a pain to clean up, particularly as there may be platforms that started using them because the Linux driver supported them. Jonathan