mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [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®