lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 29 Mar 2017 17:24:36 +0100 From: Jonathan Cameron <jic23@...23.retrosnub.co.uk> To: Julia Lawall <julia.lawall@...6.fr>, Arushi Singhal <arushisinghal19971997@...il.com> CC: outreachy-kernel@...glegroups.com, Lars-Peter Clausen <lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>, Jonathan Cameron <jic23@...nel.org>, Hartmut Knaack <knaack.h@....de>, Peter Meerwald-Stadler <pmeerw@...erw.net>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-iio@...r.kernel.org, devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org Subject: Re: [Outreachy kernel] [RESEND PATCH] staging: iio: Remove extra Parenthesis On 29 March 2017 16:38:28 BST, Julia Lawall <julia.lawall@...6.fr> wrote: > > >On Wed, 29 Mar 2017, Arushi Singhal wrote: > >> Remove the extra parenthesis remove the checkpatch issue. > >Is this a patch on the current state of the staging tree? When I do a >pull and then look at the code, the line numbers are all off. >Furthermore, there are not calls to BIT with parentheses around them, >so >it looks like this is a patch on the result of another patch that you >sent >that was not accepted. > >When Greg says resend what was not picked up, he doesn't mean resend it >as >is, but reapply the patch to the current state of the kernel, and if it >still applies correctly, send the result. I picked this up on Saturday as I had already pushed out the IIO togreg branch with the original patch and I try to never rebase that other than fast forwards. So didn't need a resend, but patch was originally correct in this case. At this time a cycle i tend to send pulls to Greg about every two weeks so there can be a bit of lag in IIO patches getting to the staging tree. Jonathan > >julia > >> >> Signed-off-by: Arushi Singhal <arushisinghal19971997@...il.com> >> --- >> drivers/staging/iio/cdc/ad7746.c | 16 ++++++++-------- >> 1 file changed, 8 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/staging/iio/cdc/ad7746.c >b/drivers/staging/iio/cdc/ad7746.c >> index 033a41fd9bee..b4afc12f827f 100644 >> --- a/drivers/staging/iio/cdc/ad7746.c >> +++ b/drivers/staging/iio/cdc/ad7746.c >> @@ -50,10 +50,10 @@ >> #define AD7746_STATUS_RDYCAP BIT(0) >> >> /* Capacitive Channel Setup Register Bit Designations >(AD7746_REG_CAP_SETUP) */ >> -#define AD7746_CAPSETUP_CAPEN (BIT(7)) >> -#define AD7746_CAPSETUP_CIN2 (BIT(6)) /* AD7746 only */ >> -#define AD7746_CAPSETUP_CAPDIFF (BIT(5)) >> -#define AD7746_CAPSETUP_CACHOP (BIT(0)) >> +#define AD7746_CAPSETUP_CAPEN BIT(7) >> +#define AD7746_CAPSETUP_CIN2 BIT(6) /* AD7746 only */ >> +#define AD7746_CAPSETUP_CAPDIFF BIT(5) >> +#define AD7746_CAPSETUP_CACHOP BIT(0) >> >> /* Voltage/Temperature Setup Register Bit Designations >(AD7746_REG_VT_SETUP) */ >> #define AD7746_VTSETUP_VTEN (1 << 7) >> @@ -61,9 +61,9 @@ >> #define AD7746_VTSETUP_VTMD_EXT_TEMP (1 << 5) >> #define AD7746_VTSETUP_VTMD_VDD_MON (2 << 5) >> #define AD7746_VTSETUP_VTMD_EXT_VIN (3 << 5) >> -#define AD7746_VTSETUP_EXTREF (BIT(4)) >> -#define AD7746_VTSETUP_VTSHORT (BIT(1)) >> -#define AD7746_VTSETUP_VTCHOP (BIT(0)) >> +#define AD7746_VTSETUP_EXTREF BIT(4) >> +#define AD7746_VTSETUP_VTSHORT BIT(1) >> +#define AD7746_VTSETUP_VTCHOP BIT(0) >> >> /* Excitation Setup Register Bit Designations (AD7746_REG_EXC_SETUP) >*/ >> #define AD7746_EXCSETUP_CLKCTRL BIT(7) >> @@ -87,7 +87,7 @@ >> #define AD7746_CONF_MODE_GAIN_CAL (6 << 0) >> >> /* CAPDAC Register Bit Designations (AD7746_REG_CAPDACx) */ >> -#define AD7746_CAPDAC_DACEN (BIT(7)) >> +#define AD7746_CAPDAC_DACEN BIT(7) >> #define AD7746_CAPDAC_DACP(x) ((x) & 0x7F) >> >> /* >> -- >> 2.11.0 >> >> -- >> 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@...glegroups.com. >> To post to this group, send email to >outreachy-kernel@...glegroups.com. >> To view this discussion on the web visit >https://groups.google.com/d/msgid/outreachy-kernel/20170329151712.GA5651%40arushi-HP-Pavilion-Notebook. >> For more options, visit https://groups.google.com/d/optout. >> -- Sent from my Android device with K-9 Mail. Please excuse my brevity.
Powered by blists - more mailing lists