Hi,
This is great advice, thanks. I will resubmit this patch again.

Best regards,
pengfuyuan





------------------ ???????? ------------------
??????:                                                                         
                                               "Patrik Jakobsson"               
                                                                     
<[email protected]&gt;;
????????:&nbsp;2022??6??2??(??????) ????4:25
??????:&nbsp;"1064094935"<[email protected]&gt;;
????:&nbsp;"David Airlie"<[email protected]&gt;;"Daniel 
Vetter"<[email protected]&gt;;"dri-devel"<[email protected]&gt;;"linux-kernel"<[email protected]&gt;;"pengfuyuan"<[email protected]&gt;;"k2ci"<[email protected]&gt;;
????:&nbsp;Re: [PATCH 2/5] drm/gma500: Fix spelling typo in comment



On Fri, May 27, 2022 at 5:42 AM <[email protected]&gt; wrote:
&gt;
&gt; From: pengfuyuan <[email protected]&gt;
&gt;
&gt; Fix spelling typo in comment.
&gt;
&gt; Reported-by: k2ci <[email protected]&gt;
&gt; Signed-off-by: pengfuyuan <[email protected]&gt;
&gt; ---
&gt;&nbsp; drivers/gpu/drm/gma500/psb_intel_sdvo_regs.h | 2 +-
&gt;&nbsp; 1 file changed, 1 insertion(+), 1 deletion(-)
&gt;
&gt; diff --git a/drivers/gpu/drm/gma500/psb_intel_sdvo_regs.h 
b/drivers/gpu/drm/gma500/psb_intel_sdvo_regs.h
&gt; index 600e79744d68..7efbd7c45c90 100644
&gt; --- a/drivers/gpu/drm/gma500/psb_intel_sdvo_regs.h
&gt; +++ b/drivers/gpu/drm/gma500/psb_intel_sdvo_regs.h
&gt; @@ -371,7 +371,7 @@ struct psb_intel_sdvo_tv_format {
&gt;
&gt;&nbsp; #define 
SDVO_CMD_SET_TV_FORMAT&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
 0x29
&gt;
&gt; -/** Returns the resolutiosn that can be used with the given TV format */
&gt; +/** Returns the resolution that can be used with the given TV format */

Hi,
Can you also fix the /** at the start of the comment. One patch for
both changes is fine.

-Patrik

&gt;&nbsp; #define 
SDVO_CMD_GET_SDTV_RESOLUTION_SUPPORT&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
 0x83
&gt;&nbsp; struct psb_intel_sdvo_sdtv_resolution_request {
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; unsigned int ntsc_m:1;
&gt; --
&gt; 2.25.1
&gt;
&gt;
&gt; No virus found
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
 Checked by Hillstone Network AntiVirus

Reply via email to