From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 78CA1C4361B for ; Sun, 13 Dec 2020 17:06:17 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 374402313B for ; Sun, 13 Dec 2020 17:06:17 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 374402313B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=VD9GqAamrP6nwqge45oEnPyhJUllUwk693fyrOo50vc=; b=ZlgjUMQVECJnEYCClxMYvRSrE 8zMT+dWgqNUgztBYyLeoUQB1YaXtAIu8Bp+FP3npFsHSaemmzWtCS9yKw29MukAIxSmxxOhjlicMW dDXAkC9XS9sK9vECucwKLf5jDrrot0as7Y+E0rbSRDelUwBhdNLKqidpsQA8vv9iFnQjSBcjtWceM x+zUEJlh2SwqyzHVII8W1zvGsZzLWd+5rD7A+ouboJTyLHzjZq4s44g/n4qBCaT7j6WLrAhpznoDI GjCfaumlQNVDTG9wDnEx6O0V/GzAULQgRroELgoCaOEjoGA1X8/opL9Gy5otX4SKvZrzkQ8jd6qSU aaPqeoqYA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1koUny-0003qV-1u; Sun, 13 Dec 2020 17:05:06 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1koUnu-0003pt-Hj for linux-arm-kernel@lists.infradead.org; Sun, 13 Dec 2020 17:05:03 +0000 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 B2B862313B; Sun, 13 Dec 2020 17:04:58 +0000 (UTC) Date: Sun, 13 Dec 2020 17:04:53 +0000 From: Jonathan Cameron To: Andy Shevchenko Subject: Re: [PATCH 1/6] iio:pressure:ms5637: switch to probe_new Message-ID: <20201213170453.2de3a963@archlinux> In-Reply-To: References: <20201209234857.1521453-1-alexandre.belloni@bootlin.com> <20201209234857.1521453-2-alexandre.belloni@bootlin.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201213_120502_707661_D6C15FC9 X-CRM114-Status: GOOD ( 17.32 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree , Alexandre Belloni , Lars-Peter Clausen , linux-iio , Linux Kernel Mailing List , Peter Meerwald-Stadler , linux-arm Mailing List Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sat, 12 Dec 2020 15:26:17 +0200 Andy Shevchenko wrote: > On Thu, Dec 10, 2020 at 2:01 AM Alexandre Belloni > wrote: > > > > Switch to the modern i2c probe_new callback and drop the i2c_device_id > > array. > > First part is okay. > The second is interesting. It depends if we would like to keep a > possibility to instantiate devices from user space (strictly speaking > it's an ABI breakage). > We've also been bitten by this recently via greybus which does it's instantiations using the i2c_device_id table as I understand it. That's resulted in us reverting a few specific cases where we'd done pretty much what you have done here. So drop that part of the change. Thanks, Jonathan _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel