=?gb18030?b?u9i4tKO6IFtQQVRDSCAyLzVdIGRybS9nbWE1MDA6?==?gb18030?b?IEZpeCBzcGVsbGluZyB0eXBvIGluIGNvbW1lbnQ=?=

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Best regards,
pengfuyuan


------------------ 原始邮件 ------------------
发件人: "Patrik Jakobsson" <patrik.r.jakobsson@xxxxxxxxx>;
发送时间: 2022年6月2日(星期四) 下午4:25
收件人: "1064094935"<1064094935@xxxxxx>;
抄送: "David Airlie"<airlied@xxxxxxxx>;"Daniel Vetter"<daniel@xxxxxxxx>;"dri-devel"<dri-devel@xxxxxxxxxxxxxxxxxxxxx>;"linux-kernel"<linux-kernel@xxxxxxxxxxxxxxx>;"pengfuyuan"<pengfuyuan@xxxxxxxxxx>;"k2ci"<kernel-bot@xxxxxxxxxx>;
主题: Re: [PATCH 2/5] drm/gma500: Fix spelling typo in comment

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

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

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux