From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f51.google.com (mail-lf1-f51.google.com [209.85.167.51]) (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 C045732E73E for ; Thu, 18 Jun 2026 23:51:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781826688; cv=none; b=ScjFuaf7W6yeKUC96gX+sY2hj3Gx3g8ym/N5togPCbiUReZYUBlC69RVy3KFoRYoHms7w+YrgxOXJlJmyKpunZIwYWSjBB9T8GcgU8nxDaKiwWYFTP7Akr3yu9pVQdaWev9YAzTbCV4iBqUlBkO9wpAYu3jKE6K96r6KvkINM3c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781826688; c=relaxed/simple; bh=3A7AtmR5Fjx/ufI6DBaF4ORsth5x4xO8x5gV+HyqeX8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=HRWSMZkD7ljoUjzCVtQVZVNd45SnK1TpJQeVMuth7oLOv5Gx8DvRhzsE2FufHBkJx5jD2HlA0hzgczYqqoRqZ3aF/g2utMqTOhVgdWiMye79peb1bWdtrVY/jNsGjK3T997mF7m+avdKChOuTtdjfHpEhq/M8p8e+bMZuY/LIi8= 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=m/bjOfWw; arc=none smtp.client-ip=209.85.167.51 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="m/bjOfWw" Received: by mail-lf1-f51.google.com with SMTP id 2adb3069b0e04-5ad536cd0c6so1133361e87.2 for ; Thu, 18 Jun 2026 16:51:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781826685; x=1782431485; 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=8uZEinBiEzubyVwK+mDqbuOj5jjsF/asva00uL4IEmk=; b=m/bjOfWwXJ0oy3lGJP6pc0/K4a7XworB7sVG5CT2iwHdvaLT+Rdfxi+aROThlo7nNG yckdVBf+eNKXa5Aa+U0zMV7OtffprSoubo4Fp5GCKyLm6ZRxcJ4fnGl9xSsYPDoQXmrL DA0+SDuH+bmB41M9kI8UlQYrzifSZkyHYowmCkwE5/O/A4dkKsyPjQ99fBRKX72V7/Z0 hbW8OhtYiwlvYEQ/4h4kF+X705J8yYGMRZKLaA+OiwUHqVZ2wgnu/sX1T3NTlpglIS2S dJov4MVNF4Bjc0Mo2dhI0YD0GGdwOaRfl+myHUrpuyrDplQjTDfSEYntkFNlo+MHF0pQ s0Xw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781826685; x=1782431485; 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=8uZEinBiEzubyVwK+mDqbuOj5jjsF/asva00uL4IEmk=; b=ms31ENSDKLr1PTBWCoUTsGGWUUIKoM5CLD4obLgdEMdWI0m7WoBrXqt8i1gXGpFl2a PwExS1QuqOofOh4YK3KTGpAOi07SzjGjSAViM2ZpZ4LMMpn128seJ7rdjSg95kJrYiKn V9k9NJDSppA+DcbtsxJzUGFwMPPwxqTi4hkzodFMjb6P8VXjLGXDMZPz5l5/p7l4ssfD IZsrYptKazXrwlO3dIbb6MkUEt2i4nYTKgqQbNaFjv5k/IHH3JAxzAVB4UTqYydq2A0K fKLCiGWyRRYz//fH7fNi0lQ1nu4cgW49Zp0tzT8E2pfYXpBNIv6beuuNSZQDCVsOF1qJ CLlg== X-Forwarded-Encrypted: i=1; AFNElJ970K7Dl8bUkQhJ4r3fmYlC1QTWDm/A5ANB+eni0niRtUng2f8Uhiifh4oJrkukjFPf/KESsEqDZ6EN3ew=@vger.kernel.org X-Gm-Message-State: AOJu0Yy/j69l0xYeGwL+lk6R0Nr0HP0q4J1+ZqtwGRyS+qtjTeWNVwP0 vlDzVjEeqxI877uGDn/0aOTcDkD4DNblmwKcNN8qdsrTVPbHvHLD1E3j X-Gm-Gg: AfdE7cnLAappf6YCKoyi+qGBg6tKTcFjnH2RaKPO9OSW5QHpEDPPDKWfuOy0YzZqeH8 y20a/Sp0boo7wyQUKmch3fMIK0AiUtXcbdnqnDHazpbSMOdHNTLYqoh1hjv+HRjIGRgriZ5+Njm ayv/QRr8hVy2v7ZzmID7KizcOTxpjbqqe8q8p02/nSPZlLji9EpOkDSXbBDrp3/n6AoRKS0WAle 3ALWbR2sgjgd1um2agAjvB8DrdOiDMmMgyWNPPNVWedcSXIJ+Ca1Zkwv9z/hBk8eOrX0vdMAHTo +BNIUx47W5Qr631GpH8xCjnJLp39YVcxc7sy+SeYjKCR6OncLG+rxnNblKqhQN07EzgiB28jpz1 5ysnoU87RogAYl5QxYmj2mnUzEmjI23qGrnUbYBZWssNQNfEw5t/uWaV5dD/oGDSNw528RU8LFY sXX/pqY/AZrU4mFw== X-Received: by 2002:ac2:44d4:0:b0:5aa:8823:aeba with SMTP id 2adb3069b0e04-5ad57720e67mr181855e87.47.1781826684481; Thu, 18 Jun 2026 16:51:24 -0700 (PDT) Received: from localhost ([2001:678:a5c:1204:3d4f:47f9:ac7a:fb85]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5ad57245f5csm197309e87.28.2026.06.18.16.51.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 18 Jun 2026 16:51:24 -0700 (PDT) Date: Fri, 19 Jun 2026 01:51:23 +0200 From: Klara Modin To: Rosen Penev Cc: linux-wireless@vger.kernel.org, Felix Fietkau , Lorenzo Bianconi , Ryder Lee , Shayne Chen , Sean Wang , Matthias Brugger , AngeloGioacchino Del Regno , "open list:ARM/Mediatek SoC support" , "moderated list:ARM/Mediatek SoC support" , "moderated list:ARM/Mediatek SoC support" Subject: Re: [PATCH wireless-next] wifi: mt76: fix of_get_mac_address error handling Message-ID: References: <20260427051746.954704-1-rosenp@gmail.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: On 2026-06-18 16:01:45 -0700, Rosen Penev wrote: > On Thu, Jun 18, 2026 at 2:47 PM Klara Modin wrote: > > > > Hi, > > > > On 2026-04-26 22:17:46 -0700, Rosen Penev wrote: > > > Check return value instead of is_valid_ether_addr. The latter is handled > > > by the former. > > > > > > Signed-off-by: Rosen Penev > > > --- > > > drivers/net/wireless/mediatek/mt76/eeprom.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/net/wireless/mediatek/mt76/eeprom.c b/drivers/net/wireless/mediatek/mt76/eeprom.c > > > index 93d91264687f..0f6ccf6ed53d 100644 > > > --- a/drivers/net/wireless/mediatek/mt76/eeprom.c > > > +++ b/drivers/net/wireless/mediatek/mt76/eeprom.c > > > @@ -93,7 +93,7 @@ mt76_eeprom_override(struct mt76_phy *phy) > > > if (err == -EPROBE_DEFER) > > > return err; > > > > > > - if (!is_valid_ether_addr(phy->macaddr)) { > > > + if (err) { > > > eth_random_addr(phy->macaddr); > > > dev_info(dev->dev, > > > "Invalid MAC address, using random address %pM\n", > > > -- > > > 2.54.0 > > > > > > > Recently I have started to see randomized MAC-addresses on my x86 laptop > > with a MT7922 and the above message printed in the kernel log. I have > > CONFIG_OF turned on, but since this is an ACPI system the device is not > > described by any device tree and the earlier of_get_mac_address() likely > > fails with -ENODEV. Looking at the !CONFIG_OF stub for > > of_get_mac_address it always returns -ENODEV, meaning this will always > > randomize the mac in that case too. > IIRC, the normal device_get_mac_address supports nvmem now. Does that > fix your use case? I tried this: diff --git a/drivers/net/wireless/mediatek/mt76/eeprom.c b/drivers/net/wireless/mediatek/mt76/eeprom.c index b99d7452800f..243a8f2c7bda 100644 --- a/drivers/net/wireless/mediatek/mt76/eeprom.c +++ b/drivers/net/wireless/mediatek/mt76/eeprom.c @@ -174,10 +174,9 @@ int mt76_eeprom_override(struct mt76_phy *phy) { struct mt76_dev *dev = phy->dev; - struct device_node *np = dev->dev->of_node; int err; - err = of_get_mac_address(np, phy->macaddr); + err = device_get_mac_address(dev->dev, phy->macaddr); if (err == -EPROBE_DEFER) return err; but I still get a random MAC. > > > > Reverting this patch fixes the issue and the correct MAC address is > > used. I'm not sure if there is any case where of_get_mac_addres() could > > fail in a way that results in a valid MAC address but it seems unlikely > > to me. > > > > Regards, > > Klara Modin