From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 89FE03CEB8A for ; Wed, 6 May 2026 22:49:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778107782; cv=none; b=UNjs50zm/z+gd126vWySmAJE5GjSABiTHJE37Kkbv/yI/CkrpTjXM6KBTmtA4Jr2QQKw8CbJbhYbNZ1u1COi5SsFmEVDfTyqwcnURsFedKHM9BIDbZfUcJMOu8ogvQpIFqMI0EF+hhqxdCY1WVsn2l1YWhUv2B5zFoP2WUXWWYM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778107782; c=relaxed/simple; bh=W2bOwmgBgbRPOFoHwIWTGbgslfjmN0yKNnT3yBKu9PY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=luoFeJKAZXJXE6r6YYMmIN49KTjlFLd8ioO/g7P7aRPgB8PSp+bwY5n09rCVXm/FqLWUwF0bBKr0YRpMQ1+jSJpllE2DvzSHNyA8F/dqMQpmkTQ6W5WY5Wbp3/oszi2k9GlC+EZKsF/1QNnwIqfg256tdBlsiTVAl2tYxfhe+zM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=purestorage.com; spf=fail smtp.mailfrom=purestorage.com; dkim=pass (2048-bit key) header.d=purestorage.com header.i=@purestorage.com header.b=LPByxj0j; arc=none smtp.client-ip=209.85.214.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=purestorage.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=purestorage.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=purestorage.com header.i=@purestorage.com header.b="LPByxj0j" Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-2aaed195901so1075465ad.0 for ; Wed, 06 May 2026 15:49:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=purestorage.com; s=google2022; t=1778107781; x=1778712581; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=wB9Jbl6OwNsn+HVcduUaP/1MdPcWg7cvGBqOaohDZOA=; b=LPByxj0jry93gsoWzoEAYFKLLpeHZrQgXuvjVOXb5uqOEHTsRW1dput8oG/77Dmy2t xWAkcXsoiEcDv8z3ybnEWP4tnp/v3KJEh203mjWslrxq/dsnvVpGU9Id4kNWQOMHc8ij WeChbpFENQ4tscumo8QXimNOA/m1IBIzkae+lJM0zAsst0ORIi9iLK84+Ll+u2CJn4YK lc382p/iTJAT7sjV/ikjcAAMHcQDXJL1oQjg9kIoFliLty1Q97/dzJvFgfccfAhr5o8J KmY8Hms2rm1bu+d2Po3P0MG8WOuwM7OdFidmHFdCGMMf8i/EjSwHDhfzUSUhieyVqO2I g1hw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778107781; x=1778712581; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=wB9Jbl6OwNsn+HVcduUaP/1MdPcWg7cvGBqOaohDZOA=; b=npqjCF0j21ZwmGBs4ZJ066A85782V5pWLZuDII5/LARzA3vlZ75yg3Zh1dl1qREv2a zOotMN7ssmE+4GYColxn/fegee6zDC5it2yAXsZFoHGD5/Lu9kZHlF9UHUIGAY5V/dQD 5bBz+b7KN7ZxOlUjXyaxaXXAMSh7N2lgzLSH8WUkOtBZk6mkXWTKkZ5ApAZQo/r6nKgR HvHPHc+5F6dcHNT04BO73YwhsYVTeAxqltrL5dA97imSPS4ypfLSX0N2979oRB+QMJEt Peym+v5MuG2yZFCsoiUYiViWdxxipSKiC7l5o19I2aXTcoEwTSePP0x32lPx2q4O+x4r DsWA== X-Forwarded-Encrypted: i=1; AFNElJ8+icKMocDUCqr+A/7kJ2o/iMhHqZM2mKCX/ODhLAGRgd1bYy23htCGkkK5fHjtSEqB/vhUy1YB/qU9gjc=@vger.kernel.org X-Gm-Message-State: AOJu0YxV5E7AWMwBufA0+HsCJyC8l3KMopzXL18pVlwC5T8/jmG0yFJh 2BhdVzlcF+Vf5ynlY32hpmpQNCCZt8KRASI2E4M/GyXBYgQUuECTzXB2Zcg94PY+6O0= X-Gm-Gg: AeBDievqcjpWuUwqK0lflsa03JU3X6yhkvDL3X6hNwafN1qpmz7suwAJCBlqu96AKbW JA2mE+hb/I8RbY1Rfx0w/xe3W9gNRpBQ6s+MA63UPw6GqDVjWL1sq2EtqzPEdBT6GKTJwDdHqus W617oxij73uZmjzCWF/1yN0VhVAMGKk0gVMo3GlExc8OnI8ds2HB5PzzABPjJ8p3I1oNixVMoJq 4iqtLCLores01eY77/PZ2Go93Rw1WlcDnJR5DWLX9buV+7lJAKeCKpjz6a/E86k9HMVvm4YLA9T 8d4PHUID3gXbpQXym/QhJWr6JIhXsAqwHX0VallSshnxc7IA71Y5XtwsQLhQ7hiQKoVnOUeaF6F CNTxGQ0WK2ubmKzlBLazNjNExK4ldavqLlf6g0+4anONlDhJxDOGGVpwtF/zXTKJBaVphSVujUk XVTrX3uS3IEsafawLkhUT6lzMW2pOob2JrkoGWf0oijIZ2rxX4PObx4hr36nk7eCPJHw== X-Received: by 2002:a17:903:187:b0:2b2:be01:5532 with SMTP id d9443c01a7336-2ba799d4662mr61394475ad.35.1778107780763; Wed, 06 May 2026 15:49:40 -0700 (PDT) Received: from medusa.lab.kspace.sh ([208.88.152.253]) by smtp.googlemail.com with ESMTPSA id d9443c01a7336-2babaaed237sm2689955ad.27.2026.05.06.15.49.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 May 2026 15:49:40 -0700 (PDT) Date: Wed, 6 May 2026 15:49:35 -0700 From: Mohamed Khalfella To: Jacob Keller Cc: Paul Menzel , Tony Nguyen , Przemek Kitszel , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [Intel-wired-lan] [PATCH] i40e: Fix i40e_debug() to use struct i40e_hw argument Message-ID: <20260506224935.GD10532-mkhalfella@purestorage.com> References: <20260428181450.2622899-1-mkhalfella@purestorage.com> <558971b7-caea-471a-8fe2-73ba6cc0790a@molgen.mpg.de> <20260429165229.GF2686-mkhalfella@purestorage.com> <10637b35-766c-454b-b9fc-ac9e6df36da5@intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <10637b35-766c-454b-b9fc-ac9e6df36da5@intel.com> On Wed 2026-05-06 14:57:26 -0700, Jacob Keller wrote: > On 4/29/2026 9:52 AM, Mohamed Khalfella wrote: > > On Wed 2026-04-29 13:02:00 +0200, Paul Menzel wrote: > >> Dear Mohamed, > >> > >> > >> Thank you for your patch. > >> > >> Am 28.04.26 um 20:14 schrieb Mohamed Khalfella: > >>> i40e_debug() macro takes struct i40e_hw *h as first argument. But the > >>> macro body uses hw instead of h. This has been working so far because hw > >>> happen to be the name of the variable in the context where the marco is > >> > >> marco → ma*cr*o > > > > Good catch. Also 'happen' should be 'happens' > > > >> > >>> expanded. Fix the macro to use the passed argument. > >> > >> I’d add a Fixes: tag, but the maintainers might have more input. > > > > Yes, I should have added Fixes: tag. I will leave it to the maintainer > > to decide if v2 is needed to fix the spelling mistakes and add Fixes > > tag. > > > > Fixes: 5dfd37c37a44 ("i40e: Split i40e_osdep.h") > > > Please send a v2 with the fixes tag and typo. It will make it easier to > avoid losing this data. Done https://lore.kernel.org/all/20260506224123.691160-1-mkhalfella@purestorage.com/