Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752429AbdFTUqT (ORCPT ); Tue, 20 Jun 2017 16:46:19 -0400 Received: from mail-wr0-f196.google.com ([209.85.128.196]:35966 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751036AbdFTUqR (ORCPT ); Tue, 20 Jun 2017 16:46:17 -0400 From: Pali =?utf-8?q?Roh=C3=A1r?= To: Andy Shevchenko Subject: Re: [PATCH] acer-wmi: Using zero as the first WMI instance number Date: Tue, 20 Jun 2017 22:46:12 +0200 User-Agent: KMail/1.13.7 (Linux/3.13.0-117-generic; KDE/4.14.2; x86_64; ; ) Cc: "Lee, Chun-Yi" , Darren Hart , Platform Driver , "linux-kernel@vger.kernel.org" , "Lee, Chun-Yi" , Andy Shevchenko References: <20170620090623.18364-1-jlee@suse.com> <20170620164851.GA32699@pali> In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart3625433.iIrMdDToXn"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <201706202246.12941@pali> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1669 Lines: 53 --nextPart3625433.iIrMdDToXn Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On Tuesday 20 June 2017 19:22:46 Andy Shevchenko wrote: > On Tue, Jun 20, 2017 at 7:48 PM, Pali Roh=C3=A1r > wrote: > > On Tuesday 20 June 2017 17:06:23 Lee, Chun-Yi wrote: > >> Pali Roh=C3=A1r found that there have some wmi query/evaluation > >> code that they used 'one' as the first WMI instance number. > >> But the number is indexed from zero that it's must less than > >> the instance_count in _WDG. > >>=20 > >> This patch changes those instance number from one to zero. > >>=20 > >> Cc: Darren Hart > >> Cc: Andy Shevchenko > >> Cc: Pali Roh=C3=A1r > >> Signed-off-by: "Lee, Chun-Yi" > >=20 > > Looks good, Reviewed-by: Pali Roh=C3=A1r >=20 > Unfortunately patchwork ignores this tag. > So, in the future please: > - put a tag on a separate line > - do _not_ prepend it by any characters including white spaces > (except new line :-) ) I have not known that those lines are parsed automatically. Will do it in future! > No need to resend this one. >=20 > Pushed to testing, thanks! =2D-=20 Pali Roh=C3=A1r pali.rohar@gmail.com --nextPart3625433.iIrMdDToXn Content-Type: application/pgp-signature; name=signature.asc Content-Description: This is a digitally signed message part. -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEABECAAYFAllJiZQACgkQi/DJPQPkQ1I/cwCdG8Ukpx+pQZz+2aIz82ZMgX6i WdEAn2af+CX5sfgcfcDL5bFOxwszf5aq =bTv3 -----END PGP SIGNATURE----- --nextPart3625433.iIrMdDToXn--