From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) (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 3DFF53290A6 for ; Tue, 8 Sep 2026 05:25:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788845154; cv=none; b=FqlkDrE2tBNJ21vt4xdawN2Ugpt0FZI1zKW5SUmu8/tBpgUmnnPtkuHIcVf+HCqSK2vcC3iZWSmZ0a3D+iOXTM5RFKMfj7OdycuMKznuKjeyTre0Qidrdb5EN5luQ+nKF89hv3nzNZfdLvzcdNbFiONqqkiLlcRaGb4TLAZYB/Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788845154; c=relaxed/simple; bh=x8m6O+VGBf2mh+d9UTHXImpGsH4/EOQxuIFX8xSzjGA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=u4p1pdeNzj3I/foDJJStLSRtFfVwRLVdcuO+ZIXkwaKp28GCPDz8oHPaY+4enPEGXGZfEZ0ExooMUo3ke/ho24DCTHNGWJMuLN06t4ny3kbmMEx9tR7baYXu65KqMjNIq9Z6eQjhQxYZL0LDXo3TnSa6g7O4u93qCMTdFLW9AYM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=nz5k+qoD; arc=none smtp.client-ip=209.85.214.176 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="nz5k+qoD" Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-2d53197d8b5so36949235ad.3 for ; Mon, 07 Sep 2026 22:25:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788845152; x=1789449952; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=cAp8mTkH9nPCXUPKfddbuA6JymE/Gu0j627y3YqdlVI=; b=nz5k+qoDxmhy+uL4z07k6tY8ULwpn0SIVgQGuoqotUPz7lNawW65mqwBEEY5RTwky9 x38XVYDHA6B/k6ElAZEGnyiVsRV9y5KTiQ1QqiNVUVVDpnZj1vrBQ+C5FVv5a9kwOS63 Z4uXouxjPeOeYZq1+pIhFkjmhRPa2mRgju92tFU0uJs7LfC4kJQTR9k76w3SM0W0Bojm Ta2VcSKObXzeOXsf1SkondzhEWfiUr5Rm+EeJRYtY52eA8SiXJgck6uYWVTQumhrmAS4 9V7nbWV3ZkTvYDLN3R5KsIcd31vpdZJSe7QowHbmj948xmuxQ6M7oFwREPtzor/bSQdy l6YQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788845152; x=1789449952; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=cAp8mTkH9nPCXUPKfddbuA6JymE/Gu0j627y3YqdlVI=; b=rsZ1kZ9Xqj5LKdYH+z2HNLY291qlvsTKs6VUXwqyTW1+jxPONVWxT6NjU5bL6/Z8xD Q5XFhlGP2AaetSb1Adbl2aUsjLxOSOPfCgPgohVIStz9Q/nUerzgw3l8MONrUF/ggFr+ DrC0sPAGB8oe5OW2Azncj44vbCB+rf7dpLQ/26h4gdrbYaTSshqGGTsCRh9vaL4ocPjZ pyE/78mpjoACtVyWvIqccbUuYFuAU7etcvpyP3o5IRbTzYCVuHcy7gKghmnb3ZOxGiWx uqRvZRlXZ5gtJxQzdMhE4QRUb1T15kvMlP/pqUwuf4JglKe+MvkWV+hOOqYPtPqSkRLO MlOA== X-Forwarded-Encrypted: i=1; AKwUvByyOIHIFpLElmJTFHStJTX3gSbaOdTIZksRxHFLTvK92WWM6OT/GH6crPYsDCcL1vuqFjyrscq0oZ6z1WI=@vger.kernel.org X-Gm-Message-State: AFuF++muVxEj+RRYG9aGFdHSrwKqZDLBFKrAxeZDX7mCFUzUEkbM0bMx G4NOLiClBm7PjOezer85KYKz4XvF61ppuhF2FTUPN3bqpXPokRadqQYB X-Gm-Gg: AYBFou092Kb6mB4U+ATLtMCGQaezH+s528doxtPfNbq/OarR8ufTn4LCFRY1SA38hdn vf1+uOfOJWYGfLrqWG33BIp4wTYD5BdWpymuxSdcDoCMhdI6E8T6QhZ+cNA/Fs5Sm2Thnh6Dpw2 UvbQvQnPPm+lrjR4CnGVpJWUYkznwpg8uT6I9/+u7Vo3/Nthd5tMZEt39enGE0axFH2k0eUKVGS TgAOJ45WcVh5EAdQeEg3LC/0B5/kK1X67um9ADtyIiC7NU0GooLRLf8wxKrJyme0bEzoQeMZ7oB H+NNEUUDyY/giCMUmM/cGQccvB6urFCjqFzEPHVuNKka9U+gv+YQHUc/f/972i6ZEzDl+EyQgrx 0T1aMlpPVPvOnyaeHT02suJTAfIeZ7UEQCGagPYH51FSXXg7kldjjiirYc2jTaMgTnTxkOmjPgz pnJR+aOT7Shm4FrgjDLZYg8IA4Qtk5NL1Qf9eU2Dmusu0lBrEgkqrMxSkdvYioxmzAcdx2Ig4fp I4UcImFk8kbObBexaWvyLM+ X-Received: by 2002:a17:90b:1b44:b0:398:ba96:1afd with SMTP id 98e67ed59e1d1-39b2613249fmr36702489a91.8.1788845152391; Mon, 07 Sep 2026 22:25:52 -0700 (PDT) Received: from volcano9f6e-hostos.amd.com ([165.204.217.251]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-14324410092sm30022861c88.14.2026.09.07.22.25.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Sep 2026 22:25:51 -0700 (PDT) From: Hemanth Selam To: Andrew Lunn Cc: "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Sudarsana Kalluru , Manish Chopra , Andrew Lunn , Simon Horman , Edward Cree , Shuah Khan , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-net-drivers@amd.com, linux-kselftest@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH v2 6/6] net: usb: fix repeated words in log messages Date: Tue, 8 Sep 2026 10:55:44 +0530 Message-ID: <20260908052544.27165-1-hemanth.selam@gmail.com> X-Mailer: git-send-email 2.48.1 In-Reply-To: <7c8cb3f3-8eab-494b-9f97-b30693a361eb@lunn.ch> References: <7c8cb3f3-8eab-494b-9f97-b30693a361eb@lunn.ch> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On Mon, Sep 07, 2026 at 02:49:32PM +0200, Andrew Lunn wrote: > The same thing twice. Same capitalisation. Is one could actually be > correct. Please see if you can find the data sheet for this device and > see if it gives any clues. I could not get hold of the ASIX data sheet itself, so this is from the register definitions in the driver rather than from the vendor document. The register is named in asix.h: /* AX88772 & AX88178 Medium Mode Register */ and it is written by AX_CMD_WRITE_MEDIUM_MODE (0x1b). So the register is the "Medium Mode" register, and neither word is a stray duplicate of the other: the message is naming the register and then repeating "mode" for it. The sibling read function words it the way I would expect: netdev_err(dev->net, "Error reading Medium Status register: %02x\n", ret); which names the register and stops there. The other thing that pushed me towards it being redundant is what the value actually is: int asix_write_medium_mode(struct usbnet *dev, u16 mode, int in_pm) ... netdev_err(dev->net, "Failed to write Medium Mode mode to 0x%04x: %02x\n", mode, ret); The 0x%04x is 'mode', the value being written, not an address. So the sentence is "failed to write the Medium Mode to ", and the second "mode" has nothing left to refer to. The same string is in u-boot's copy of the driver, but that was copied from Linux, so it is not independent confirmation. If you would rather not take my reading of it without the data sheet, I am happy to just drop this one. Otherwise I will resend it as-is in a later batch of the split series; it is not in the batch I am sending now. Thanks, Hemanth