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;
Message-ID: <9c311d86-303b-3673-a4e4-52f7d21947db@intel.com>
Date:   Wed, 13 Jul 2022 18:55:02 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.11.0
Subject: Re: [PATCH] acpi/acpica: fix repeated words in comments
Content-Language: en-US
To:     Jilin Yuan <yuanjilin@cdjrlc.com>, <robert.moore@intel.com>,
        <lenb@kernel.org>
CC:     <linux-acpi@vger.kernel.org>, <devel@acpica.org>,
        <linux-kernel@vger.kernel.org>
References: <20220710045218.55021-1-yuanjilin@cdjrlc.com>
From:   "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
Organization: Intel Technology Poland Sp. z o. o., KRS 101882, ul. Slowackiego
 173, 80-298 Gdansk
In-Reply-To: <20220710045218.55021-1-yuanjilin@cdjrlc.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZWVZeXMxa0hlQVpYdWZnb3FtSEVnSlRXbm5HU3ZXZXRtZUJMVEpaSlhodTBB?=
 =?utf-8?B?NTJTbkJRdHVBU1N6SUtLQXZFTm5aaml2T3NtempZcERuZHl5dkcyR3JhZmpp?=
 =?utf-8?B?REI4U0EwbFFlQ1dWNHY4Yi9zWmhCMXVKbzlOKzQxUzNqOE5pRk1NYkFwTU5E?=
 =?utf-8?B?ZE1YdlRYMW9xNXRrVUtMRE1zMUpGUWF6Ym1icVpNcEZ0S3lIOEZjVWtUSmd0?=
 =?utf-8?B?UXlHc1BmVkM0cVREN3FpWHhpOVRnTWVMdUJBZ3EzOWdTWFdUczV0NFIxUDFZ?=
 =?utf-8?B?SHREQkpzN2pvdHVrak5lZW9qU1RLVXBNUFZ1VnVqNmp5SjNVZkhlTG40aFFt?=
 =?utf-8?B?NlVFS2hCUERMTU96emZpR0FhcldYVU1GVjh3b3dQSlcxWXh6ZVZGUTdxc0w0?=
 =?utf-8?B?M0FPSjNJbC9kTnVFazhsTFhVWVVhS1g1elNLaDY4dHppaCs5Z1hmYU1mVnZp?=
 =?utf-8?B?U0t0dFU4MnN3VThnYTNyaFVCOG0vZjRGUENMY09meUFaWkUwS0U2N1dQMHcy?=
 =?utf-8?B?SnNKdTFSdEVqMnNQZzJSS2VwRWt1SDlXZ2NFNGlueEROSXhuUENTZE8vSGxS?=
 =?utf-8?B?OGdhMEhaRVBSNjRCbkVONVJaSkc1Q2w3bFpNQ00rUXZrRUhzZEMxK3BqTDl0?=
 =?utf-8?B?MklrZWV6MEFxZEVsMkZTem9SMGovelF5czhhTnNWczZrTGE2OU1wNm9ybHYx?=
 =?utf-8?B?Z0xTZ2E4RWpQUm1GMkFFQ2szWUZtdTBmRWxZZHk4TElVc0g0MTlMUW1zVnVJ?=
 =?utf-8?B?ZGk5MlF4VDExN1EwMXo1SlBVV3hCMERWbTNYeE02aUErWkJKUDBzaWNxNmlk?=
 =?utf-8?B?dDZKMzZnZlBoeWtNNnZ5NENNK2pEOWY0UWZvanVCTlc2QXpwaGNreGIyaU4v?=
 =?utf-8?B?S3VJbXN1WTZnNUhDUysreXNXUkJmLy8yZmEyL052VW1lc3gxNXd3UUtFcTha?=
 =?utf-8?B?ODJTR1o0Vk9ZRGJqYlYwL3YxOU9SMGVZVlNPa25FT3hncU5qbTMyditUU253?=
 =?utf-8?B?SXJyZTRMck54emlPNlhLNHlPWVBuMkdHdFpDTEVHOFNJVlpGdjB0NFRIaWRy?=
 =?utf-8?B?cElhWjFHM05oUHUzVW1UdmVGS0wrbytuMWxxN0RzZE1rTkN0M2FJaVdOTGtk?=
 =?utf-8?B?SjAyVXUrZFBPWVBQQXFMaUNNb3A0bFBCVlBldk5hWVZjUnJKM3FTME9rc3pQ?=
 =?utf-8?B?ditIbnJybnhtV1p3b0I3Z1d0OU1Eb2xodG5tcWVnbHU0dU0vdTF3NW9IS3A2?=
 =?utf-8?B?cFdpUStoZk52cEhaNjFyVWRsMzNsMzlDWmlRdVdxOEI2MGZQS1E3M2ZTRnhW?=
 =?utf-8?B?a2NNNEtuU3FsTEQxRUcxQ1hWc2k5eEFDa0QyK29MekZtdURKTnByZHIzZXJ6?=
 =?utf-8?B?K2Nja29NcFRWOWtwTUdoc2dWQWZDbHRISEVvRU1WZGpWSXovbVducUdjM1lZ?=
 =?utf-8?B?NlNNNlkvdWk4MmJpWm9QTkhLWG9EZXNyZFl6Q0dMZXIranNkNjR1bmhQS0Fo?=
 =?utf-8?B?VFlia1RTWDFsdmNVQnBlYStNWXcyRmlPOEVqVzM3TUdySnQxeFhveEVwTGkv?=
 =?utf-8?B?RGJCWGJlVU95RDUzSE1iMTM1dDhCNndCTWl0OU9jN0FqWE05UFAxNTlLbzcz?=
 =?utf-8?B?cjZoSzhzNkdleXhmMUp6amc1YVNMVStESFcvM2lCUHAvVWNlY0NLUTRHeUtL?=
 =?utf-8?B?d2JXYzQ5NmhyUlJ4UUlmN2xnT0dYY3FzNzYxWnFCN1hXSGZINDNuSTdGNGZ0?=
 =?utf-8?B?THdEUDhsTGlYdjZaazFIcHk4bzFxeWltdzl4ZmdEZmQ1WHVqZWNvSzcvd0ZI?=
 =?utf-8?B?NmNqV0E3eFVjeVJsNkE5TXUwVEJNS2hJbk5XYUpVbU55NGF5aVd4VjRKVVg3?=
 =?utf-8?B?bUpGb1FKdURwY0J6V0RnTXN0cnNvLzIrSjFZckVKdklvMGN6UEJaRFhVZXh4?=
 =?utf-8?B?ZnNuTjRPQmhJOW9BalZGTU1BWExQTm9rdXRSSUpSV3BMUGttOFVmU1RNUEpu?=
 =?utf-8?B?RGsyUWh2Nkg5Ymk3dlYwU1J5TTZPQllaS1dMQ0hmTHc1R05zUktvNEJkMk9l?=
 =?utf-8?B?ZGVWQzRCcnU0R3NuTE84aDdubkk4SUc2WHVNV2hXeXFOVVFobGxYdXhndEg4?=
 =?utf-8?B?NFhiODkxWHlFZFNZeXZDZ3MzbDdKelhaUHhRbkgyQllHNGRlVklTdGxWMVox?=
 =?utf-8?B?M2c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: b7c16615-03c1-41de-ff68-08da64f07171
