lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 11 Jun 2021 09:35:04 +0800 From: Yang Yingliang <yangyingliang@...wei.com> To: Andrew Lunn <andrew@...n.ch> CC: <linux-kernel@...r.kernel.org>, <netdev@...r.kernel.org>, <davem@...emloft.net>, <kuba@...nel.org> Subject: Re: [PATCH net-next] net: mdio: mscc-miim: Use devm_platform_get_and_ioremap_resource() Hi, On 2021/6/11 0:01, Andrew Lunn wrote: >> - dev->regs = devm_ioremap_resource(&pdev->dev, res); >> + dev->regs = devm_platform_get_and_ioremap_resource(pdev, 0, &res); >> if (IS_ERR(dev->regs)) { > Here, only dev->regs is considered. > >> dev_err(&pdev->dev, "Unable to map MIIM registers\n"); >> return PTR_ERR(dev->regs); >> } > > >> + dev->phy_regs = devm_platform_get_and_ioremap_resource(pdev, 1, &res); >> + if (res && IS_ERR(dev->phy_regs)) { > Here you look at both res and dev->phy_regs. > > This seems inconsistent. Can devm_platform_get_and_ioremap_resource() > return success despite res being NULL? No, if res is NULL, devm_platform_get_and_ioremap_resource() returns failed. But, before this patch, if the internal phy res is NULL, it doesn't return error code, so I checked the res to make sure it doesn't change the origin code logic. Thanks, Yang > > Andrew > .
Powered by blists - more mailing lists