Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752033AbeANKvR (ORCPT + 1 other); Sun, 14 Jan 2018 05:51:17 -0500 Received: from mail.kernel.org ([198.145.29.99]:46402 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751287AbeANKvP (ORCPT ); Sun, 14 Jan 2018 05:51:15 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EDE0F21775 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jic23@kernel.org Date: Sun, 14 Jan 2018 10:51:10 +0000 From: Jonathan Cameron To: Javier Martinez Canillas Cc: Dmitry Mastykin , Andy Shevchenko , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Jacopo Mondi , Dan Carpenter , Rob Herring , linux-iio , Linux Kernel Mailing List Subject: Re: [PATCH v2] iio: adc: max9611: fix module auto-loading Message-ID: <20180114105110.14ec8c8f@archlinux> In-Reply-To: References: <1514556006-23293-1-git-send-email-mastichi@gmail.com> <20171229175449.760e734d@archlinux> <20180101095342.4d4d5325@archlinux> X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Wed, 10 Jan 2018 12:01:07 +0100 Javier Martinez Canillas wrote: > Hello Jonathan, > > On Mon, Jan 1, 2018 at 10:53 AM, Jonathan Cameron wrote: > > [snip] > > > > > I may well be missing some subtle detail of course having spent only a few > > minute looking at this! > > > > Your understanding is correct. This change has nothing to do with > module autoloading. > > The .probe_new callback is only used to avoid requiring an I2C device > ID table since the old .probe callback has a struct i2c_device_id as a > parameter and so requires a table even in OF (or ACPI) only drivers. OK, then let us revisit this patch once we have an answer on your one fixing the autoprobing more generally. Until we then we don't know whether we need to add the i2c table in or not whether this suffices. Jonathan > > > Jonathan > > Best regards, > Javier > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html