* [PATCH 0/2] IIO coding tasks
@ 2017-03-21 13:30 Arushi Singhal
2017-03-21 13:30 ` [PATCH 1/2] staging: ad7759: Replace mlock with driver private lock Arushi Singhal
` (2 more replies)
0 siblings, 3 replies; 14+ messages in thread
From: Arushi Singhal @ 2017-03-21 13:30 UTC (permalink / raw)
To: jic23
Cc: knaack.h, outreachy-kernel, Michael.Hennerich, lars, gregkh,
speakup, devel, linux-kernel, pmeerw, linux-iio, Arushi Singhal
Patchseries of IIO coding tasks
Arushi Singhal (2):
staging: ad7759: Replace mlock with driver private lock
staging: iio: ade7759: Move header content to implementation file
drivers/staging/iio/meter/ade7759.c | 56 +++++++++++++++++++++++++++++++++--
drivers/staging/iio/meter/ade7759.h | 53 ---------------------------------
2 files changed, 54 insertions(+), 57 deletions(-)
--
2.11.0
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] staging: ad7759: Replace mlock with driver private lock
2017-03-21 13:30 [PATCH 0/2] IIO coding tasks Arushi Singhal
@ 2017-03-21 13:30 ` Arushi Singhal
2017-03-21 13:30 ` [PATCH 2/2] staging: iio: ade7759: Move header content to implementation file Arushi Singhal
2017-03-21 16:03 ` [Outreachy kernel] [PATCH 0/2] IIO coding tasks Alison Schofield
2 siblings, 0 replies; 14+ messages in thread
From: Arushi Singhal @ 2017-03-21 13:30 UTC (permalink / raw)
To: jic23
Cc: knaack.h, outreachy-kernel, Michael.Hennerich, lars, gregkh,
speakup, devel, linux-kernel, pmeerw, linux-iio, Arushi Singhal
The IIO subsystem is redefining iio_dev->mlock to be used by
the IIO core only for protecting device operating mode changes.
ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* modes.
In this driver, mlock was being used to protect hardware state
changes. Replace it with a lock in the devices global data.
Signed-off-by: Arushi Singhal <arushisinghal19971997@gmail.com>
---
drivers/staging/iio/meter/ade7759.c | 4 ++--
drivers/staging/iio/meter/ade7759.h | 1 +
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/iio/meter/ade7759.c b/drivers/staging/iio/meter/ade7759.c
index 944ee3401029..1d1e0b33021f 100644
--- a/drivers/staging/iio/meter/ade7759.c
+++ b/drivers/staging/iio/meter/ade7759.c
@@ -381,7 +381,7 @@ static ssize_t ade7759_write_frequency(struct device *dev,
if (!val)
return -EINVAL;
- mutex_lock(&indio_dev->mlock);
+ mutex_lock(&st->buf_lock);
t = 27900 / val;
if (t > 0)
@@ -402,7 +402,7 @@ static ssize_t ade7759_write_frequency(struct device *dev,
ret = ade7759_spi_write_reg_16(dev, ADE7759_MODE, reg);
out:
- mutex_unlock(&indio_dev->mlock);
+ mutex_unlock(&st->buf_lock);
return ret ? ret : len;
}
diff --git a/drivers/staging/iio/meter/ade7759.h b/drivers/staging/iio/meter/ade7759.h
index f0716d2fdf8e..4f69bb93cc45 100644
--- a/drivers/staging/iio/meter/ade7759.h
+++ b/drivers/staging/iio/meter/ade7759.h
@@ -42,6 +42,7 @@
* @buf_lock: mutex to protect tx and rx
* @tx: transmit buffer
* @rx: receive buffer
+ * @lock protect sensor state
**/
struct ade7759_state {
struct spi_device *us;
--
2.11.0
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] staging: iio: ade7759: Move header content to implementation file
2017-03-21 13:30 [PATCH 0/2] IIO coding tasks Arushi Singhal
2017-03-21 13:30 ` [PATCH 1/2] staging: ad7759: Replace mlock with driver private lock Arushi Singhal
@ 2017-03-21 13:30 ` Arushi Singhal
2017-03-21 16:03 ` [Outreachy kernel] [PATCH 0/2] IIO coding tasks Alison Schofield
2 siblings, 0 replies; 14+ messages in thread
From: Arushi Singhal @ 2017-03-21 13:30 UTC (permalink / raw)
To: jic23
Cc: knaack.h, outreachy-kernel, Michael.Hennerich, lars, gregkh,
speakup, devel, linux-kernel, pmeerw, linux-iio, Arushi Singhal
The contents of ade7759.h are only used in ade7759.c.
Move the header contents to the implemtation file,
and delete the header file.
Signed-off-by: Arushi Singhal <arushisinghal19971997@gmail.com>
---
drivers/staging/iio/meter/ade7759.c | 52 ++++++++++++++++++++++++++++++++++-
drivers/staging/iio/meter/ade7759.h | 54 -------------------------------------
2 files changed, 51 insertions(+), 55 deletions(-)
diff --git a/drivers/staging/iio/meter/ade7759.c b/drivers/staging/iio/meter/ade7759.c
index 1d1e0b33021f..86ac103dffe5 100644
--- a/drivers/staging/iio/meter/ade7759.c
+++ b/drivers/staging/iio/meter/ade7759.c
@@ -21,7 +21,57 @@
#include <linux/iio/iio.h>
#include <linux/iio/sysfs.h>
#include "meter.h"
-#include "ade7759.h"
+
+#define ADE7759_WAVEFORM 0x01
+#define ADE7759_AENERGY 0x02
+#define ADE7759_RSTENERGY 0x03
+#define ADE7759_STATUS 0x04
+#define ADE7759_RSTSTATUS 0x05
+#define ADE7759_MODE 0x06
+#define ADE7759_CFDEN 0x07
+#define ADE7759_CH1OS 0x08
+#define ADE7759_CH2OS 0x09
+#define ADE7759_GAIN 0x0A
+#define ADE7759_APGAIN 0x0B
+#define ADE7759_PHCAL 0x0C
+#define ADE7759_APOS 0x0D
+#define ADE7759_ZXTOUT 0x0E
+#define ADE7759_SAGCYC 0x0F
+#define ADE7759_IRQEN 0x10
+#define ADE7759_SAGLVL 0x11
+#define ADE7759_TEMP 0x12
+#define ADE7759_LINECYC 0x13
+#define ADE7759_LENERGY 0x14
+#define ADE7759_CFNUM 0x15
+#define ADE7759_CHKSUM 0x1E
+#define ADE7759_DIEREV 0x1F
+
+#define ADE7759_READ_REG(a) a
+#define ADE7759_WRITE_REG(a) ((a) | 0x80)
+
+#define ADE7759_MAX_TX 6
+#define ADE7759_MAX_RX 6
+#define ADE7759_STARTUP_DELAY 1000
+
+#define ADE7759_SPI_SLOW (u32)(300 * 1000)
+#define ADE7759_SPI_BURST (u32)(1000 * 1000)
+#define ADE7759_SPI_FAST (u32)(2000 * 1000)
+
+/**
+ * struct ade7759_state - device instance specific data
+ * @us: actual spi_device
+ * @buf_lock: mutex to protect tx and rx
+ * @tx: transmit buffer
+ * @rx: receive buffer
+ * @lock protect sensor state
+ **/
+
+struct ade7759_state {
+ struct spi_device *us;
+ struct mutex buf_lock; /* protect tx and rx */
+ u8 tx[ADE7759_MAX_TX] ____cacheline_aligned;
+ u8 rx[ADE7759_MAX_RX];
+};
static int ade7759_spi_write_reg_8(struct device *dev,
u8 reg_address,
diff --git a/drivers/staging/iio/meter/ade7759.h b/drivers/staging/iio/meter/ade7759.h
index 4f69bb93cc45..e69de29bb2d1 100644
--- a/drivers/staging/iio/meter/ade7759.h
+++ b/drivers/staging/iio/meter/ade7759.h
@@ -1,54 +0,0 @@
-#ifndef _ADE7759_H
-#define _ADE7759_H
-
-#define ADE7759_WAVEFORM 0x01
-#define ADE7759_AENERGY 0x02
-#define ADE7759_RSTENERGY 0x03
-#define ADE7759_STATUS 0x04
-#define ADE7759_RSTSTATUS 0x05
-#define ADE7759_MODE 0x06
-#define ADE7759_CFDEN 0x07
-#define ADE7759_CH1OS 0x08
-#define ADE7759_CH2OS 0x09
-#define ADE7759_GAIN 0x0A
-#define ADE7759_APGAIN 0x0B
-#define ADE7759_PHCAL 0x0C
-#define ADE7759_APOS 0x0D
-#define ADE7759_ZXTOUT 0x0E
-#define ADE7759_SAGCYC 0x0F
-#define ADE7759_IRQEN 0x10
-#define ADE7759_SAGLVL 0x11
-#define ADE7759_TEMP 0x12
-#define ADE7759_LINECYC 0x13
-#define ADE7759_LENERGY 0x14
-#define ADE7759_CFNUM 0x15
-#define ADE7759_CHKSUM 0x1E
-#define ADE7759_DIEREV 0x1F
-
-#define ADE7759_READ_REG(a) a
-#define ADE7759_WRITE_REG(a) ((a) | 0x80)
-
-#define ADE7759_MAX_TX 6
-#define ADE7759_MAX_RX 6
-#define ADE7759_STARTUP_DELAY 1000
-
-#define ADE7759_SPI_SLOW (u32)(300 * 1000)
-#define ADE7759_SPI_BURST (u32)(1000 * 1000)
-#define ADE7759_SPI_FAST (u32)(2000 * 1000)
-
-/**
- * struct ade7759_state - device instance specific data
- * @us: actual spi_device
- * @buf_lock: mutex to protect tx and rx
- * @tx: transmit buffer
- * @rx: receive buffer
- * @lock protect sensor state
- **/
-struct ade7759_state {
- struct spi_device *us;
- struct mutex buf_lock;
- u8 tx[ADE7759_MAX_TX] ____cacheline_aligned;
- u8 rx[ADE7759_MAX_RX];
-};
-
-#endif
--
2.11.0
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 13:30 [PATCH 0/2] IIO coding tasks Arushi Singhal
2017-03-21 13:30 ` [PATCH 1/2] staging: ad7759: Replace mlock with driver private lock Arushi Singhal
2017-03-21 13:30 ` [PATCH 2/2] staging: iio: ade7759: Move header content to implementation file Arushi Singhal
@ 2017-03-21 16:03 ` Alison Schofield
2017-03-21 16:36 ` Arushi Singhal
2 siblings, 1 reply; 14+ messages in thread
From: Alison Schofield @ 2017-03-21 16:03 UTC (permalink / raw)
To: Arushi Singhal
Cc: jic23, knaack.h, outreachy-kernel, Michael.Hennerich, lars,
gregkh, speakup, devel, linux-kernel, pmeerw, linux-iio
On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> Patchseries of IIO coding tasks
This wouldn't be a patchset. Although they touch the same
driver, the changes are unrelated. See more below...
>
> Arushi Singhal (2):
> staging: ad7759: Replace mlock with driver private lock
This one is already been submitted. If you have a v2 for it, then v2
the original patch.
> staging: iio: ade7759: Move header content to implementation file
This patch is done and applied already. See the Coding Task #1 notes
in the IIO Tasks page.
alisons
>
> drivers/staging/iio/meter/ade7759.c | 56 +++++++++++++++++++++++++++++++++--
> drivers/staging/iio/meter/ade7759.h | 53 ---------------------------------
> 2 files changed, 54 insertions(+), 57 deletions(-)
>
> --
> 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@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/20170321133021.6737-1-arushisinghal19971997%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 16:03 ` [Outreachy kernel] [PATCH 0/2] IIO coding tasks Alison Schofield
@ 2017-03-21 16:36 ` Arushi Singhal
2017-03-21 16:39 ` Julia Lawall
0 siblings, 1 reply; 14+ messages in thread
From: Arushi Singhal @ 2017-03-21 16:36 UTC (permalink / raw)
To: Alison Schofield
Cc: Jonathan Cameron, Hartmut Knaack, outreachy-kernel,
Michael Hennerich, lars, Greg KH, speakup, driverdevel,
linux-kernel, Peter Meerwald-Stadler, linux-iio
[-- Attachment #1: Type: text/plain, Size: 1917 bytes --]
On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <amsfield22@gmail.com>
wrote:
> On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > Patchseries of IIO coding tasks
>
> This wouldn't be a patchset. Although they touch the same
> driver, the changes are unrelated. See more below...
>
This I have send as a Patchset because as you have mentioned below that the
[PATCH 2/2] was already done before by someone but I think so it is not yet
applied in the Greg's tree yet.So I have done both the changes and as they
should be applied one after other so that's why I have send them as
Patchset.
> >
> > Arushi Singhal (2):
> > staging: ad7759: Replace mlock with driver private lock
>
> This one is already been submitted. If you have a v2 for it, then v2
> the original patch.
>
> Is it submitted by me only before? And this is not the v2.
I have just resed it.
> > staging: iio: ade7759: Move header content to implementation file
>
> This patch is done and applied already. See the Coding Task #1 notes
> in the IIO Tasks page.
>
Not at applied I think so.
Thanks
Arushi
>
> alisons
>
> >
> > drivers/staging/iio/meter/ade7759.c | 56
> +++++++++++++++++++++++++++++++++--
> > drivers/staging/iio/meter/ade7759.h | 53
> ---------------------------------
> > 2 files changed, 54 insertions(+), 57 deletions(-)
> >
> > --
> > 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@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/20170321133021.6737-1-
> arushisinghal19971997%40gmail.com.
> > For more options, visit https://groups.google.com/d/optout.
>
[-- Attachment #2: Type: text/html, Size: 3444 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 16:36 ` Arushi Singhal
@ 2017-03-21 16:39 ` Julia Lawall
2017-03-21 17:02 ` Alison Schofield
0 siblings, 1 reply; 14+ messages in thread
From: Julia Lawall @ 2017-03-21 16:39 UTC (permalink / raw)
To: Arushi Singhal
Cc: Alison Schofield, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
[-- Attachment #1: Type: text/plain, Size: 2877 bytes --]
On Tue, 21 Mar 2017, Arushi Singhal wrote:
> On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <amsfield22@gmail.com>
> wrote:
> On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > Patchseries of IIO coding tasks
>
> This wouldn't be a patchset. Although they touch the same
> driver, the changes are unrelated. See more below...
>
> This I have send as a Patchset because as you have mentioned below that the
> [PATCH 2/2] was already done before by someone but I think so it is not yet
> applied in the Greg's tree yet.So I have done both the changes and as they
> should be applied one after other so that's why I have send them as
> Patchset.
For the IIO patches, it is better to work on the IIO tree, not Greg's.
Greg manages staging, not IIO. The IIO patches should appear in Greg's
tree eventually.
julia
> >
> > Arushi Singhal (2):
> > staging: ad7759: Replace mlock with driver private lock
>
> This one is already been submitted. If you have a v2 for it,
> then v2
> the original patch.
>
> Is it submitted by me only before? And this is not the v2.
> I have just resed it.
> > staging: iio: ade7759: Move header content to implementation
> file
>
> This patch is done and applied already. See the Coding Task #1
> notes
> in the IIO Tasks page.
>
> Not at applied I think so.
> Thanks
> Arushi
>
> alisons
>
> >
> > drivers/staging/iio/meter/ade7759.c | 56
> +++++++++++++++++++++++++++++++++--
> > drivers/staging/iio/meter/ade7759.h | 53
> ---------------------------------
> > 2 files changed, 54 insertions(+), 57 deletions(-)
> >
> > --
> > 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@googlegroups.com.
> > To post to this group, send email to
> outreachy-kernel@googlegroups.com.
> > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737-1-ar
> ushisinghal19971997%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 visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0H2z8x
> taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 16:39 ` Julia Lawall
@ 2017-03-21 17:02 ` Alison Schofield
0 siblings, 0 replies; 14+ messages in thread
From: Alison Schofield @ 2017-03-21 17:02 UTC (permalink / raw)
To: Julia Lawall
Cc: Arushi Singhal, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
>
>
> On Tue, 21 Mar 2017, Arushi Singhal wrote:
>
> > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <amsfield22@gmail.com>
> > wrote:
> > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > > Patchseries of IIO coding tasks
> >
> > This wouldn't be a patchset. Although they touch the same
> > driver, the changes are unrelated. See more below...
> >
> > This I have send as a Patchset because as you have mentioned below that the
> > [PATCH 2/2] was already done before by someone but I think so it is not yet
> > applied in the Greg's tree yet.So I have done both the changes and as they
> > should be applied one after other so that's why I have send them as
> > Patchset.
>
> For the IIO patches, it is better to work on the IIO tree, not Greg's.
> Greg manages staging, not IIO. The IIO patches should appear in Greg's
> tree eventually.
>
> julia
We didn't direct applicants to create an iio tree. At this point,
it seems more than is necessary. They can follow the directions in the
task descriptions and avoid the collisions.
Of course, they are welcome to create a tree to iio/testing.
(IMHO it's more overhead/busy work and maybe not the best use
of time in the home stretch of the application period.)
alisons
>
> > >
> > > Arushi Singhal (2):
> > > staging: ad7759: Replace mlock with driver private lock
> >
> > This one is already been submitted. If you have a v2 for it,
> > then v2
> > the original patch.
> >
> > Is it submitted by me only before? And this is not the v2.
> > I have just resed it.
> > > staging: iio: ade7759: Move header content to implementation
> > file
> >
> > This patch is done and applied already. See the Coding Task #1
> > notes
> > in the IIO Tasks page.
> >
> > Not at applied I think so.
> > Thanks
> > Arushi
> >
> > alisons
> >
> > >
> > > drivers/staging/iio/meter/ade7759.c | 56
> > +++++++++++++++++++++++++++++++++--
> > > drivers/staging/iio/meter/ade7759.h | 53
> > ---------------------------------
> > > 2 files changed, 54 insertions(+), 57 deletions(-)
> > >
> > > --
> > > 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@googlegroups.com.
> > > To post to this group, send email to
> > outreachy-kernel@googlegroups.com.
> > > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737-1-ar
> > ushisinghal19971997%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 visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0H2z8x
> > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
> > For more options, visit https://groups.google.com/d/optout.
> >
> >
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
@ 2017-03-21 17:02 ` Alison Schofield
0 siblings, 0 replies; 14+ messages in thread
From: Alison Schofield @ 2017-03-21 17:02 UTC (permalink / raw)
To: Julia Lawall
Cc: Arushi Singhal, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
>
>
> On Tue, 21 Mar 2017, Arushi Singhal wrote:
>
> > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <amsfield22@gmail.com>
> > wrote:
> > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > > Patchseries of IIO coding tasks
> >
> > This wouldn't be a patchset.� Although they touch the same
> > driver, the changes are unrelated.� See more below...
> >
> > This I have send as a Patchset because as you have mentioned below that the
> > [PATCH 2/2] was already done before by someone but I think so it is not yet
> > applied in the Greg's tree yet.So I have done both the changes and as they
> > should be applied one after other so that's why I have send them as
> > Patchset.
>
> For the IIO patches, it is better to work on the IIO tree, not Greg's.
> Greg manages staging, not IIO. The IIO patches should appear in Greg's
> tree eventually.
>
> julia
We didn't direct applicants to create an iio tree. At this point,
it seems more than is necessary. They can follow the directions in the
task descriptions and avoid the collisions.
Of course, they are welcome to create a tree to iio/testing.
(IMHO it's more overhead/busy work and maybe not the best use
of time in the home stretch of the application period.)
alisons
>
> > >
> > > Arushi Singhal (2):
> > >� �staging: ad7759: Replace mlock with driver private lock
> >
> > This one is already been submitted.� If you have a v2 for it,
> > then v2
> > the original patch.
> >
> > Is it submitted by me only before? And this is not the v2.
> > I have just resed it.
> > >� �staging: iio: ade7759: Move header content to implementation
> > file
> >
> > This patch is done and applied already.� See the Coding Task #1
> > notes
> > in the IIO Tasks page.
> >
> > Not at applied I think so.
> > Thanks
> > Arushi
> >
> > alisons
> >
> > >
> > >� drivers/staging/iio/meter/ade7759.c� �| 56
> > +++++++++++++++++++++++++++++++++--
> > >� drivers/staging/iio/meter/ade7759.h� �| 53
> > ---------------------------------
> > >� 2 files changed, 54 insertions(+), 57 deletions(-)
> > >
> > > --
> > > 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@googlegroups.com.
> > > To post to this group, send email to
> > outreachy-kernel@googlegroups.com.
> > > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737-1-ar
> > ushisinghal19971997%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 visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0H2z8x
> > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
> > For more options, visit https://groups.google.com/d/optout.
> >
> >
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 17:02 ` Alison Schofield
(?)
@ 2017-03-21 17:09 ` Julia Lawall
-1 siblings, 0 replies; 14+ messages in thread
From: Julia Lawall @ 2017-03-21 17:09 UTC (permalink / raw)
To: Alison Schofield
Cc: Julia Lawall, Arushi Singhal, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
[-- Attachment #1: Type: text/plain, Size: 4461 bytes --]
On Tue, 21 Mar 2017, Alison Schofield wrote:
> On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
> >
> >
> > On Tue, 21 Mar 2017, Arushi Singhal wrote:
> >
> > > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <amsfield22@gmail.com>
> > > wrote:
> > > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > > > Patchseries of IIO coding tasks
> > >
> > > This wouldn't be a patchset. Although they touch the same
> > > driver, the changes are unrelated. See more below...
> > >
> > > This I have send as a Patchset because as you have mentioned below that the
> > > [PATCH 2/2] was already done before by someone but I think so it is not yet
> > > applied in the Greg's tree yet.So I have done both the changes and as they
> > > should be applied one after other so that's why I have send them as
> > > Patchset.
> >
> > For the IIO patches, it is better to work on the IIO tree, not Greg's.
> > Greg manages staging, not IIO. The IIO patches should appear in Greg's
> > tree eventually.
> >
> > julia
>
> We didn't direct applicants to create an iio tree. At this point,
> it seems more than is necessary. They can follow the directions in the
> task descriptions and avoid the collisions.
I meant just pull from the IIO tree you mentioned, work on that, submit
patches, and then expect to see the patches show up there at some time in
the future.
If someone make IIO patches on Greg's tree, then they will have an out of
date view.
julia
>
> Of course, they are welcome to create a tree to iio/testing.
>
> (IMHO it's more overhead/busy work and maybe not the best use
> of time in the home stretch of the application period.)
>
> alisons
> >
> > > >
> > > > Arushi Singhal (2):
> > > > staging: ad7759: Replace mlock with driver private lock
> > >
> > > This one is already been submitted. If you have a v2 for it,
> > > then v2
> > > the original patch.
> > >
> > > Is it submitted by me only before? And this is not the v2.
> > > I have just resed it.
> > > > staging: iio: ade7759: Move header content to implementation
> > > file
> > >
> > > This patch is done and applied already. See the Coding Task #1
> > > notes
> > > in the IIO Tasks page.
> > >
> > > Not at applied I think so.
> > > Thanks
> > > Arushi
> > >
> > > alisons
> > >
> > > >
> > > > drivers/staging/iio/meter/ade7759.c | 56
> > > +++++++++++++++++++++++++++++++++--
> > > > drivers/staging/iio/meter/ade7759.h | 53
> > > ---------------------------------
> > > > 2 files changed, 54 insertions(+), 57 deletions(-)
> > > >
> > > > --
> > > > 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@googlegroups.com.
> > > > To post to this group, send email to
> > > outreachy-kernel@googlegroups.com.
> > > > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737-1-ar
> > > ushisinghal19971997%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 visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0H2z8x
> > > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.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/20170321170221.GB2793%40d830.WORKGROUP.
> For more options, visit https://groups.google.com/d/optout.
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 17:02 ` Alison Schofield
(?)
(?)
@ 2017-03-21 17:22 ` Arushi Singhal
2017-03-21 17:27 ` Julia Lawall
2017-03-21 17:33 ` Alison Schofield
-1 siblings, 2 replies; 14+ messages in thread
From: Arushi Singhal @ 2017-03-21 17:22 UTC (permalink / raw)
To: Alison Schofield
Cc: Julia Lawall, Jonathan Cameron, Hartmut Knaack, outreachy-kernel,
Michael Hennerich, lars, Greg KH, speakup, driverdevel,
linux-kernel, Peter Meerwald-Stadler, linux-iio
[-- Attachment #1: Type: text/plain, Size: 3974 bytes --]
On Tue, Mar 21, 2017 at 10:32 PM, Alison Schofield <amsfield22@gmail.com>
wrote:
> On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
> >
> >
> > On Tue, 21 Mar 2017, Arushi Singhal wrote:
> >
> > > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <
> amsfield22@gmail.com>
> > > wrote:
> > > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > > > Patchseries of IIO coding tasks
> > >
> > > This wouldn't be a patchset. Although they touch the same
> > > driver, the changes are unrelated. See more below...
> > >
> > > This I have send as a Patchset because as you have mentioned below
> that the
> > > [PATCH 2/2] was already done before by someone but I think so it is
> not yet
> > > applied in the Greg's tree yet.So I have done both the changes and as
> they
> > > should be applied one after other so that's why I have send them as
> > > Patchset.
> >
> > For the IIO patches, it is better to work on the IIO tree, not Greg's.
> > Greg manages staging, not IIO. The IIO patches should appear in Greg's
> > tree eventually.
> >
> > julia
>
> We didn't direct applicants to create an iio tree. At this point,
> it seems more than is necessary. They can follow the directions in the
> task descriptions and avoid the collisions.
>
> Of course, they are welcome to create a tree to iio/testing.
>
> (IMHO it's more overhead/busy work and maybe not the best use
> of time in the home stretch of the application period.)
>
> alisons
>
Hi Alison
As you have mentioned that my [PATCH 2/2] is already being done someone. So
how can I make the changes of [PATCH 1/2] on top of it as [PATCH 2/2] is
not yet applied on the staging tree.
Please suggest me.
Thanks
Arushi
> >
> > > >
> > > > Arushi Singhal (2):
> > > > staging: ad7759: Replace mlock with driver private lock
> > >
> > > This one is already been submitted. If you have a v2 for it,
> > > then v2
> > > the original patch.
> > >
> > > Is it submitted by me only before? And this is not the v2.
> > > I have just resed it.
> > > > staging: iio: ade7759: Move header content to implementation
> > > file
> > >
> > > This patch is done and applied already. See the Coding Task #1
> > > notes
> > > in the IIO Tasks page.
> > >
> > > Not at applied I think so.
> > > Thanks
> > > Arushi
> > >
> > > alisons
> > >
> > > >
> > > > drivers/staging/iio/meter/ade7759.c | 56
> > > +++++++++++++++++++++++++++++++++--
> > > > drivers/staging/iio/meter/ade7759.h | 53
> > > ---------------------------------
> > > > 2 files changed, 54 insertions(+), 57 deletions(-)
> > > >
> > > > --
> > > > 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@googlegroups.com.
> > > > To post to this group, send email to
> > > outreachy-kernel@googlegroups.com.
> > > > To view this discussion on the web visithttps://groups.google.
> com/d/msgid/outreachy-kernel/20170321133021.6737-1-ar
> > > ushisinghal19971997%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 visithttps://groups.google.
> com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0H2z8x
> > > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
> > > For more options, visit https://groups.google.com/d/optout.
> > >
> > >
>
>
[-- Attachment #2: Type: text/html, Size: 6386 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 17:22 ` Arushi Singhal
@ 2017-03-21 17:27 ` Julia Lawall
2017-03-21 17:57 ` Jonathan Cameron
2017-03-21 17:33 ` Alison Schofield
1 sibling, 1 reply; 14+ messages in thread
From: Julia Lawall @ 2017-03-21 17:27 UTC (permalink / raw)
To: Arushi Singhal
Cc: Alison Schofield, Julia Lawall, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
[-- Attachment #1: Type: text/plain, Size: 4991 bytes --]
On Tue, 21 Mar 2017, Arushi Singhal wrote:
>
>
> On Tue, Mar 21, 2017 at 10:32 PM, Alison Schofield <amsfield22@gmail.com>
> wrote:
> On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
> >
> >
> > On Tue, 21 Mar 2017, Arushi Singhal wrote:
> >
> > > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield
> <amsfield22@gmail.com>
> > > wrote:
> > > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi
> Singhal wrote:
> > > > Patchseries of IIO coding tasks
> > >
> > > This wouldn't be a patchset. Although they touch the
> same
> > > driver, the changes are unrelated. See more below...
> > >
> > > This I have send as a Patchset because as you have mentioned
> below that the
> > > [PATCH 2/2] was already done before by someone but I think
> so it is not yet
> > > applied in the Greg's tree yet.So I have done both the
> changes and as they
> > > should be applied one after other so that's why I have send
> them as
> > > Patchset.
> >
> > For the IIO patches, it is better to work on the IIO tree, not
> Greg's.
> > Greg manages staging, not IIO. The IIO patches should appear
> in Greg's
> > tree eventually.
> >
> > julia
>
> We didn't direct applicants to create an iio tree. At this
> point,
> it seems more than is necessary. They can follow the directions
> in the
> task descriptions and avoid the collisions.
>
> Of course, they are welcome to create a tree to iio/testing.
>
> (IMHO it's more overhead/busy work and maybe not the best use
> of time in the home stretch of the application period.)
>
> alisons
>
>
> Hi Alison
> As you have mentioned that my [PATCH 2/2] is already being done someone. So
> how can I make the changes of [PATCH 1/2] on top of it as [PATCH 2/2] is not
> yet applied on the staging tree.
Make a patch that applies to the current state of the IIO tree. Just
clone that one like you cloned the staging one.
julia
> Please suggest me.
> Thanks
> Arushi
> >
> > > >
> > > > Arushi Singhal (2):
> > > > staging: ad7759: Replace mlock with driver private
> lock
> > >
> > > This one is already been submitted. If you have a v2
> for it,
> > > then v2
> > > the original patch.
> > >
> > > Is it submitted by me only before? And this is not the v2.
> > > I have just resed it.
> > > > staging: iio: ade7759: Move header content to
> implementation
> > > file
> > >
> > > This patch is done and applied already. See the
> Coding Task #1
> > > notes
> > > in the IIO Tasks page.
> > >
> > > Not at applied I think so.
> > > Thanks
> > > Arushi
> > >
> > > alisons
> > >
> > > >
> > > > drivers/staging/iio/meter/ade7759.c | 56
> > > +++++++++++++++++++++++++++++++++--
> > > > drivers/staging/iio/meter/ade7759.h | 53
> > > ---------------------------------
> > > > 2 files changed, 54 insertions(+), 57 deletions(-)
> > > >
> > > > --
> > > > 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@googlegroups.com.
> > > > To post to this group, send email to
> > > outreachy-kernel@googlegroups.com.
> > > > To view this discussion on the webvisithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737
> -1-ar
> > > ushisinghal19971997%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 webvisithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0
> H2z8x
> > > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
> > > For more options, visit https://groups.google.com/d/optout.
> > >
> > >
>
>
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 17:22 ` Arushi Singhal
2017-03-21 17:27 ` Julia Lawall
@ 2017-03-21 17:33 ` Alison Schofield
1 sibling, 0 replies; 14+ messages in thread
From: Alison Schofield @ 2017-03-21 17:33 UTC (permalink / raw)
To: Arushi Singhal
Cc: Julia Lawall, Jonathan Cameron, Hartmut Knaack, outreachy-kernel,
Michael Hennerich, lars, Greg KH, speakup, driverdevel,
linux-kernel, Peter Meerwald-Stadler, linux-iio
On Tue, Mar 21, 2017 at 10:52:46PM +0530, Arushi Singhal wrote:
> On Tue, Mar 21, 2017 at 10:32 PM, Alison Schofield <amsfield22@gmail.com>
> wrote:
>
> > On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
> > >
> > >
> > > On Tue, 21 Mar 2017, Arushi Singhal wrote:
> > >
> > > > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield <
> > amsfield22@gmail.com>
> > > > wrote:
> > > > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi Singhal wrote:
> > > > > Patchseries of IIO coding tasks
> > > >
> > > > This wouldn't be a patchset. Although they touch the same
> > > > driver, the changes are unrelated. See more below...
> > > >
> > > > This I have send as a Patchset because as you have mentioned below
> > that the
> > > > [PATCH 2/2] was already done before by someone but I think so it is
> > not yet
> > > > applied in the Greg's tree yet.So I have done both the changes and as
> > they
> > > > should be applied one after other so that's why I have send them as
> > > > Patchset.
> > >
> > > For the IIO patches, it is better to work on the IIO tree, not Greg's.
> > > Greg manages staging, not IIO. The IIO patches should appear in Greg's
> > > tree eventually.
> > >
> > > julia
> >
> > We didn't direct applicants to create an iio tree. At this point,
> > it seems more than is necessary. They can follow the directions in the
> > task descriptions and avoid the collisions.
> >
> > Of course, they are welcome to create a tree to iio/testing.
> >
> > (IMHO it's more overhead/busy work and maybe not the best use
> > of time in the home stretch of the application period.)
> >
> > alisons
> >
>
> Hi Alison
> As you have mentioned that my [PATCH 2/2] is already being done someone. So
> how can I make the changes of [PATCH 1/2] on top of it as [PATCH 2/2] is
> not yet applied on the staging tree.
> Please suggest me.
> Thanks
> Arushi
Arushi -
I don't see anything needing to be done by you!
Your ad7759 mlock patch is awaiting review. It's not your issue
to keep up with all changes going into the file. When it gets applied
will most likely merge with no merge issue at all.
alisons
>
> > >
> > > > >
> > > > > Arushi Singhal (2):
> > > > > staging: ad7759: Replace mlock with driver private lock
> > > >
> > > > This one is already been submitted. If you have a v2 for it,
> > > > then v2
> > > > the original patch.
> > > >
> > > > Is it submitted by me only before? And this is not the v2.
> > > > I have just resed it.
> > > > > staging: iio: ade7759: Move header content to implementation
> > > > file
> > > >
> > > > This patch is done and applied already. See the Coding Task #1
> > > > notes
> > > > in the IIO Tasks page.
> > > >
> > > > Not at applied I think so.
> > > > Thanks
> > > > Arushi
> > > >
> > > > alisons
> > > >
> > > > >
> > > > > drivers/staging/iio/meter/ade7759.c | 56
> > > > +++++++++++++++++++++++++++++++++--
> > > > > drivers/staging/iio/meter/ade7759.h | 53
> > > > ---------------------------------
> > > > > 2 files changed, 54 insertions(+), 57 deletions(-)
> > > > >
> > > > > --
> > > > > 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@googlegroups.com.
> > > > > To post to this group, send email to
> > > > outreachy-kernel@googlegroups.com.
> > > > > To view this discussion on the web visithttps://groups.google.
> > com/d/msgid/outreachy-kernel/20170321133021.6737-1-ar
> > > > ushisinghal19971997%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 visithttps://groups.google.
> > com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0H2z8x
> > > > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
> > > > For more options, visit https://groups.google.com/d/optout.
> > > >
> > > >
> >
> >
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
2017-03-21 17:27 ` Julia Lawall
@ 2017-03-21 17:57 ` Jonathan Cameron
0 siblings, 0 replies; 14+ messages in thread
From: Jonathan Cameron @ 2017-03-21 17:57 UTC (permalink / raw)
To: Julia Lawall, Arushi Singhal
Cc: Alison Schofield, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
On 21 March 2017 17:27:02 GMT+00:00, Julia Lawall <julia.lawall@lip6.fr> wrote:
>
>
>On Tue, 21 Mar 2017, Arushi Singhal wrote:
>
>>
>>
>> On Tue, Mar 21, 2017 at 10:32 PM, Alison Schofield
><amsfield22@gmail.com>
>> wrote:
>> On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
>> >
>> >
>> > On Tue, 21 Mar 2017, Arushi Singhal wrote:
>> >
>> > > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield
>> <amsfield22@gmail.com>
>> > > wrote:
>> > > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi
>> Singhal wrote:
>> > > > Patchseries of IIO coding tasks
>> > >
>> > > This wouldn't be a patchset. Although they touch the
>> same
>> > > driver, the changes are unrelated. See more below...
>> > >
>> > > This I have send as a Patchset because as you have
>mentioned
>> below that the
>> > > [PATCH 2/2] was already done before by someone but I think
>> so it is not yet
>> > > applied in the Greg's tree yet.So I have done both the
>> changes and as they
>> > > should be applied one after other so that's why I have send
>> them as
>> > > Patchset.
>> >
>> > For the IIO patches, it is better to work on the IIO tree,
>not
>> Greg's.
>> > Greg manages staging, not IIO. The IIO patches should appear
>> in Greg's
>> > tree eventually.
>> >
>> > julia
>>
>> We didn't direct applicants to create an iio tree. At this
>> point,
>> it seems more than is necessary. They can follow the
>directions
>> in the
>> task descriptions and avoid the collisions.
>>
>> Of course, they are welcome to create a tree to iio/testing.
>>
>> (IMHO it's more overhead/busy work and maybe not the best use
>> of time in the home stretch of the application period.)
>>
>> alisons
>>
>>
>> Hi Alison
>> As you have mentioned that my [PATCH 2/2] is already being done
>someone. So
>> how can I make the changes of [PATCH 1/2] on top of it as [PATCH 2/2]
>is not
>> yet applied on the staging tree.
>
>Make a patch that applies to the current state of the IIO tree. Just
>clone that one like you cloned the staging one.
>
>julia
>
Slight amendment,. Normally would be the togreg branch which doesn't rebase.
With all the current churn might be best to use the testing branch and i will fix up any issues if i
have to rebase as result of build tests.
Will.send Greg a pull later anyway, so should be less difference after he pulls that.
J
>> Please suggest me.
>> Thanks
>> Arushi
>> >
>> > > >
>> > > > Arushi Singhal (2):
>> > > > staging: ad7759: Replace mlock with driver
>private
>> lock
>> > >
>> > > This one is already been submitted. If you have a v2
>> for it,
>> > > then v2
>> > > the original patch.
>> > >
>> > > Is it submitted by me only before? And this is not the v2.
>> > > I have just resed it.
>> > > > staging: iio: ade7759: Move header content to
>> implementation
>> > > file
>> > >
>> > > This patch is done and applied already. See the
>> Coding Task #1
>> > > notes
>> > > in the IIO Tasks page.
>> > >
>> > > Not at applied I think so.
>> > > Thanks
>> > > Arushi
>> > >
>> > > alisons
>> > >
>> > > >
>> > > > drivers/staging/iio/meter/ade7759.c | 56
>> > > +++++++++++++++++++++++++++++++++--
>> > > > drivers/staging/iio/meter/ade7759.h | 53
>> > > ---------------------------------
>> > > > 2 files changed, 54 insertions(+), 57 deletions(-)
>> > > >
>> > > > --
>> > > > 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@googlegroups.com.
>> > > > To post to this group, send email to
>> > > outreachy-kernel@googlegroups.com.
>> > > > To view this discussion on the
>webvisithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737
>> -1-ar
>> > > ushisinghal19971997%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
>webvisithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0
>> H2z8x
>> > > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
>> > > For more options, visit https://groups.google.com/d/optout.
>> > >
>> > >
>>
>>
>>
>>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks
@ 2017-03-21 17:57 ` Jonathan Cameron
0 siblings, 0 replies; 14+ messages in thread
From: Jonathan Cameron @ 2017-03-21 17:57 UTC (permalink / raw)
To: Julia Lawall, Arushi Singhal
Cc: Alison Schofield, Jonathan Cameron, Hartmut Knaack,
outreachy-kernel, Michael Hennerich, lars, Greg KH, speakup,
driverdevel, linux-kernel, Peter Meerwald-Stadler, linux-iio
On 21 March 2017 17:27:02 GMT+00:00, Julia Lawall <julia.lawall@lip6.fr> wrote:
>
>
>On Tue, 21 Mar 2017, Arushi Singhal wrote:
>
>>
>>
>> On Tue, Mar 21, 2017 at 10:32 PM, Alison Schofield
><amsfield22@gmail.com>
>> wrote:
>> On Tue, Mar 21, 2017 at 05:39:38PM +0100, Julia Lawall wrote:
>> >
>> >
>> > On Tue, 21 Mar 2017, Arushi Singhal wrote:
>> >
>> > > On Tue, Mar 21, 2017 at 9:33 PM, Alison Schofield
>> <amsfield22@gmail.com>
>> > > wrote:
>> > > On Tue, Mar 21, 2017 at 07:00:17PM +0530, Arushi
>> Singhal wrote:
>> > > > Patchseries of IIO coding tasks
>> > >
>> > > This wouldn't be a patchset. Although they touch the
>> same
>> > > driver, the changes are unrelated. See more below...
>> > >
>> > > This I have send as a Patchset because as you have
>mentioned
>> below that the
>> > > [PATCH 2/2] was already done before by someone but I think
>> so it is not yet
>> > > applied in the Greg's tree yet.So I have done both the
>> changes and as they
>> > > should be applied one after other so that's why I have send
>> them as
>> > > Patchset.
>> >
>> > For the IIO patches, it is better to work on the IIO tree,
>not
>> Greg's.
>> > Greg manages staging, not IIO. The IIO patches should appear
>> in Greg's
>> > tree eventually.
>> >
>> > julia
>>
>> We didn't direct applicants to create an iio tree. At this
>> point,
>> it seems more than is necessary. They can follow the
>directions
>> in the
>> task descriptions and avoid the collisions.
>>
>> Of course, they are welcome to create a tree to iio/testing.
>>
>> (IMHO it's more overhead/busy work and maybe not the best use
>> of time in the home stretch of the application period.)
>>
>> alisons
>>
>>
>> Hi Alison
>> As you have mentioned that my [PATCH 2/2] is already being done
>someone. So
>> how can I make the changes of [PATCH 1/2] on top of it as [PATCH 2/2]
>is not
>> yet applied on the staging tree.
>
>Make a patch that applies to the current state of the IIO tree. Just
>clone that one like you cloned the staging one.
>
>julia
>
Slight amendment,. Normally would be the togreg branch which doesn't rebase.
With all the current churn might be best to use the testing branch and i will fix up any issues if i
have to rebase as result of build tests.
Will.send Greg a pull later anyway, so should be less difference after he pulls that.
J
>> Please suggest me.
>> Thanks
>> Arushi
>> >
>> > > >
>> > > > Arushi Singhal (2):
>> > > > staging: ad7759: Replace mlock with driver
>private
>> lock
>> > >
>> > > This one is already been submitted. If you have a v2
>> for it,
>> > > then v2
>> > > the original patch.
>> > >
>> > > Is it submitted by me only before? And this is not the v2.
>> > > I have just resed it.
>> > > > staging: iio: ade7759: Move header content to
>> implementation
>> > > file
>> > >
>> > > This patch is done and applied already. See the
>> Coding Task #1
>> > > notes
>> > > in the IIO Tasks page.
>> > >
>> > > Not at applied I think so.
>> > > Thanks
>> > > Arushi
>> > >
>> > > alisons
>> > >
>> > > >
>> > > > drivers/staging/iio/meter/ade7759.c | 56
>> > > +++++++++++++++++++++++++++++++++--
>> > > > drivers/staging/iio/meter/ade7759.h | 53
>> > > ---------------------------------
>> > > > 2 files changed, 54 insertions(+), 57 deletions(-)
>> > > >
>> > > > --
>> > > > 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@googlegroups.com.
>> > > > To post to this group, send email to
>> > > outreachy-kernel@googlegroups.com.
>> > > > To view this discussion on the
>webvisithttps://groups.google.com/d/msgid/outreachy-kernel/20170321133021.6737
>> -1-ar
>> > > ushisinghal19971997%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
>webvisithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF9dVy33Dsv0
>> H2z8x
>> > > taNeMOW7SQgr4qa4wLwz6xFNVTsUA%40mail.gmail.com.
>> > > For more options, visit https://groups.google.com/d/optout.
>> > >
>> > >
>>
>>
>>
>>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2017-03-21 18:04 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-21 13:30 [PATCH 0/2] IIO coding tasks Arushi Singhal
2017-03-21 13:30 ` [PATCH 1/2] staging: ad7759: Replace mlock with driver private lock Arushi Singhal
2017-03-21 13:30 ` [PATCH 2/2] staging: iio: ade7759: Move header content to implementation file Arushi Singhal
2017-03-21 16:03 ` [Outreachy kernel] [PATCH 0/2] IIO coding tasks Alison Schofield
2017-03-21 16:36 ` Arushi Singhal
2017-03-21 16:39 ` Julia Lawall
2017-03-21 17:02 ` Alison Schofield
2017-03-21 17:02 ` Alison Schofield
2017-03-21 17:09 ` Julia Lawall
2017-03-21 17:22 ` Arushi Singhal
2017-03-21 17:27 ` Julia Lawall
2017-03-21 17:57 ` Jonathan Cameron
2017-03-21 17:57 ` Jonathan Cameron
2017-03-21 17:33 ` Alison Schofield
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.