2016-11-21 21:50:22

by Moritz Fischer

[permalink] [raw]
Subject: [PATCH 1/2] MAINTAINERS: Update fpga-mgr entry with new linux-fpga mailing list

Update the fpga-mgr framework entry with new [email protected]
mailing list.

Signed-off-by: Moritz Fischer <[email protected]>
Cc: Alan Tull <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
Cc: Michal Simek <[email protected]>
Cc: Sören Brinkmann <[email protected]>
Cc: David S. Miller <[email protected]>
Cc: Geert Uytterhoeven <[email protected]>
Cc: Andrew Morton <[email protected]>
Cc: [email protected]
Cc: [email protected]
---

Hi all,

Alan & me have been talking about this during Plumbers, so I went ahead
and worked on getting it all set up.

The list is now up and running & you can sign up if you're interested,
we'll be using it in parallel to LKML in the beginning I think.

I'm currently working on getting a list archive setup over at marc.info.

If you have comments or suggestions on alternatives, send me an email.

Cheers,

Moritz
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 5e25ba1..19527a0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5005,6 +5005,7 @@ K: fmc_d.*register
FPGA MANAGER FRAMEWORK
M: Alan Tull <[email protected]>
R: Moritz Fischer <[email protected]>
+L: [email protected]
S: Maintained
F: drivers/fpga/
F: include/linux/fpga/fpga-mgr.h
--
2.7.4


2016-11-21 21:50:24

by Moritz Fischer

[permalink] [raw]
Subject: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

Add myself as co-maintainer to fpga mgr framework.

Signed-off-by: Moritz Fischer <[email protected]>
Cc: Alan Tull <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
Cc: [email protected]
Cc: [email protected]
---
Hi all,

Lately we've fallen behind a bit on reviewing patches lately.

'Behind the scenes', Alan has worked on getting a git tree setup
for us on kernel.org and I've worked on getting the mailing list setup.
The mailing list is up and running (see patch [1/2]) of this series.

Greg said he'd be happy to take pull requests from now on, so I hope
once we have figured out the exact workflow things will be much smoother
getting your patches merged into the fpga mgr framework.

Feel free to comment / speak up if you're not cool with this change.

Thanks,

Moritz

---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 19527a0..1856594 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5004,7 +5004,7 @@ K: fmc_d.*register

FPGA MANAGER FRAMEWORK
M: Alan Tull <[email protected]>
-R: Moritz Fischer <[email protected]>
+M: Moritz Fischer <[email protected]>
L: [email protected]
S: Maintained
F: drivers/fpga/
--
2.7.4

2016-11-21 22:09:32

by Marek Vasut

[permalink] [raw]
Subject: Re: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

On 11/21/2016 10:49 PM, Moritz Fischer wrote:
> Add myself as co-maintainer to fpga mgr framework.
>
> Signed-off-by: Moritz Fischer <[email protected]>
> Cc: Alan Tull <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: [email protected]
> Cc: [email protected]

Acked-by: Marek Vasut <[email protected]>

> ---
> Hi all,
>
> Lately we've fallen behind a bit on reviewing patches lately.
>
> 'Behind the scenes', Alan has worked on getting a git tree setup
> for us on kernel.org and I've worked on getting the mailing list setup.
> The mailing list is up and running (see patch [1/2]) of this series.
>
> Greg said he'd be happy to take pull requests from now on, so I hope
> once we have figured out the exact workflow things will be much smoother
> getting your patches merged into the fpga mgr framework.
>
> Feel free to comment / speak up if you're not cool with this change.
>
> Thanks,
>
> Moritz
>
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 19527a0..1856594 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5004,7 +5004,7 @@ K: fmc_d.*register
>
> FPGA MANAGER FRAMEWORK
> M: Alan Tull <[email protected]>
> -R: Moritz Fischer <[email protected]>
> +M: Moritz Fischer <[email protected]>
> L: [email protected]
> S: Maintained
> F: drivers/fpga/
>


--
Best regards,
Marek Vasut

2016-11-22 02:44:34

by atull

[permalink] [raw]
Subject: Re: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

On Mon, 21 Nov 2016, Moritz Fischer wrote:

> Add myself as co-maintainer to fpga mgr framework.
>
> Signed-off-by: Moritz Fischer <[email protected]>
> Cc: Alan Tull <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> ---
> Hi all,
>
> Lately we've fallen behind a bit on reviewing patches lately.

Hi Moritz,

drivers/fpga has been in the upstream kernel a year now. Most of that
time, traffic has been very slow. Recently we had more traffic while
I was travelling and moving to a new office, both cases leaving me
with bad network connectivity. Things will probably return to normal.
I appreciate your passion and all your effort reviewing stuff. I
don't see a need for two maintainers at this point.

Alan

>
> 'Behind the scenes', Alan has worked on getting a git tree setup
> for us on kernel.org and I've worked on getting the mailing list setup.
> The mailing list is up and running (see patch [1/2]) of this series.
>
> Greg said he'd be happy to take pull requests from now on, so I hope
> once we have figured out the exact workflow things will be much smoother
> getting your patches merged into the fpga mgr framework.
>
> Feel free to comment / speak up if you're not cool with this change.
>
> Thanks,
>
> Moritz
>
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 19527a0..1856594 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5004,7 +5004,7 @@ K: fmc_d.*register
>
> FPGA MANAGER FRAMEWORK
> M: Alan Tull <[email protected]>
> -R: Moritz Fischer <[email protected]>
> +M: Moritz Fischer <[email protected]>
> L: [email protected]
> S: Maintained
> F: drivers/fpga/
> --
> 2.7.4
>
>

2016-11-22 08:04:42

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

On 22.11.2016 03:29, atull wrote:
> On Mon, 21 Nov 2016, Moritz Fischer wrote:
>
>> Add myself as co-maintainer to fpga mgr framework.
>>
>> Signed-off-by: Moritz Fischer <[email protected]>
>> Cc: Alan Tull <[email protected]>
>> Cc: Greg Kroah-Hartman <[email protected]>
>> Cc: [email protected]
>> Cc: [email protected]
>> ---
>> Hi all,
>>
>> Lately we've fallen behind a bit on reviewing patches lately.
>
> Hi Moritz,
>
> drivers/fpga has been in the upstream kernel a year now. Most of that
> time, traffic has been very slow. Recently we had more traffic while
> I was travelling and moving to a new office, both cases leaving me
> with bad network connectivity. Things will probably return to normal.
> I appreciate your passion and all your effort reviewing stuff. I
> don't see a need for two maintainers at this point.

TBH. I think it is not a bad option. I do normally have backup person
for all repos I do maintain. It doesn't mean that second maintainer does
something but it has all accesses to repos you maintain.
It means if something really happens to you (hopefully not) than this
person can continue in this work without any delay which is not a bad
thing.
It is really just about talking to each other what that second person
will do - probably just review patches as is done now. You can also
learn from each other.
I would like to be involved more in this but unfortunately I don't have
enough time to do it properly.

Regarding maintaining this repo. It is just standard process. Apply
sensible things, well described and test it. And then send pull request
to Greg based on signed tags and you are done.
Greg should told you what should be the base which you should use for
pull request. Someone is taking patches based on rc1 tag, someone is
rebasing it on the final tag.

Acked-by: Michal Simek <[email protected]>

Thanks,
Michal


2016-11-22 08:07:06

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

On Tue, Nov 22, 2016 at 08:48:57AM +0100, Michal Simek wrote:
> On 22.11.2016 03:29, atull wrote:
> > On Mon, 21 Nov 2016, Moritz Fischer wrote:
> >
> >> Add myself as co-maintainer to fpga mgr framework.
> >>
> >> Signed-off-by: Moritz Fischer <[email protected]>
> >> Cc: Alan Tull <[email protected]>
> >> Cc: Greg Kroah-Hartman <[email protected]>
> >> Cc: [email protected]
> >> Cc: [email protected]
> >> ---
> >> Hi all,
> >>
> >> Lately we've fallen behind a bit on reviewing patches lately.
> >
> > Hi Moritz,
> >
> > drivers/fpga has been in the upstream kernel a year now. Most of that
> > time, traffic has been very slow. Recently we had more traffic while
> > I was travelling and moving to a new office, both cases leaving me
> > with bad network connectivity. Things will probably return to normal.
> > I appreciate your passion and all your effort reviewing stuff. I
> > don't see a need for two maintainers at this point.
>
> TBH. I think it is not a bad option. I do normally have backup person
> for all repos I do maintain. It doesn't mean that second maintainer does
> something but it has all accesses to repos you maintain.
> It means if something really happens to you (hopefully not) than this
> person can continue in this work without any delay which is not a bad
> thing.
> It is really just about talking to each other what that second person
> will do - probably just review patches as is done now. You can also
> learn from each other.
> I would like to be involved more in this but unfortunately I don't have
> enough time to do it properly.
>
> Regarding maintaining this repo. It is just standard process. Apply
> sensible things, well described and test it. And then send pull request
> to Greg based on signed tags and you are done.
> Greg should told you what should be the base which you should use for
> pull request. Someone is taking patches based on rc1 tag, someone is
> rebasing it on the final tag.

