From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B9B6F464F2; Thu, 3 Apr 2025 11:44:12 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A4E6F4060C; Thu, 3 Apr 2025 11:44:12 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mails.dpdk.org (Postfix) with ESMTP id 280E54060B for ; Thu, 3 Apr 2025 11:44:11 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 35747106F; Thu, 3 Apr 2025 02:44:13 -0700 (PDT) Received: from [10.1.35.62] (JR4XG4HTQC.cambridge.arm.com [10.1.35.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DB7B83F63F; Thu, 3 Apr 2025 02:44:09 -0700 (PDT) Message-ID: <9087e420-32c0-48e7-90dd-cf748e8b1f0f@arm.com> Date: Thu, 3 Apr 2025 10:44:08 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v12 07/10] dts: replace use of sanity check in comment To: Stephen Hemminger , dev@dpdk.org Cc: Paul Szczepanek , Patrick Robb References: <20240529233811.663211-1-stephen@networkplumber.org> <20250402232441.440346-1-stephen@networkplumber.org> <20250402232441.440346-8-stephen@networkplumber.org> Content-Language: en-GB From: Luca Vizzarro In-Reply-To: <20250402232441.440346-8-stephen@networkplumber.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Reviewed-by: Luca Vizzarro I'm happy with the change. The original intention of the comment was to make sure that non-involved ports were still being allowed, and that there were no side effects. The comment change somewhat becomes redundant as it's repeating the subsequent message. So if it were to stay as proposed, I guess it'd drop it altogether.