* [PATCH] drm/doc: Fix up some kms function names @ 2017-02-22 8:47 Archit Taneja 2017-02-24 13:41 ` Eric Engestrom 0 siblings, 1 reply; 5+ messages in thread From: Archit Taneja @ 2017-02-22 8:47 UTC (permalink / raw) To: daniel.vetter; +Cc: dri-devel A couple of the kms functions didn't have the correct/newest names. This prevented them to be identified as refs in the html doc. Signed-off-by: Archit Taneja <architt@codeaurora.org> --- include/drm/drm_mode_config.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h index 6cd541d69c2e..888cc038899b 100644 --- a/include/drm/drm_mode_config.h +++ b/include/drm/drm_mode_config.h @@ -267,7 +267,7 @@ struct drm_mode_config_funcs { * passed-in &drm_atomic_state. This hook is called when the caller * encountered a &drm_modeset_lock deadlock and needs to drop all * already acquired locks as part of the deadlock avoidance dance - * implemented in drm_modeset_lock_backoff(). + * implemented in drm_modeset_backoff(). * * Any duplicated state must be invalidated since a concurrent atomic * update might change it, and the drm atomic interfaces always apply @@ -285,8 +285,8 @@ struct drm_mode_config_funcs { * itself. Note that the core first calls drm_atomic_state_clear() to * avoid code duplicate between the clear and free hooks. * - * Drivers that implement this must call drm_atomic_state_default_free() - * to release common resources. + * Drivers that implement this must call + * drm_atomic_state_default_release() to release common resources. */ void (*atomic_state_free)(struct drm_atomic_state *state); }; -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/doc: Fix up some kms function names 2017-02-22 8:47 [PATCH] drm/doc: Fix up some kms function names Archit Taneja @ 2017-02-24 13:41 ` Eric Engestrom 2017-02-26 13:44 ` Archit Taneja 0 siblings, 1 reply; 5+ messages in thread From: Eric Engestrom @ 2017-02-24 13:41 UTC (permalink / raw) To: Archit Taneja; +Cc: daniel.vetter, dri-devel On Wednesday, 2017-02-22 14:17:41 +0530, Archit Taneja wrote: > A couple of the kms functions didn't have the correct/newest names. > This prevented them to be identified as refs in the html doc. > > Signed-off-by: Archit Taneja <architt@codeaurora.org> Thanks! Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> Tested-by: Eric Engestrom <eric.engestrom@imgtec.com> There's a lot of other symbols not ref-linked because of incorrect syntax, including in this file; can I interest you in fixing those as well? :) > --- > include/drm/drm_mode_config.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h > index 6cd541d69c2e..888cc038899b 100644 > --- a/include/drm/drm_mode_config.h > +++ b/include/drm/drm_mode_config.h > @@ -267,7 +267,7 @@ struct drm_mode_config_funcs { > * passed-in &drm_atomic_state. This hook is called when the caller > * encountered a &drm_modeset_lock deadlock and needs to drop all > * already acquired locks as part of the deadlock avoidance dance > - * implemented in drm_modeset_lock_backoff(). > + * implemented in drm_modeset_backoff(). > * > * Any duplicated state must be invalidated since a concurrent atomic > * update might change it, and the drm atomic interfaces always apply > @@ -285,8 +285,8 @@ struct drm_mode_config_funcs { > * itself. Note that the core first calls drm_atomic_state_clear() to > * avoid code duplicate between the clear and free hooks. > * > - * Drivers that implement this must call drm_atomic_state_default_free() > - * to release common resources. > + * Drivers that implement this must call > + * drm_atomic_state_default_release() to release common resources. > */ > void (*atomic_state_free)(struct drm_atomic_state *state); > }; > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > hosted by The Linux Foundation > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/doc: Fix up some kms function names 2017-02-24 13:41 ` Eric Engestrom @ 2017-02-26 13:44 ` Archit Taneja 2017-02-26 21:47 ` Daniel Vetter 0 siblings, 1 reply; 5+ messages in thread From: Archit Taneja @ 2017-02-26 13:44 UTC (permalink / raw) To: Eric Engestrom; +Cc: daniel.vetter, dri-devel On 2/24/2017 7:11 PM, Eric Engestrom wrote: > On Wednesday, 2017-02-22 14:17:41 +0530, Archit Taneja wrote: >> A couple of the kms functions didn't have the correct/newest names. >> This prevented them to be identified as refs in the html doc. >> >> Signed-off-by: Archit Taneja <architt@codeaurora.org> > > Thanks! > Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> > Tested-by: Eric Engestrom <eric.engestrom@imgtec.com> > > There's a lot of other symbols not ref-linked because of incorrect syntax, > including in this file; can I interest you in fixing those as well? :) Yeah, I'll try to fix them as I continue reading the docs :) Archit > >> --- >> include/drm/drm_mode_config.h | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h >> index 6cd541d69c2e..888cc038899b 100644 >> --- a/include/drm/drm_mode_config.h >> +++ b/include/drm/drm_mode_config.h >> @@ -267,7 +267,7 @@ struct drm_mode_config_funcs { >> * passed-in &drm_atomic_state. This hook is called when the caller >> * encountered a &drm_modeset_lock deadlock and needs to drop all >> * already acquired locks as part of the deadlock avoidance dance >> - * implemented in drm_modeset_lock_backoff(). >> + * implemented in drm_modeset_backoff(). >> * >> * Any duplicated state must be invalidated since a concurrent atomic >> * update might change it, and the drm atomic interfaces always apply >> @@ -285,8 +285,8 @@ struct drm_mode_config_funcs { >> * itself. Note that the core first calls drm_atomic_state_clear() to >> * avoid code duplicate between the clear and free hooks. >> * >> - * Drivers that implement this must call drm_atomic_state_default_free() >> - * to release common resources. >> + * Drivers that implement this must call >> + * drm_atomic_state_default_release() to release common resources. >> */ >> void (*atomic_state_free)(struct drm_atomic_state *state); >> }; >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> hosted by The Linux Foundation >> -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/doc: Fix up some kms function names 2017-02-26 13:44 ` Archit Taneja @ 2017-02-26 21:47 ` Daniel Vetter 2017-02-27 3:27 ` Archit Taneja 0 siblings, 1 reply; 5+ messages in thread From: Daniel Vetter @ 2017-02-26 21:47 UTC (permalink / raw) To: Archit Taneja; +Cc: daniel.vetter, dri-devel On Sun, Feb 26, 2017 at 07:14:07PM +0530, Archit Taneja wrote: > > > On 2/24/2017 7:11 PM, Eric Engestrom wrote: > > On Wednesday, 2017-02-22 14:17:41 +0530, Archit Taneja wrote: > > > A couple of the kms functions didn't have the correct/newest names. > > > This prevented them to be identified as refs in the html doc. > > > > > > Signed-off-by: Archit Taneja <architt@codeaurora.org> > > > > Thanks! > > Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> > > Tested-by: Eric Engestrom <eric.engestrom@imgtec.com> > > > > There's a lot of other symbols not ref-linked because of incorrect syntax, > > including in this file; can I interest you in fixing those as well? :) > > Yeah, I'll try to fix them as I continue reading the docs :) Since you have drm-misc commit rights, I assume you'll push this directly? -Daniel > > Archit > > > > > > --- > > > include/drm/drm_mode_config.h | 6 +++--- > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h > > > index 6cd541d69c2e..888cc038899b 100644 > > > --- a/include/drm/drm_mode_config.h > > > +++ b/include/drm/drm_mode_config.h > > > @@ -267,7 +267,7 @@ struct drm_mode_config_funcs { > > > * passed-in &drm_atomic_state. This hook is called when the caller > > > * encountered a &drm_modeset_lock deadlock and needs to drop all > > > * already acquired locks as part of the deadlock avoidance dance > > > - * implemented in drm_modeset_lock_backoff(). > > > + * implemented in drm_modeset_backoff(). > > > * > > > * Any duplicated state must be invalidated since a concurrent atomic > > > * update might change it, and the drm atomic interfaces always apply > > > @@ -285,8 +285,8 @@ struct drm_mode_config_funcs { > > > * itself. Note that the core first calls drm_atomic_state_clear() to > > > * avoid code duplicate between the clear and free hooks. > > > * > > > - * Drivers that implement this must call drm_atomic_state_default_free() > > > - * to release common resources. > > > + * Drivers that implement this must call > > > + * drm_atomic_state_default_release() to release common resources. > > > */ > > > void (*atomic_state_free)(struct drm_atomic_state *state); > > > }; > > > -- > > > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > > > hosted by The Linux Foundation > > > > > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > hosted by The Linux Foundation > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/doc: Fix up some kms function names 2017-02-26 21:47 ` Daniel Vetter @ 2017-02-27 3:27 ` Archit Taneja 0 siblings, 0 replies; 5+ messages in thread From: Archit Taneja @ 2017-02-27 3:27 UTC (permalink / raw) To: Daniel Vetter; +Cc: daniel.vetter, dri-devel On 02/27/2017 03:17 AM, Daniel Vetter wrote: > On Sun, Feb 26, 2017 at 07:14:07PM +0530, Archit Taneja wrote: >> >> >> On 2/24/2017 7:11 PM, Eric Engestrom wrote: >>> On Wednesday, 2017-02-22 14:17:41 +0530, Archit Taneja wrote: >>>> A couple of the kms functions didn't have the correct/newest names. >>>> This prevented them to be identified as refs in the html doc. >>>> >>>> Signed-off-by: Archit Taneja <architt@codeaurora.org> >>> >>> Thanks! >>> Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> >>> Tested-by: Eric Engestrom <eric.engestrom@imgtec.com> >>> >>> There's a lot of other symbols not ref-linked because of incorrect syntax, >>> including in this file; can I interest you in fixing those as well? :) >> >> Yeah, I'll try to fix them as I continue reading the docs :) > > Since you have drm-misc commit rights, I assume you'll push this directly? Yes. Applied. Archit > -Daniel > >> >> Archit >> >>> >>>> --- >>>> include/drm/drm_mode_config.h | 6 +++--- >>>> 1 file changed, 3 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h >>>> index 6cd541d69c2e..888cc038899b 100644 >>>> --- a/include/drm/drm_mode_config.h >>>> +++ b/include/drm/drm_mode_config.h >>>> @@ -267,7 +267,7 @@ struct drm_mode_config_funcs { >>>> * passed-in &drm_atomic_state. This hook is called when the caller >>>> * encountered a &drm_modeset_lock deadlock and needs to drop all >>>> * already acquired locks as part of the deadlock avoidance dance >>>> - * implemented in drm_modeset_lock_backoff(). >>>> + * implemented in drm_modeset_backoff(). >>>> * >>>> * Any duplicated state must be invalidated since a concurrent atomic >>>> * update might change it, and the drm atomic interfaces always apply >>>> @@ -285,8 +285,8 @@ struct drm_mode_config_funcs { >>>> * itself. Note that the core first calls drm_atomic_state_clear() to >>>> * avoid code duplicate between the clear and free hooks. >>>> * >>>> - * Drivers that implement this must call drm_atomic_state_default_free() >>>> - * to release common resources. >>>> + * Drivers that implement this must call >>>> + * drm_atomic_state_default_release() to release common resources. >>>> */ >>>> void (*atomic_state_free)(struct drm_atomic_state *state); >>>> }; >>>> -- >>>> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >>>> hosted by The Linux Foundation >>>> >> >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> hosted by The Linux Foundation >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-02-27 3:27 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-02-22 8:47 [PATCH] drm/doc: Fix up some kms function names Archit Taneja 2017-02-24 13:41 ` Eric Engestrom 2017-02-26 13:44 ` Archit Taneja 2017-02-26 21:47 ` Daniel Vetter 2017-02-27 3:27 ` Archit Taneja
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.