Date: Sat, 30 Mar 2013 09:43:26 +0100
From: Jean Delvare <khali@linux-fr.org>
To: Lars-Peter Clausen <lars@metafoo.de>
Cc: Wolfram Sang <wsa@the-dreams.de>, Ben Dooks <ben-linux@fluff.org>,
        linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
        Guan Xuetao <gxt@mprc.pku.edu.cn>,
        Mauro Carvalho Chehab <mchehab@redhat.com>,
        Ben Hutchings <bhutchings@solarflare.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Alan Cox <alan@linux.intel.com>,
        Nicolas Ferre <nicolas.ferre@atmel.com>,
        Aaro Koskinen <aaro.koskinen@iki.fi>, Shawn Guo <shawn.guo@linaro.org>,
        Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Lars Poeschel <poeschel@lemonage.de>
Subject: Re: [PATCH 3/6] i2c: Ignore return value of i2c_del_adapter()
Message-ID: <20130330094326.63d7e2a0@endymion.delvare>
In-Reply-To: <51569E4A.2080006@metafoo.de>
References: <1362853009-20789-1-git-send-email-lars@metafoo.de>
	<1362853009-20789-4-git-send-email-lars@metafoo.de>
	<20130330085539.442321fa@endymion.delvare>
	<51569E4A.2080006@metafoo.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2215
Lines: 65

On Sat, 30 Mar 2013 09:11:54 +0100, Lars-Peter Clausen wrote:
> Hi Jean,
> 
> On 03/30/2013 08:55 AM, Jean Delvare wrote:
> > Hi Lars-Peter,
> > 
> > On Sat,  9 Mar 2013 19:16:46 +0100, Lars-Peter Clausen wrote:
> >> i2c_del_adapter() always returns 0.
> > 
> > I beg you pardon? i2c_del_adapter() starts with:
> > 
> > int i2c_del_adapter(struct i2c_adapter *adap)
> > {
> > 	int res = 0;
> > 	struct i2c_adapter *found;
> > 	struct i2c_client *client, *next;
> > 
> > 	/* First make sure that this adapter was ever added */
> > 	mutex_lock(&core_lock);
> > 	found = idr_find(&i2c_adapter_idr, adap->nr);
> > 	mutex_unlock(&core_lock);
> > 	if (found != adap) {
> > 		pr_debug("i2c-core: attempting to delete unregistered "
> > 			 "adapter [%s]\n", adap->name);
> > 		return -EINVAL;
> > 	}
> > 
> > 	/* Tell drivers about this removal */
> > 	mutex_lock(&core_lock);
> > 	res = bus_for_each_drv(&i2c_bus_type, NULL, adap,
> > 			       __process_removed_adapter);
> > 	mutex_unlock(&core_lock);
> > 	if (res)
> > 		return res;
> > (...)
> > 
> > So, no, it doesn't "always return 0".
> > 
> 
> Patch 1 and 2 in this series remove those two instances:
> https://lkml.org/lkml/2013/3/9/72
> https://lkml.org/lkml/2013/3/9/86
> 
> For an explanation why this should be done please take a look at the cover
> letter to this patch series https://lkml.org/lkml/2013/3/9/73

Well well... Either you think this must be done and isn't questionable,
and you shouldn't bother Cc'ing a dozen driver maintainers and waiting
for them to ack. Or you think this is something for discussion and you
should consistently Cc all interested parties on the whole patch
series. If you send me one random patch in the middle of a series and
expect me to go fish for the missing parts so that I can make sense of
it all and make sane and useful comments, well this isn't going to
happen, sorry.

Now with the above pointers, I can actually make useful comments, and I
will.

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/