From nobody Sat Jul 25 03:20:16 2026 Received: from mail-pf1-f176.google.com (mail-pf1-f176.google.com [209.85.210.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 1F3EB29405 for ; Mon, 20 Jul 2026 00:30:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.176 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784507448; cv=none; b=eLIj6EO/9keSDVtX0//OCBjZ6Qw8hmc4gMvvsQuy1CW4gHA89ETyNGYyIzjEHFY1lyLF/zMoCkCKOXTA6vH4vjtQPXwv6AhM2VZ7MAUVBI8kiPbOrX5G4r/ZgQbNoRXvt+3Q2HK/VyQ7aUyXpFKElZti+rW2ff24cSHqcnDU41I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784507448; c=relaxed/simple; bh=+Eo5NWVkrzbtPKFw6Lp9lya3M8MKBRSoua+JLpJoOs8=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=rJVFdYUizKsMhyPxexY/OWCfIXKvh/4Z+qzDeeSFnbdZlWwnzVvlXXhg2wOQP1WJLpLPUz3tVitUZHTPtZrpKZyUmRXGLySyvzuNMB4Bt2OVt7PkcT0ldCFaYd5UegSy9cZOJWt8p6SFUz+PqLBYbLvY8Icqjs8juXLaokRoW0U= 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=bNjyAg3B; arc=none smtp.client-ip=209.85.210.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="bNjyAg3B" Received: by mail-pf1-f176.google.com with SMTP id d2e1a72fcca58-848d21bbaffso9406171b3a.0 for ; Sun, 19 Jul 2026 17:30:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784507446; x=1785112246; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=hbxsCH9pMgD6SOGxQcpGqXiSoFQuY8yErZj8atWFslk=; b=bNjyAg3BlOqKRruOZPq0UlQoNvWnG8LCwKcLof1mFNKVUBRmlolvAiM8FY8ZsQY4mj hXStf3MBG4kTFz2aAEffpGWAlTfeDpTUUTdzDnIJTO6RtEQkYVU1PBGWpahCMmyAAEXU jMF1jihN5o+jf6hkO7jQdU4WEkBuyslRFgkj0hevYaesl0/qK7feopFmumF69RFyxDUx vzkAh4yUlbrpiHreP2sn+lH0vordv0QGz/4oyX2CPuqbNWpmNNLkve29koNVOT/JDv2q RgQzqtInpLwg297Oisrnth54W5FuRVXTYOAsqaMpBZ+J1NH1wMRQbdKSwY/6hFNXoJTB z8ew== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784507446; x=1785112246; h=content-transfer-encoding:mime-version: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=hbxsCH9pMgD6SOGxQcpGqXiSoFQuY8yErZj8atWFslk=; b=D5cKKBrFr0Oa9fQUgPBrqJy3PrwHnQxIZ2BnxyCSqVUISTmq7DZ2+GSbopayQxEd6w qdTZ6fr4QwG4+9FvBntkG9AY2Q3SslpQHyRw2bEvmpx1DXeQgPmwAfpZl/yAme1/Asux wHdC24qQT2isBAL5gpCHgmahsfVlYntF3qMtJN9hurxVSvvooBhpWMEWKA/Q4i6yFCsm oyDWSJKJEKB+DzCY49NlIgUY16eon9HXUsv+iL+DOQ5SArELR7APY9MBtdKWoLYt1mCK tm3RbFYKGb3HL6Qk+atIcA6nC/u/lyA+ZiB1pvfAK8oCRetCi+VfSSnDiwVrkntR7lx6 92cA== X-Forwarded-Encrypted: i=1; AHgh+RqIW9bsjTV47x3tfJIaji3YqufePmHUGPn2DZAzwJ3p+641M2LOQHEvjsbuYPxJ+Y+9Ae/0d/irftyBVnI=@vger.kernel.org X-Gm-Message-State: AOJu0YyEfsur1tFKUKBtsaEqXdxmqp9eV8TeDSB7PsPRgzUsxe/7Z9fv kWQto0iLuh0TYEw8nMECJpmmCL94bDGDYq0DZvJL2gdQ0pDTGUCxLzOj X-Gm-Gg: AfdE7cmSs65i9NsaMyAnJbxH5dqW2QI8S8fi786regyCboubcrcc7yCvdLEhDu4FBmU dWu2ohTicm9aTTbktxEZA8ftt3EIxRRb+be0vWgYvuksN2EE3m2Q3nVchea2ZN8LiILhdcV+DEe bK8WxMpz1ZiGB0IUVq+VDvmBEGA4pFPrI6hUkwt4mlVcFlbz0gAkOUuwDTCzgGJNJ3rIQ6rTr2j 9zg+BnLY5ZVO7lO2CWOpWsaewjznMyDd0J+Hr+qX1zsiFBsCmdAVZcgv1VWJ/Ci72gkE0vcEPJQ WmhKPukgdkCCdEtZYHye0rREX1jMLyYJ+PMlcIpywWzwH9cZC/3iDwuRhkhfU/fOlK5ZrJrpjPF P4sFUltWZ1FFAn4Nh6GMfLBIUxKqWC7PzwoE2yr5NcS05c9fESY58faJdSYexMO2qUyDgOm4RVJ GdRahrtaRWFDhE7nUmWuodmYG/ns6XzSix4LS9m/S/tLuSWoYcQAWUzpoUrqv+3YwPfwMWRAnKE j3w1u0KavhPCpO73SiY8Cdnraz3AGur2e/JGkLGwudcKfF+UxU4IT0= X-Received: by 2002:a05:6a00:23c8:b0:848:587f:8ac7 with SMTP id d2e1a72fcca58-84c294bbaf0mr12410088b3a.52.1784507446016; Sun, 19 Jul 2026 17:30:46 -0700 (PDT) Received: from ryzen.lan ([2601:644:8000:7a86::e35]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84c2af6e9a3sm4639048b3a.41.2026.07.19.17.30.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 19 Jul 2026 17:30:45 -0700 (PDT) From: Rosen Penev To: linux-spi@vger.kernel.org Cc: Mark Brown , linux-kernel@vger.kernel.org (open list) Subject: [PATCHv2] spi: ppc4xx: devm-ify probe and drop manual resource management Date: Sun, 19 Jul 2026 17:30:43 -0700 Message-ID: <20260720003043.1536358-1-rosenp@gmail.com> X-Mailer: git-send-email 2.55.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Replace open-coded resource handling with devm helpers: - spi_alloc_host -> devm_spi_alloc_host - of_address_to_resource + ioremap + request_mem_region -> devm_platform_ioremap_resource - request_irq -> devm_request_irq - remove now-unused mapbase/mapsize fields from struct ppc4xx_spi - move of_node_put(opbnp) earlier to simplify error paths - delete the entire error-unwinding goto chain Move devm_platform_ioremap_resource() and platform_get_irq() up in order to avoid doing work if -EPROBE_DEFER is returned. Assisted-by: opencode:big-pickle Signed-off-by: Rosen Penev --- v2: handle potential -EPROBE_DEFER first. drivers/spi/spi-ppc4xx.c | 97 +++++++++------------------------------- 1 file changed, 22 insertions(+), 75 deletions(-) diff --git a/drivers/spi/spi-ppc4xx.c b/drivers/spi/spi-ppc4xx.c index 46ac58dfb3fc..95c034cf2336 100644 --- a/drivers/spi/spi-ppc4xx.c +++ b/drivers/spi/spi-ppc4xx.c @@ -112,8 +112,6 @@ struct ppc4xx_spi { struct spi_bitbang bitbang; struct completion done; =20 - u64 mapbase; - u64 mapsize; int irqnum; /* need this to set the SPI clock */ unsigned int opb_freq; @@ -337,14 +335,24 @@ static int spi_ppc4xx_of_probe(struct platform_device= *op) struct ppc4xx_spi *hw; struct spi_controller *host; struct spi_bitbang *bbp; - struct resource resource; struct device_node *np =3D op->dev.of_node; struct device *dev =3D &op->dev; struct device_node *opbnp; int ret; const unsigned int *clk; + void __iomem *regs; + int irqnum; + + regs =3D devm_platform_ioremap_resource(op, 0); + if (IS_ERR(regs)) + return PTR_ERR(regs); =20 - host =3D spi_alloc_host(dev, sizeof(*hw)); + /* Request IRQ */ + irqnum =3D platform_get_irq(op, 0); + if (irqnum < 0) + return irqnum; + + host =3D devm_spi_alloc_host(dev, sizeof(*hw)); if (host =3D=3D NULL) return -ENOMEM; host->dev.of_node =3D np; @@ -379,88 +387,31 @@ static int spi_ppc4xx_of_probe(struct platform_device= *op) opbnp =3D of_find_compatible_node(NULL, NULL, "ibm,opb"); if (opbnp =3D=3D NULL) { dev_err(dev, "OPB: cannot find node\n"); - ret =3D -ENODEV; - goto free_host; + return -ENODEV; } /* Get the clock (Hz) for the OPB */ clk =3D of_get_property(opbnp, "clock-frequency", NULL); + of_node_put(opbnp); if (clk =3D=3D NULL) { dev_err(dev, "OPB: no clock-frequency property set\n"); - of_node_put(opbnp); - ret =3D -ENODEV; - goto free_host; + return -ENODEV; } hw->opb_freq =3D *clk; hw->opb_freq >>=3D 2; - of_node_put(opbnp); - - ret =3D of_address_to_resource(np, 0, &resource); - if (ret) { - dev_err(dev, "error while parsing device node resource\n"); - goto free_host; - } - hw->mapbase =3D resource.start; - hw->mapsize =3D resource_size(&resource); - - /* Sanity check */ - if (hw->mapsize < sizeof(struct spi_ppc4xx_regs)) { - dev_err(dev, "too small to map registers\n"); - ret =3D -EINVAL; - goto free_host; - } - - /* Request IRQ */ - ret =3D platform_get_irq(op, 0); - if (ret < 0) - goto free_host; - hw->irqnum =3D ret; - - ret =3D request_irq(hw->irqnum, spi_ppc4xx_int, - 0, "spi_ppc4xx_of", (void *)hw); - if (ret) { - dev_err(dev, "unable to allocate interrupt\n"); - goto free_host; - } =20 - if (!request_mem_region(hw->mapbase, hw->mapsize, DRIVER_NAME)) { - dev_err(dev, "resource unavailable\n"); - ret =3D -EBUSY; - goto request_mem_error; - } + hw->regs =3D regs; + hw->irqnum =3D irqnum; =20 - hw->regs =3D ioremap(hw->mapbase, sizeof(struct spi_ppc4xx_regs)); - - if (!hw->regs) { - dev_err(dev, "unable to memory map registers\n"); - ret =3D -ENXIO; - goto map_io_error; - } + ret =3D devm_request_irq(&op->dev, hw->irqnum, spi_ppc4xx_int, + 0, "spi_ppc4xx_of", hw); + if (ret) + return dev_err_probe(dev, ret, "unable to allocate interrupt\n"); =20 spi_ppc4xx_enable(hw); =20 /* Finally register our spi controller */ dev->dma_mask =3D 0; - ret =3D spi_bitbang_start(bbp); - if (ret) { - dev_err(dev, "failed to register SPI host\n"); - goto unmap_regs; - } - - dev_info(dev, "driver initialized\n"); - - return 0; - -unmap_regs: - iounmap(hw->regs); -map_io_error: - release_mem_region(hw->mapbase, hw->mapsize); -request_mem_error: - free_irq(hw->irqnum, hw); -free_host: - spi_controller_put(host); - - dev_err(dev, "initialization failed\n"); - return ret; + return spi_bitbang_start(bbp); } =20 static void spi_ppc4xx_of_remove(struct platform_device *op) @@ -469,10 +420,6 @@ static void spi_ppc4xx_of_remove(struct platform_devic= e *op) struct ppc4xx_spi *hw =3D spi_controller_get_devdata(host); =20 spi_bitbang_stop(&hw->bitbang); - release_mem_region(hw->mapbase, hw->mapsize); - free_irq(hw->irqnum, hw); - iounmap(hw->regs); - spi_controller_put(host); } =20 static const struct of_device_id spi_ppc4xx_of_match[] =3D { --=20 2.55.0