From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f182.google.com (mail-pg1-f182.google.com [209.85.215.182]) (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 8B7B01DC9B3 for ; Mon, 2 Feb 2026 05:41:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.182 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770010881; cv=none; b=eWZL6QpXfC6uUCHuv/bVX7twIZLsZTtXpSpuMptXYjteI/cL6/o5+VVc1V5UR/84QEXQ70oOIhuQuSz/T0xRfquRqapw/F/k4fqtoMrDKSaMjQ+cjc7TrHnBF8xKfXYeWMhm0Zw4kgplDoB/xCzDqt/2ANHp/TKc3qS3lsOjTVw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770010881; c=relaxed/simple; bh=374fjqtyEdgVhwWsr2HpCJEGgFgY/9sUJ7ytMr818tY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=r45Klwciz6uivUqjdhnI3McR8a0rusge1mWDuiOta/BpGoLxx+LZYz/mYr6uier2KxXkolkEVd048L+JHbshU4IpH+4kJFungOgDyLI7TdZF/ymVY8SvaQmOnIZOvoWcFptmPzGFTfh0xiQB4qWNHGnAqY/yWgHktNyBXPKP/Uw= 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=QTeLgWnu; arc=none smtp.client-ip=209.85.215.182 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="QTeLgWnu" Received: by mail-pg1-f182.google.com with SMTP id 41be03b00d2f7-c5513f598c0so1659051a12.0 for ; Sun, 01 Feb 2026 21:41:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1770010880; x=1770615680; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=w/OodEpFjOF1QKMpfh/He7lSljXTwFAUK5yvvx1mDI8=; b=QTeLgWnu2AGrIkeTPzfC/TsxCMZ38dbbNuzLzu4rCNAW4JDbwWdfuR+0mHIA6bsB7W gsR4CTO/Y3gERnLkCGCmSs0yZ1sJldiGDDT+Yp2ZVlKPlqE0NKGWSM4zuCJm6Mib2TJO hw7yV5Dr4BoDumArNC9n0oPM1OU37ZCWcrzm2e7NCZrSVWPm1XX+ev1vbNO+tfOWP7Rg 5qLqRg+jOOwlLtr6F2AEZw5DA53ZS3cmiuAiSeCGWd4XDI0fzNmO9zpXBTdXXfyB2Mq5 ieZZlz6bdScIv0GhgrxgoAiABGxSskz6yeYbn5HsU3F803/O5TiryllzYmWWN8+JLL+m G36Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770010880; x=1770615680; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=w/OodEpFjOF1QKMpfh/He7lSljXTwFAUK5yvvx1mDI8=; b=uWxTIC4j48GfrBxo9vhUMEUXazzSIxYjWvdZFICjJH/XV6Q1U+wbEFotjq0WSiJuka OrULyTTD595P6qiGi1tfbMzP00RftJyKFX7M132My7TH6WVwDMb8+luhNUxOqGPVWceq qrZBdptt63Th7itRiQJKkFYJqVZclThoMd6ay60JCzHMlDUIbmCzQmjWMoTI5l1A2xvJ ElrqWo6o2L0UvGIrguaM6AmxLpkR0k7jwOqlEVPWtC1twhMMDtyjDPmCVrngAM7dFDos jIjfMm9t0oXLXguxF4aGCuC7rqUIZMp14Z6UjkyzkHliBd/HqpUGT6qgIYrUDtal/nYp C9cQ== X-Forwarded-Encrypted: i=1; AJvYcCXvYRo0nN/mHanbyBKetjV1bkRiGQsLwDYwn1QxbU8KrXTtK22VJfUP+lIgqdo5HO+UEWD/VnArCKPxvDo=@vger.kernel.org X-Gm-Message-State: AOJu0Ywjzsq71mHAViOm9cEHCTpDTvkUImPnRZmxZ6nne0UKau1RUUX7 eVMDaN8Wj1ltLYG8Ze+moK3dt/xAvnyO1vElNXMoGYmoQ6l3Cec+Q1+V X-Gm-Gg: AZuq6aLLw59cME1GXuM/tUEtISu/SIajZgcGIl4GBMWCNWhF6g7lUW41HQCMoRXb34Y xBjuokxbHZ+hCPjvnT62D5rj/4x9LHnY8k9CjA/GFVs0oVWzpO8Rej2XoOf48qb9LxV69z1nPqD bGhv9U3mynzRVBvmvvtmyHt91ryYSCU54AUijyVkeqnSDghI0KeIkPPwhUxCCUPDj0/AFKguI0u GQi7r0X0nX2hTxdqRmSKi6Za9B6GQNvhobbo9m3jQ48P2faAUcSTKHOdTFn5i96xUAKNuKY87Y5 rM127kxgHZ9TfqFHDf0zDQVPMHj70av1l3jxQvKR68Ya/2lke/mrPsr292BydgZk4a1uQvtNo1T /g4FCLWwDkD8FFUkVW8aFXOFOzzItKIAAV1+cjFKPC3rD88Wanfo0bzJmcrv2MPdmI4qb24gY75 WCePRCR3qIygadB0gWGCQ5b0BNYok/8zH4WuB7OI6lpW9NpvGajG64oRTk5UVytFgrJ/wMvjh+U Gk= X-Received: by 2002:a05:6a20:12d6:b0:351:2c6e:6246 with SMTP id adf61e73a8af0-392e0148042mr8658013637.56.1770010879891; Sun, 01 Feb 2026 21:41:19 -0800 (PST) Received: from [192.168.0.100] (60-250-196-139.hinet-ip.hinet.net. [60.250.196.139]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c64276efb16sm13296475a12.4.2026.02.01.21.41.16 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 01 Feb 2026 21:41:19 -0800 (PST) Message-ID: <217f7407-ede2-4be8-bbd8-f2e499170e24@gmail.com> Date: Mon, 2 Feb 2026 13:41:13 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [net-next,v10,3/3] net: stmmac: dwmac-nuvoton: Add dwmac glue for Nuvoton MA35 family To: Jakub Kicinski Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, openbmc@lists.ozlabs.org, ychuang3@nuvoton.com, andrew+netdev@lunn.ch, linux-stm32@st-md-mailman.stormreply.com, yclu4@nuvoton.com, pabeni@redhat.com, davem@davemloft.net, andrew@lunn.ch, peppe.cavallaro@st.com, joabreu@synopsys.com, linux-kernel@vger.kernel.org, conor+dt@kernel.org, richardcochran@gmail.com, krzk+dt@kernel.org, edumazet@google.com, robh@kernel.org, schung@nuvoton.com, netdev@vger.kernel.org, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com References: <20260129054445.416242-4-a0987203069@gmail.com> <20260131015008.970729-1-kuba@kernel.org> Content-Language: en-US From: Joey Lu In-Reply-To: <20260131015008.970729-1-kuba@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 1/31/2026 9:50 AM, Jakub Kicinski wrote: > This is an AI-generated review of your patch. The human sending this > email has considered the AI review valid, or at least plausible. > > For local reproduction steps see https://netdev-ai.bots.linux.dev/ai-local.html > --- > net: stmmac: dwmac-nuvoton: Add dwmac glue for Nuvoton MA35 family > > Adds Gigabit Ethernet support for Nuvoton MA35 series SoCs using > the stmmac driver framework. > >> diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-nuvoton.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-nuvoton.c >> --- /dev/null >> +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-nuvoton.c > [ ... ] > >> +static struct nvt_priv_data * >> +nvt_gmac_setup(struct platform_device *pdev, struct plat_stmmacenet_data *plat) >> +{ >> + struct device *dev = &pdev->dev; >> + struct nvt_priv_data *bsp_priv; > [ ... ] > >> + bsp_priv->regmap = >> + syscon_regmap_lookup_by_phandle_args(dev->of_node, "nuvoton,sys", 1, &macid); >> + if (IS_ERR(bsp_priv->regmap)) { >> + dev_err_probe(dev, PTR_ERR(bsp_priv->regmap), "Failed to get sys register\n"); >> + return ERR_PTR(-ENODEV); > ^^^^^^ > > Should this propagate the actual error code from > syscon_regmap_lookup_by_phandle_args() instead of always returning > -ENODEV? If the syscon lookup returns -EPROBE_DEFER, this will lose > that error code and prevent the driver from properly deferring its > probe when dependencies aren't ready yet. > > Other dwmac drivers (dwmac-loongson1, dwmac-starfive, dwmac-sun8i) use > something like: > > return dev_err_probe(dev, PTR_ERR(regmap), "...\n"); > > or > > return PTR_ERR(regmap); > > to propagate the error. Thanks for pointing this out. I'll fix it in the next revision. Joey > >> + } > [ ... ]