Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH] drm/gvt : fix error message typo
To:     Peng Hao <penghao122@sina.com.cn>, zhenyuw@linux.intel.com,
        jani.nikula@linux.intel.com, joonas.lahtinen@linux.intel.com,
        airlied@linux.ie
Cc:     intel-gvt-dev@lists.freedesktop.org, linux-kernel@vger.kernel.org,
        Peng Hao <peng.hao2@zte.com.cn>
References: <1538669074-26012-1-git-send-email-penghao122@sina.com.cn>
From:   Zhi Wang <zhi.a.wang@intel.com>
Message-ID: <659e1b3d-8b17-e380-4196-340596ec9740@intel.com>
Date:   Fri, 5 Oct 2018 04:13:15 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <1538669074-26012-1-git-send-email-penghao122@sina.com.cn>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi:

Thanks for the patches. :) Several places needs to be improved:

1. The prefix of the patch should be: "drm/i915/gvt:".
2. You can just cc the maintainers of gvt since the gvt is a sub-module 
of i915.
3. Better refine the tittle of the patch and also the commit message. An 
informative title and commit message would be more friendly to people 
who reading this later, even it's only a small typo. :)

Again. Thanks for the patch. :) I will fix that for you and commit it, 
so you don't need to send it again.

Thanks,
Zhi.

On 10/04/18 12:04, Peng Hao wrote:
> 
> From: Peng Hao <peng.hao2@zte.com.cn>
> 
> gvt_err message typo.
> 
> Signed-off-by: Peng Hao <peng.hao2@zte.com.cn>
> ---
>   drivers/gpu/drm/i915/gvt/gvt.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/gvt/gvt.c b/drivers/gpu/drm/i915/gvt/gvt.c
> index 46c8b72..31d6809 100644
> --- a/drivers/gpu/drm/i915/gvt/gvt.c
> +++ b/drivers/gpu/drm/i915/gvt/gvt.c
> @@ -438,7 +438,7 @@ int intel_gvt_init_device(struct drm_i915_private *dev_priv)
>   
>   	ret = intel_gvt_debugfs_init(gvt);
>   	if (ret)
> -		gvt_err("debugfs registeration failed, go on.\n");
> +		gvt_err("debugfs registration failed, go on.\n");
>   
>   	gvt_dbg_core("gvt device initialization is done\n");
>   	dev_priv->gvt = gvt;
>