Greg doesn't care what base you use, as long as you don't rebase
patches. What subsystem does that? I need to go yell at someone...

And I take patches just as easily, what ever works best for the
subsystem.

thanks,

greg k-h

2016-11-22 08:28:45

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

On 22.11.2016 09:07, Greg Kroah-Hartman wrote:
> On Tue, Nov 22, 2016 at 08:48:57AM +0100, Michal Simek wrote:
>> On 22.11.2016 03:29, atull wrote:
>>> On Mon, 21 Nov 2016, Moritz Fischer wrote:
>>>
>>>> Add myself as co-maintainer to fpga mgr framework.
>>>>
>>>> Signed-off-by: Moritz Fischer <[email protected]>
>>>> Cc: Alan Tull <[email protected]>
>>>> Cc: Greg Kroah-Hartman <[email protected]>
>>>> Cc: [email protected]
>>>> Cc: [email protected]
>>>> ---
>>>> Hi all,
>>>>
>>>> Lately we've fallen behind a bit on reviewing patches lately.
>>>
>>> Hi Moritz,
>>>
>>> drivers/fpga has been in the upstream kernel a year now. Most of that
>>> time, traffic has been very slow. Recently we had more traffic while
>>> I was travelling and moving to a new office, both cases leaving me
>>> with bad network connectivity. Things will probably return to normal.
>>> I appreciate your passion and all your effort reviewing stuff. I
>>> don't see a need for two maintainers at this point.
>>
>> TBH. I think it is not a bad option. I do normally have backup person
>> for all repos I do maintain. It doesn't mean that second maintainer does
>> something but it has all accesses to repos you maintain.
>> It means if something really happens to you (hopefully not) than this
>> person can continue in this work without any delay which is not a bad
>> thing.
>> It is really just about talking to each other what that second person
>> will do - probably just review patches as is done now. You can also
>> learn from each other.
>> I would like to be involved more in this but unfortunately I don't have
>> enough time to do it properly.
>>
>> Regarding maintaining this repo. It is just standard process. Apply
>> sensible things, well described and test it. And then send pull request
>> to Greg based on signed tags and you are done.
>> Greg should told you what should be the base which you should use for
>> pull request. Someone is taking patches based on rc1 tag, someone is
>> rebasing it on the final tag.
>
> Greg doesn't care what base you use, as long as you don't rebase
> patches. What subsystem does that? I need to go yell at someone...

:-) Not sure if there is any subsystem which use this.
And for my microblaze tree I do normally base patches on the top of
rc7/8 or final tag but that's done mostly because I collect patches at
that time. But IIRC when I started with this someone mentioned that for
small number of patches this is fine.

Thanks,
Michal

2016-11-22 16:36:48

by Jason Gunthorpe

[permalink] [raw]
Subject: Re: [PATCH 2/2] MAINTAINERS: Add myself as co-maintainer to fpga mgr framework.

On Tue, Nov 22, 2016 at 08:48:57AM +0100, Michal Simek wrote:

> TBH. I think it is not a bad option. I do normally have backup person
> for all repos I do maintain. It doesn't mean that second maintainer does
> something but it has all accesses to repos you maintain.

I agree, it is a good idea to have a maintainer team model as early as
possible just because stuff does happen. We've had to recover from MIA
maintainers in RDMA and TPM, and it is not really nice. Generally
a lot of patches just get dropped :(

Jason