Please igore this one, sorry for the troble.

Best Regards,

Guixin Liu

在 2025/3/27 11:20, Guixin Liu 写道:

When the Tegra186 GPIO controller is probed through ACPI matching, the driver emits two error messages during probing: "tegra186-gpio NVDA0508:00: invalid resource (null)" "tegra186-gpio NVDA0508:00: invalid resource (null)"

Fix this by getting resource first and then do the ioremap.

Fixes: 2606e7c9f5fc ("gpio: tegra186: Add ACPI support") Signed-off-by: Guixin Liu kanie@linux.alibaba.com

---

Changes from v1 to v2:

• Add "Fixes" to commit body.

  drivers/gpio/gpio-tegra186.c | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/drivers/gpio/gpio-tegra186.c b/drivers/gpio/gpio-tegra186.c index 6895b65c86af..d27bfac6c9f5 100644 --- a/drivers/gpio/gpio-tegra186.c +++ b/drivers/gpio/gpio-tegra186.c @@ -823,6 +823,7 @@ static int tegra186_gpio_probe(struct platform_device *pdev) struct gpio_irq_chip *irq; struct tegra_gpio *gpio; struct device_node *np;

• struct resource *res; char **names; int err;

@@ -842,19 +843,19 @@ static int tegra186_gpio_probe(struct platform_device *pdev) gpio->num_banks++; /* get register apertures */

• gpio->secure = devm_platform_ioremap_resource_byname(pdev, "security");
• if (IS_ERR(gpio->secure)) {

• gpio->secure = devm_platform_ioremap_resource(pdev, 0);
  

• if (IS_ERR(gpio->secure))
  

• 	return PTR_ERR(gpio->secure);
  

• }
• gpio->base = devm_platform_ioremap_resource_byname(pdev, "gpio");
• if (IS_ERR(gpio->base)) {

• gpio->base = devm_platform_ioremap_resource(pdev, 1);
  

• if (IS_ERR(gpio->base))
  

• 	return PTR_ERR(gpio->base);
  

• }

• res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "security");
• if (!res)

• res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
  

• gpio->secure = devm_ioremap_resource(&pdev->dev, res);
• if (IS_ERR(gpio->secure))

• return PTR_ERR(gpio->secure);
  

• res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "gpio");
• if (!res)

• res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
  

• gpio->base = devm_ioremap_resource(&pdev->dev, res);
• if (IS_ERR(gpio->base))

• return PTR_ERR(gpio->base);
  

err = platform_irq_count(pdev); if (err < 0)