public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: "Deucher, Alexander" <Alexander.Deucher@amd.com>
Cc: "Alex Deucher" <alexdeucher@gmail.com>,
	"Baoyou Xie" <baoyou.xie@linaro.org>,
	"Dave Airlie" <airlied@linux.ie>, "Zhu, Rex" <Rex.Zhu@amd.com>,
	"Zhou, Jammy" <Jammy.Zhou@amd.com>,
	"Huang, JinHuiEric" <JinHuiEric.Huang@amd.com>,
	"StDenis, Tom" <Tom.StDenis@amd.com>,
	"Edward O'Callaghan" <funfunctor@folklore1984.net>,
	"Prosyak, Vitaly" <Vitaly.Prosyak@amd.com>,
	"Yang, Eric" <Eric.Yang2@amd.com>,
	"Yang, Young" <Young.Yang@amd.com>,
	"Huang, Ray" <Ray.Huang@amd.com>,
	"Dan Carpenter" <dan.carpenter@oracle.com>,
	"Cui, Flora" <Flora.Cui@amd.com>,
	"Nils Wallménius" <nils.wallmenius@gmail.com>,
	"Liu, Monk" <Monk.Liu@amd.com>, "Wang, Ken" <Ken.Wang@amd.com>,
	"Min, Frank" <Frank.Min@amd.com>,
	"tang.qiang007@zte.com.cn" <tang.qiang007@zte.com.cn>,
	"xie.baoyou@zte.com.cn" <xie.baoyou@zte.com.cn>,
	LKML <linux-kernel@vger.kernel.org>,
	"Maling list - DRI developers" <dri-devel@lists.freedesktop.org>,
	"han.fei@zte.com.cn" <han.fei@zte.com.cn>
Subject: Re: [PATCH] drm/amd/powerplay: mark symbols static where possible
Date: Mon, 24 Oct 2016 22:41:16 +0200	[thread overview]
Message-ID: <42945045.1PbGIglsmX@wuerfel> (raw)
In-Reply-To: <MWHPR12MB1694ABD9A93A102EB461C376F7A90@MWHPR12MB1694.namprd12.prod.outlook.com>

On Monday, October 24, 2016 8:07:16 PM CEST Deucher, Alexander wrote:
> > > > In fact, these functions are only used in the file in which they are
> > > > declared and don't need a declaration, but can be made static.
> > > > So this patch marks these functions with 'static'.
> > > >
> > > > Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
> > >
> > > This was already applied the last time you sent it out.  Sorry if I
> > > didn't mention that previously.
> > 
> > For some reason the patch hasn't made it into linux-next, so I can see
> > why Baoyou was getting confused here. Can you clarify what the timeline
> > is for the AMD DRM driver patches from between they get applied to the
> > AMD tree to when they make it into linux-next?
> > 
> 
> It came in late enough last cycle that it didn't make it into 4.9 (this is just a clean up not a critical bug fix), so I queued it for 4.10.  I try to reply when I apply a patch, but sometimes I miss one here and there.  Once Dave starts the drm-next tree for 4.10, it will be included in my pull request.  Pending -next patches are in my drm-next-<kernel version>-wip tree until I send Dave a formal request.
> 
> > I've occasionally had a hard time with DRM (and a few other subsystems)
> > with bugfix patches trying to find out whether they got lost or
> > whether they just haven't made it into -next but are in some other tree.
> > 
> 
> For bug fixes we usually send Dave ~weekly pull requests for each -rc as necessary.  For -next stuff, each driver usually sends at least one, sometimes several pull requests for the next merge window.

Ok, got it. Thanks for the detailed reply!

Do you think it would be appropriate to include your drm-next-wip tree in
linux-next? I think this is how a lot of the multi-level maintainer
setups work as it give faster feedback about when things break.

	Arnd

  reply	other threads:[~2016-10-24 20:42 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-22  8:56 [PATCH] drm/amd/powerplay: mark symbols static where possible Baoyou Xie
2016-10-22  9:28 ` Edward O'Callaghan
2016-10-24 16:36 ` Alex Deucher
2016-10-24 19:49   ` Arnd Bergmann
2016-10-24 20:07     ` Deucher, Alexander
2016-10-24 20:41       ` Arnd Bergmann [this message]
2016-10-25  6:41         ` Daniel Vetter
2016-10-25  7:09           ` Christian König
2016-10-25  8:11             ` Daniel Vetter
2016-10-24 20:51 ` Arnd Bergmann
     [not found]   ` <CA+DQWkxdO72H4ZbUkYyJwPOe2JGRv_Tz1eUkZ5=FJoOonoWp=Q@mail.gmail.com>
2016-10-25  8:15     ` Arnd Bergmann
2016-10-25 20:07       ` Deucher, Alexander

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=42945045.1PbGIglsmX@wuerfel \
    --to=arnd@arndb.de \
    --cc=Alexander.Deucher@amd.com \
    --cc=Eric.Yang2@amd.com \
    --cc=Flora.Cui@amd.com \
    --cc=Frank.Min@amd.com \
    --cc=Jammy.Zhou@amd.com \
    --cc=JinHuiEric.Huang@amd.com \
    --cc=Ken.Wang@amd.com \
    --cc=Monk.Liu@amd.com \
    --cc=Ray.Huang@amd.com \
    --cc=Rex.Zhu@amd.com \
    --cc=Tom.StDenis@amd.com \
    --cc=Vitaly.Prosyak@amd.com \
    --cc=Young.Yang@amd.com \
    --cc=airlied@linux.ie \
    --cc=alexdeucher@gmail.com \
    --cc=baoyou.xie@linaro.org \
    --cc=dan.carpenter@oracle.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=funfunctor@folklore1984.net \
    --cc=han.fei@zte.com.cn \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nils.wallmenius@gmail.com \
    --cc=tang.qiang007@zte.com.cn \
    --cc=xie.baoyou@zte.com.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox