Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 18 Jul 2022 14:28:46 -0400
From:   Rodrigo Vivi <rodrigo.vivi@intel.com>
To:     Jason Wang <wangborong@cdjrlc.com>
CC:     <daniel@ffwll.ch>, <jani.nikula@linux.intel.com>,
        <joonas.lahtinen@linux.intel.com>,
        <tvrtko.ursulin@linux.intel.com>, <airlied@linux.ie>,
        <John.C.Harrison@intel.com>, <matthew.d.roper@intel.com>,
        <matthew.brost@intel.com>, <andi.shyti@linux.intel.com>,
        <michal.winiarski@intel.com>, <zhou1615@umn.edu>,
        <intel-gfx@lists.freedesktop.org>,
        <dri-devel@lists.freedesktop.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] drm/i915/selftests: Fix comment typo
Message-ID: <YtWmXu10XZLSGy+w@intel.com>
References: <20220715051953.31424-1-wangborong@cdjrlc.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20220715051953.31424-1-wangborong@cdjrlc.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?zyH289lsOMEZWlJU5mzN3dhcosXJ9fQYLD221A6DEQ75FvZGi2eQTAgtT3q6?=
 =?us-ascii?Q?YLCbqTdnVYGm3V1+eghCXNt+LRrnc1uPa4Huk/nSyNQD81IN+auvBfNjm/57?=
 =?us-ascii?Q?D6f1V6lGvRQrmw27QKVXAH81VE6XQWKvCz+345FG9KqWhtTVmyDvQ53u7bNH?=
 =?us-ascii?Q?pX2u17hn9C8ttY4bCfgfbrEqiTxl1tlWQ0QB1Krpbdeztvr/HtNTSuihQn2k?=
 =?us-ascii?Q?ocEPW+heX7eiPMXhUNjH1OuxIHHNgvx04ITC15+utTQZWJ0F6/WshxFc89Xi?=
 =?us-ascii?Q?iNLaKoVR/iqsLJ9r7rpJ7ctLzhiUFd33HdndZPMPMh+zgIt3uW9RtywJVUvg?=
 =?us-ascii?Q?L+C4H1vXwkE8DPZlv2y3W6+u0u8Zg90/qztUBq/xzYPggq0/Qlg0GtSaR342?=
 =?us-ascii?Q?+tCRle65u9AR5Ef6Ja9ri080p54Jd6wmfWgAqfNWySDhlzFwFbvv+VUh96xF?=
 =?us-ascii?Q?PRvG8oSx8d6AErdP/+JTdwocrax6vljyIL0G+bRH+YTUjA+SQsdlAUEa+soz?=
 =?us-ascii?Q?A9UwlL3gkL/yGTZydWCHYeh1jZ53cnH+YMmWVlQAKC4qIE1Cxp8OshIRXj6M?=
 =?us-ascii?Q?rOBIlsL5H5Ll6Lntb7xz/e9xFURVDbSu4WWWUlZdHqzsnyUxNAXWJfsgvTbx?=
 =?us-ascii?Q?2wYKaPkklAGkMCPXgYeTUPaxwvYXWyGIGQGX6FXRYRTJfCRj8Mlp2209TJE7?=
 =?us-ascii?Q?afzYZJ7V7b2Lv+NzQ38SIiNrmH4EfMxL1GWGgyEUEYlV+hdrJECQShyhevTw?=
 =?us-ascii?Q?bw4Gbr0wkrDbD5gXbC+66BfcBFHlY1Ld4VmxYQ+JuGbMT64xh9WOHnM5+rTG?=
 =?us-ascii?Q?u92DEleAKmg2qo+1rWZmeMsRbn86/JyFkV1rV05DtmjLC++hYO8th9REzM8J?=
 =?us-ascii?Q?jObO1cZl7N/VCibcuI3umkBUnBBs00gBiC0wl8eNQeoJUVD8hMK5pLv2pLSu?=
 =?us-ascii?Q?2QWOoe7nZAvVpcshEKfYOgJ632FovYC7Xz8hm/oTAQNBSqBfWx875EQwvGwn?=
 =?us-ascii?Q?i0Y3Yo9JE4mZzhbi4iQFKzfZ4k3LdfL06/mcpfeED5EE+jMN5oxYYSNuQMmm?=
 =?us-ascii?Q?AXZM1ezPC+Iy+ZKC2JXHVw6/roIyLmsR5LzzPW7jaEqKrBdb48Q7vt+4/gZ2?=
 =?us-ascii?Q?pl+M/g6GDlYe/jgNaVbMWC5vI6zh+cmZW+LGzcpo+jdArcldtCfFscA8iImV?=
 =?us-ascii?Q?uRQZGsTR2fcGOk+nqGGjRNlMB7rddGlmu7RGQxhy05pnDsgnA+3VxZCbpVYU?=
 =?us-ascii?Q?Tv2XNBj38L3/IysRHv3hA4Qg6m2X4qqE4yNOlqSltijQDljYZBw2ZBLwr0iX?=
 =?us-ascii?Q?wWBpYi6kqJjHs554hNg8w6+xvz9fsOWKb8SVsu+UPd1ZKPm/G5D7KQCdQije?=
 =?us-ascii?Q?2mfJ9yb1IOzd8kUL2J5NQc/AcffjumCOcFNCI5/pltMQi1DcT2JXAcjdgfzF?=
 =?us-ascii?Q?spmqVI5SaRfAtX/EGgmevvPcsVVBxfP52cNc+bocPG0J6SaetEQvoAPqpTv/?=
 =?us-ascii?Q?XkbQj3GfdiiBWtCzf9jl/od1UCNUPGT3gTlAY/VXeLTE7DMNQbJ2Qmq2w32R?=
 =?us-ascii?Q?CiotiXBnZC+OY1BoUGRQDZbhSnQBr0lGc5sFvdo9roTSigNHu1yUkMMfovXY?=
 =?us-ascii?Q?uA=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 7877208c-2113-4d61-11c5-08da68eb5dac
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6059.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jul 2022 18:28:52.6694
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: xts+Dz961AI0N+fXtjRLFXThKkSZ1j0eq5EAJOkJ3Gx55+60FDj2om/n6XUWKQ3790FKrVww1wTu8n2w1L8jpw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1599
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Jul 15, 2022 at 01:19:53PM +0800, Jason Wang wrote:
> The double `wait' is duplicated in line 974, remove one.

The line number changes... if you remove the "in line #"
the msg gets better.
With that changed:

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>


> 
> Signed-off-by: Jason Wang <wangborong@cdjrlc.com>
> ---
>  drivers/gpu/drm/i915/selftests/i915_request.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c
> index c56a0c2cd2f7..ec05f578a698 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_request.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_request.c
> @@ -971,7 +971,7 @@ static struct i915_vma *empty_batch(struct drm_i915_private *i915)
>  	if (err)
>  		goto err;
>  
> -	/* Force the wait wait now to avoid including it in the benchmark */
> +	/* Force the wait now to avoid including it in the benchmark */
>  	err = i915_vma_sync(vma);
>  	if (err)
>  		goto err_pin;
> -- 
> 2.35.1
>