2024-01-22 19:15:25

by Jakub Kicinski

[permalink] [raw]
Subject: Re: [PATCH net-next 00/22] Fix MODULE_DESCRIPTION() for net (p2)

On Mon, 22 Jan 2024 10:45:21 -0800 Breno Leitao wrote:
> net: fill in MODULE_DESCRIPTION()s for Broadcom WLAN
> net: fill in MODULE_DESCRIPTION()s for wlcore
> net: fill in MODULE_DESCRIPTION()s for wl1251 and wl12xx

Thanks for making progress on these!

When you repost:
- please send these 3 to linux-wireless as a separate series
- make sure mailing lists are CCed on the cover letter
(yes, get_maintainer is embarrassingly bad at its job)
- please send 10 at-a-time, it's probably a good tradeoff
between series size and reposting risk
- please target net, I hope I convinced Paolo that it's okay :)


2024-01-23 16:25:43

by Breno Leitao

[permalink] [raw]
Subject: Re: [PATCH net-next 00/22] Fix MODULE_DESCRIPTION() for net (p2)

On Mon, Jan 22, 2024 at 10:57:08AM -0800, Jakub Kicinski wrote:
> On Mon, 22 Jan 2024 10:45:21 -0800 Breno Leitao wrote:
> > net: fill in MODULE_DESCRIPTION()s for Broadcom WLAN
> > net: fill in MODULE_DESCRIPTION()s for wlcore
> > net: fill in MODULE_DESCRIPTION()s for wl1251 and wl12xx
>
> Thanks for making progress on these!
>
> When you repost:
> - please send these 3 to linux-wireless as a separate series
> - make sure mailing lists are CCed on the cover letter
> (yes, get_maintainer is embarrassingly bad at its job)
> - please send 10 at-a-time, it's probably a good tradeoff
> between series size and reposting risk
> - please target net, I hope I convinced Paolo that it's okay :)

Sure. I will split this series in 3 and target `net`.

I suppose it is OK to send the patchsets in parallel, instead of waiting
for the first patchset to be reviewed/accepted before sending the second
part. Is this correct?

Thanks!

2024-01-23 16:36:28

by Jakub Kicinski

[permalink] [raw]
Subject: Re: [PATCH net-next 00/22] Fix MODULE_DESCRIPTION() for net (p2)

On Tue, 23 Jan 2024 08:25:28 -0800 Breno Leitao wrote:
> > When you repost:
> > - please send these 3 to linux-wireless as a separate series
> > - make sure mailing lists are CCed on the cover letter
> > (yes, get_maintainer is embarrassingly bad at its job)
> > - please send 10 at-a-time, it's probably a good tradeoff
> > between series size and reposting risk
> > - please target net, I hope I convinced Paolo that it's okay :)
>
> Sure. I will split this series in 3 and target `net`.

The wireless ones need to target wireless, just to be clear.
The rest should fit into 2 series.

> I suppose it is OK to send the patchsets in parallel, instead of waiting
> for the first patchset to be reviewed/accepted before sending the second
> part. Is this correct?

The rate limit is to avoid having to give the same feedback to multiple
series, among other things. it'd be better to send one part at a time..
it won't take that long :)

2024-01-23 17:17:47

by Breno Leitao

[permalink] [raw]
Subject: Re: [PATCH net-next 00/22] Fix MODULE_DESCRIPTION() for net (p2)

On Tue, Jan 23, 2024 at 08:35:17AM -0800, Jakub Kicinski wrote:
> On Tue, 23 Jan 2024 08:25:28 -0800 Breno Leitao wrote:
> > > When you repost:
> > > - please send these 3 to linux-wireless as a separate series
> > > - make sure mailing lists are CCed on the cover letter
> > > (yes, get_maintainer is embarrassingly bad at its job)
> > > - please send 10 at-a-time, it's probably a good tradeoff
> > > between series size and reposting risk
> > > - please target net, I hope I convinced Paolo that it's okay :)
> >
> > Sure. I will split this series in 3 and target `net`.
>
> The wireless ones need to target wireless, just to be clear.
> The rest should fit into 2 series.
>
> > I suppose it is OK to send the patchsets in parallel, instead of waiting
> > for the first patchset to be reviewed/accepted before sending the second
> > part. Is this correct?
>
> The rate limit is to avoid having to give the same feedback to multiple
> series, among other things. it'd be better to send one part at a time..
> it won't take that long :)

Sure. Thanks for the clarifications!