Re: [Outreachy kernel] [PATCH 0/2] IIO coding tasks

From: Jonathan Cameron
Date: Tue Mar 21 2017 - 14:04:48 EST




On 21 March 2017 17:27:02 GMT+00:00, Julia Lawall <julia.lawall@xxxxxxx> wrote:
>
>
>On Tue, 21 Mar 2017, Arushi Singhal wrote:
>
>>
>>
>> On Tue, Mar 21, 2017 at 10:32 PM, Alison Schofield
><amsfield22@xxxxxxxxx>
>> 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@xxxxxxxxx>
>> > > 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@xxxxxxxxxxxxxxxxx
>> > > > To post to this group, send email to
>> > > outreachy-kernel@xxxxxxxxxxxxxxxxx
>> > > > 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@xxxxxxxxxxxxxxxxx
>> > > To post to this group, send email to
>> outreachy-kernel@xxxxxxxxxxxxxxxxx
>> > > 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.