From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932972AbdCKLkr (ORCPT ); Sat, 11 Mar 2017 06:40:47 -0500 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:64612 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755348AbdCKLkj (ORCPT ); Sat, 11 Mar 2017 06:40:39 -0500 X-IronPort-AV: E=Sophos;i="5.36,146,1486422000"; d="scan'208";a="216384125" Date: Sat, 11 Mar 2017 12:40:34 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: SIMRAN SINGHAL cc: Julia Lawall , Lars-Peter Clausen , Michael.Hennerich@analog.com, Jonathan Cameron , knaack.h@gmx.de, Greg KH , linux-iio@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, outreachy-kernel@googlegroups.com Subject: Re: [Outreachy kernel] [PATCH 00/10] staging: iio: Remove exceptional & on functions name In-Reply-To: Message-ID: References: <1489203401-17518-1-git-send-email-singhalsimran0@gmail.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 11 Mar 2017, SIMRAN SINGHAL wrote: > On Sat, Mar 11, 2017 at 2:57 PM, Julia Lawall wrote: > > > > > > On Sat, 11 Mar 2017, SIMRAN SINGHAL wrote: > > > >> On Sat, Mar 11, 2017 at 12:12 PM, Julia Lawall wrote: > >> > > >> > > >> > On Sat, 11 Mar 2017, simran singhal wrote: > >> > > >> >> This patch-series removes exceptional & on functions name. > >> > > >> > The semantic patch shown does nothing to check that the use of & is > >> > exception in the given file. It just removes all the & on function names. > >> > > >> yes, I used this semantic patch only for only finding the functions with & > >> and then I find out this drivers and use semantic patch for removing &. > > > > The commit message should be written with respect to what was actually > > done, and not copied from a patch that actually did something else. The > > original semantic patch, that was probably only put in the cover letter, > > actually checked if the file mostly used function pointers in a particular > > way. > > > Julia, so what should I do? > I should not include this semantic patch as it does not check that that the use > of & is exception in the given file. > OR > As I did it manually so I should mention this. If you checked it manually, just mention that. I don't have the actual patches handy any more, but you could also say something like: Remove & from function pointers to conform to the style found elsewhere in the file. Done using the following semantic patch. This would be ok if the included semantic patch makes the change. It doesn't suggest that the semantic patch checkked for the excpeptionality. julia > > > julia > > > >> > >> > julia > >> > > >> >> > >> >> simran singhal (10): > >> >> staging: iio: ad7192: Remove exceptional & on function name > >> >> staging: iio: ad7780: Remove exceptional & on function name > >> >> staging: iio: cdc: ad7746: Remove exceptional & on function name > >> >> staging: iio: cdc: ad7152: Remove exceptional & on function name > >> >> staging: iio: adis16240: Remove exceptional & on function name > >> >> staging: iio: adis16201: Remove exceptional & on function name > >> >> staging: iio: adis16209: Remove exceptional & on function name > >> >> staging: iio: adis16203: Remove exceptional & on function name > >> >> staging: iio: resolver: Remove exceptional & on function name > >> >> staging: iio: gyro: Remove exceptional & on function name > >> >> > >> >> drivers/staging/iio/accel/adis16201.c | 4 ++-- > >> >> drivers/staging/iio/accel/adis16203.c | 4 ++-- > >> >> drivers/staging/iio/accel/adis16209.c | 4 ++-- > >> >> drivers/staging/iio/accel/adis16240.c | 4 ++-- > >> >> drivers/staging/iio/adc/ad7192.c | 12 ++++++------ > >> >> drivers/staging/iio/adc/ad7780.c | 2 +- > >> >> drivers/staging/iio/cdc/ad7152.c | 6 +++--- > >> >> drivers/staging/iio/cdc/ad7746.c | 4 ++-- > >> >> drivers/staging/iio/gyro/adis16060_core.c | 2 +- > >> >> drivers/staging/iio/resolver/ad2s1200.c | 2 +- > >> >> drivers/staging/iio/resolver/ad2s90.c | 2 +- > >> >> 11 files changed, 23 insertions(+), 23 deletions(-) > >> >> > >> >> -- > >> >> 2.7.4 > >> >> > >> >> -- > >> >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > >> >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > >> >> To post to this group, send email to outreachy-kernel@googlegroups.com. > >> >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1489203401-17518-1-git-send-email-singhalsimran0%40gmail.com. > >> >> For more options, visit https://groups.google.com/d/optout. > >> >> > >> > >> -- > >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > >> To post to this group, send email to outreachy-kernel@googlegroups.com. > >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/CALrZqyODTiF%2BgdwwYnmL1W1oi-R0u2rM4c40jb5iWXFxJpfdJw%40mail.gmail.com. > >> For more options, visit https://groups.google.com/d/optout. > >> >