<div dir="ltr">Fine, I will try to catch up.<div>Thanks anyway.</div><div><br></div><div>Have a nice day,</div><div>John</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 13, 2015 at 11:07 AM, Rob Clark <span dir="ltr"><<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Mar 12, 2015 at 10:53 PM, John Hunter <<a href="mailto:zhjwpku@gmail.com">zhjwpku@gmail.com</a>> wrote:<br>
> Got it, maybe I should work on the drm-next or latest linux master.<br>
><br>
> I am working on the drm git repo below for now as Daniel Vetter told me<br>
> git://<a href="http://people.freedesktop.org/~airlied/linux" target="_blank">people.freedesktop.org/~airlied/linux</a><br>
> I am quite confused now 'cause there are a bounch of git repo, I really<br>
> do not know which one should I git clone.<br>
><br>
<br>
</span>Hmm, I suspect airlied hasn't updated his master branch in a while,<br>
which is probably causing confusing..  using his drm-next branch or<br>
linus master would be better..  in general, drm-next for latest and<br>
greatest drm is probably best, but right at the moment there is<br>
probably not much difference from linus master.<br>
<br>
I usually end up 'git remote add'ing a bunch of different trees to<br>
track both what has already landed in linus's master branch, and also<br>
what is coming in drm-next (and various different arm trees related to<br>
other stuff I work on)..  but for drm stuff, Dave's drm-next branch is<br>
a safe bet.<br>
<br>
BR,<br>
-R<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
><br>
> On Fri, Mar 13, 2015 at 10:33 AM, Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>> wrote:<br>
>><br>
>> On Thu, Mar 12, 2015 at 4:02 AM, John Hunter <<a href="mailto:zhaojunwang@pku.edu.cn">zhaojunwang@pku.edu.cn</a>><br>
>> wrote:<br>
>> > From: John Hunter <<a href="mailto:zhjwpku@gmail.com">zhjwpku@gmail.com</a>><br>
>> ><br>
>> > IMHO, some annotations were copypaste from somewhere else, it is<br>
>> > obviously not rightly modified. Hope I am right with that.<br>
>> > ---<br>
>> >  drivers/gpu/drm/drm_crtc.c | 115<br>
>> > ++++++++++++++++++++++-----------------------<br>
>> >  1 file changed, 56 insertions(+), 59 deletions(-)<br>
>> ><br>
>> > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c<br>
>> > index f2d667b..7f771e9 100644<br>
>> > --- a/drivers/gpu/drm/drm_crtc.c<br>
>> > +++ b/drivers/gpu/drm/drm_crtc.c<br>
>> > @@ -205,12 +205,12 @@ char *drm_get_connector_status_name(enum<br>
>> > drm_connector_status status)<br>
>> >  /**<br>
>> >   * drm_mode_object_get - allocate a new identifier<br>
>> >   * @dev: DRM device<br>
>> > - * @ptr: object pointer, used to generate unique ID<br>
>> > - * @type: object type<br>
>> > + * @obj: object pointer, used to generate unique ID<br>
>> > + * @obj_type: object type<br>
>> >   *<br>
>> >   * LOCKING:<br>
>> >   *<br>
>> > - * Create a unique identifier based on @ptr in @dev's identifier space.<br>
>> > Used<br>
>> > + * Create a unique identifier based on @obj in @dev's identifier space.<br>
>> > Used<br>
>> >   * for tracking modes, CRTCs and connectors.<br>
>><br>
>> so, I don't suppose you can rebase against a newer upstream kernel?<br>
>><br>
>> From a quick look, it seems like the arg name errors where corrected<br>
>> in 065a50ed3ef75cb265e12e3e1b615db0835150bc while the description<br>
>> bodies fixes from your patch are still applicable.  In the current<br>
>> form, the patch does not apply against drm-next (or latest linux<br>
>> master), but most of it is still applicable and would be a very<br>
>> welcome improvement.<br>
>><br>
>> Thanks :-)<br>
>><br>
>> BR,<br>
>> -R<br>
>><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Best regards<br></div><div>Junwang Zhao</div><div>Microprocessor Research and Develop Center</div><div>Department of Computer Science &Technology</div><div>Peking University</div><div>Beijing, 100871, PRC</div></div></div>
</div>