X-MS-Exchange-CrossTenant-AuthSource: MW5PR11MB5810.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jul 2022 16:55:08.8008
 (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: ov6z/4SgeQgSx8eeutdg6kijfRFXWc9w4Z00sh4kFHppMfK1T8mFwfD5p4cgcbAYH2PlRjMGdHXseapu/V7f2+OuPFTXBtxNhkEO2GKFzgo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4517
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        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 7/10/2022 6:52 AM, Jilin Yuan wrote:
>   Delete the redundant word 'is'.
>   Delete the redundant word 'a'.
>
> Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com>

Please note that proposed ACPICA changes need to be submitted to the 
upstream ACPICA project on github.

If your change proposal is suitable for that code base, please submit a 
pull request for it in there.


> ---
>   drivers/acpi/acpica/exconvrt.c | 2 +-
>   drivers/acpi/acpica/nsprepkg.c | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/acpi/acpica/exconvrt.c b/drivers/acpi/acpica/exconvrt.c
> index 8de5d47ad485..7a66a70fa974 100644
> --- a/drivers/acpi/acpica/exconvrt.c
> +++ b/drivers/acpi/acpica/exconvrt.c
> @@ -446,7 +446,7 @@ acpi_ex_convert_to_string(union acpi_operand_object * obj_desc,
>   			 * Explicit conversion from the to_decimal_string ASL operator.
>   			 *
>   			 * From ACPI: "If the input is a buffer, it is converted to a
> -			 * a string of decimal values separated by commas."
> +			 * string of decimal values separated by commas."
>   			 */
>   			base = 10;
>   
> diff --git a/drivers/acpi/acpica/nsprepkg.c b/drivers/acpi/acpica/nsprepkg.c
> index 82932c9a774b..2afc5baa325c 100644
> --- a/drivers/acpi/acpica/nsprepkg.c
> +++ b/drivers/acpi/acpica/nsprepkg.c
> @@ -540,7 +540,7 @@ acpi_ns_check_package_list(struct acpi_evaluate_info *info,
>   
>   			/*
>   			 * Make sure package is large enough for the Count and is
> -			 * is as large as the minimum size
> +			 * as large as the minimum size
>   			 */
>   			expected_count = (u32)(*sub_elements)->integer.value;
>   			if (sub_package->package.count < expected_count) {