On Wed 2015-04-08 10:45:07, Lee Jones wrote:
> On Wed, 08 Apr 2015, Pavel Machek wrote:
>
> > On Wed 2015-04-08 08:26:34, Lee Jones wrote:
> > > On Tue, 07 Apr 2015, Eduardo Valentin wrote:
> > >
> > > >
> > > > This is a simple automated notification to let you know that
> > > > I've just added the patch titled
> > > >
> > > > drivers: thermal: st: remove several sparse warnings
> > > >
> > > > to my thermal-soc git tree which can be found at
> > > >
> > > > git://git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal.git
> > > > in the fixes branch.
> > > >
> > > > The patch will show up in the next release of the linux-next tree
> > > > (usually sometime within the next 24 hours during the week.)
> > > >
> > > > The patch will hopefully also be merged in Linus's tree for the
> > > > next -rc kernel release.
> > > >
> > > > If you have any questions about this process, please let me know.
> > > >
> > > > All the best,
> > > >
> > > > Eduardo Valentin
> > > >
> > > > ----------
> > > > From 541d529f9845d249d1cb84f1b395e48f0a117e3f Mon Sep 17 00:00:00 2001
> > > > From: Eduardo Valentin <edubez...@gmail.com>
> > > > Date: Tue, 7 Apr 2015 13:42:12 -0700
> > > > Subject: drivers: thermal: st: remove several sparse warnings
> > > >
> > > > Simple patch to make symbols static. Symbols that are not
> > > > shared with other parts of the kernel can be made static.
> > > > This change also removes several sparse complains.
> > > >
> > > > Cc: Zhang Rui <rui.zh...@intel.com>
> > > > Cc: Lee Jones <lee.jo...@linaro.org>
> > > > Cc: Pavel Machek <pa...@ucw.cz>
> > > > Cc: Ajit Pal Singh <ajitpal.si...@st.com>
> > > > Cc: linux...@vger.kernel.org
> > > > Cc: linux-kernel@vger.kernel.org
> > >
> > > Hold on a second, you can't do that.
> > >
> > > Adding these Cc: tags means "this person has been given the opportunity
> > > to comment on the patch but has chosen not to". However, you've
> > > applied this patch ~1min after sending it to the lists.
> >
> > Cc means -- that person was Cced.
>
> You just made that up out of thin air. That's not what it means at all.
>
> Documentation/SubmittingPatches:
>
> "If a person has had the opportunity to comment on a patch,
> but has not provided such comments, you may optionally add a
> "Cc:" tag to the patch. This is the only tag which might be
> added without an explicit action by the person it names - but
> it should indicate that this person was copied on the patch.
> This tag documents that potentially interested parties have
> been included in the discussion."
>
> To add people on Cc, then immediately apply the patch is fundamentally
> wrong.
That is not how I seen it working before.
> > And there's no harm in having patch available in git for easy testing.
>
> If we weren't so deep into the release cycle, I'd agree with you. But
> even if we weren't so close to the merge-window, I'd still expect a
> note to the tune of "tentatively applying this for reason X for early
> soak testing in -next". Although X would have to be a pretty good
> reason, as that's not usually how we do things.
Patches should spend a week or so in -next (that's documented
somewhere), so people in the Cc list will have their chance to
comment.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures)
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
