From mboxrd@z Thu Jan 1 00:00:00 1970 From: Antti Palosaari Subject: Re: [PATCH v7 16/24] i2c: allow adapter drivers to override the adapter locking Date: Mon, 2 May 2016 22:20:19 +0300 Message-ID: <5727A873.4080400@iki.fi> References: <1461165484-2314-1-git-send-email-peda@axentia.se> <1461165484-2314-17-git-send-email-peda@axentia.se> <20160428205018.GA3553@katana> <470abe38-ab5f-2d0a-305b-e1a3253ce5a9@axentia.se> <20160429071604.GB1870@katana> <357e6fda-73b3-fb7f-c341-97f09af1943f@axentia.se> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <357e6fda-73b3-fb7f-c341-97f09af1943f-koto5C5qi+TLoDKTGw+V6w@public.gmane.org> Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Peter Rosin , Wolfram Sang Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jonathan Corbet , Peter Korsgaard , Guenter Roeck , Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , Mauro Carvalho Chehab , Rob Herring , Frank Rowand , Grant Likely , Andrew Morton , "David S. Miller" , Greg Kroah-Hartman , Kalle Valo , Jiri Slaby , Daniel Baluta , Lucas De Marchi , Adriana Reus , Matt Ranostay , Krzysztof Kozlowski , Hans List-Id: devicetree@vger.kernel.org On 04/29/2016 12:16 PM, Peter Rosin wrote: > On 2016-04-29 09:16, Wolfram Sang wrote: >>> Yes, obviously... I'll make that change locally and wait for the rest. >> Another nit: You could use '--strict' with checkpatch and see if you >> want to fix the issues reported. I am not keen on those (except for >> 'space around operators'), it's a matter of taste I guess, but maybe you >> like some of the suggestions. >> > Yes, they look like reasonable complaints. > > So, I fixed all of them locally except the complaint about lack of comment > on the new struct mutex member in struct si2168_dev (patch 21/24), > because that patch is Anttis and he's the maintainer of that driver... > > Antti, if you want that fixed as part of this series, send a suitable comment > for the mutex this way and I'll incorporate it. Ah, I never ran checkpatch with --strict option... CHECK: struct mutex definition without comment #202: FILE: drivers/media/dvb-frontends/si2168_priv.h:32: + struct mutex i2c_mutex; If you wish you could add some comment for it, but for me it is still pretty much self explaining. It is lock to protect firmware command execution. Command is executed always with I2C write and then poll reply using I2C read until it timeouts or answers with "ready" status. regards Antti -- http://palosaari.fi/