* [PATCH] drm/drm_atomic.h: fixup typos
@ 2023-05-17 8:23 Sui Jingfeng
2023-05-17 8:38 ` Javier Martinez Canillas
0 siblings, 1 reply; 4+ messages in thread
From: Sui Jingfeng @ 2023-05-17 8:23 UTC (permalink / raw)
To: Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann,
David Airlie, Daniel Vetter, suijingfeng, liyi
Cc: dri-devel, linux-kernel, loongson-kernel
Signed-off-by: Sui Jingfeng <suijingfeng@loongson.cn>
---
include/drm/drm_atomic.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index c023c682301b..0b5271cfb029 100644
--- a/include/drm/drm_atomic.h
+++ b/include/drm/drm_atomic.h
@@ -394,8 +394,7 @@ struct drm_atomic_state {
*
* Indicates whether or not this atomic state was duplicated using
* drm_atomic_helper_duplicate_state(). Drivers and atomic helpers
- * should use this to fixup normal inconsistencies in duplicated
- * states.
+ * should use this to fixup inconsistencies in duplicated states.
*/
bool duplicated : 1;
struct __drm_planes_state *planes;
@@ -412,7 +411,7 @@ struct drm_atomic_state {
*
* Used for signaling unbound planes/connectors.
* When a connector or plane is not bound to any CRTC, it's still important
- * to preserve linearity to prevent the atomic states from being freed to early.
+ * to preserve linearity to prevent the atomic states from being freed too early.
*
* This commit (if set) is not bound to any CRTC, but will be completed when
* drm_atomic_helper_commit_hw_done() is called.
--
2.25.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/drm_atomic.h: fixup typos
2023-05-17 8:23 [PATCH] drm/drm_atomic.h: fixup typos Sui Jingfeng
@ 2023-05-17 8:38 ` Javier Martinez Canillas
2023-05-17 8:48 ` Sui Jingfeng
2023-05-17 8:53 ` Sui Jingfeng
0 siblings, 2 replies; 4+ messages in thread
From: Javier Martinez Canillas @ 2023-05-17 8:38 UTC (permalink / raw)
To: Sui Jingfeng, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, suijingfeng,
liyi
Cc: loongson-kernel, linux-kernel, dri-devel
Hello Sui,
I would still add something to the commit description even when your
changes are trivial.
Sui Jingfeng <suijingfeng@loongson.cn> writes:
> Signed-off-by: Sui Jingfeng <suijingfeng@loongson.cn>
> ---
The fixes look good to me though
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
--
Best regards,
Javier Martinez Canillas
Core Platforms
Red Hat
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/drm_atomic.h: fixup typos
2023-05-17 8:38 ` Javier Martinez Canillas
@ 2023-05-17 8:48 ` Sui Jingfeng
2023-05-17 8:53 ` Sui Jingfeng
1 sibling, 0 replies; 4+ messages in thread
From: Sui Jingfeng @ 2023-05-17 8:48 UTC (permalink / raw)
To: Javier Martinez Canillas, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, liyi
Cc: loongson-kernel, linux-kernel, dri-devel
Hi, thanks a lot
On 2023/5/17 16:38, Javier Martinez Canillas wrote:
> Hello Sui,
>
> I would still add something to the commit description even when your
> changes are trivial.
Ok, I remember this, I will respin patch with this instruction in mind
in the future.
>
> Sui Jingfeng <suijingfeng@loongson.cn> writes:
>
>> Signed-off-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> ---
> The fixes look good to me though
>
> Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/drm_atomic.h: fixup typos
2023-05-17 8:38 ` Javier Martinez Canillas
2023-05-17 8:48 ` Sui Jingfeng
@ 2023-05-17 8:53 ` Sui Jingfeng
1 sibling, 0 replies; 4+ messages in thread
From: Sui Jingfeng @ 2023-05-17 8:53 UTC (permalink / raw)
To: Javier Martinez Canillas, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, liyi
Cc: loongson-kernel, linux-kernel, dri-devel
On 2023/5/17 16:38, Javier Martinez Canillas wrote:
> Hello Sui,
>
> I would still add something to the commit description even when your
> changes are trivial.
OK, that sounds fine. I'm agree.
> Sui Jingfeng <suijingfeng@loongson.cn> writes:
>
>> Signed-off-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> ---
> The fixes look good to me though
>
> Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-05-17 8:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-17 8:23 [PATCH] drm/drm_atomic.h: fixup typos Sui Jingfeng
2023-05-17 8:38 ` Javier Martinez Canillas
2023-05-17 8:48 ` Sui Jingfeng
2023-05-17 8:53 ` Sui Jingfeng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®