| 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
| ||
|
Message-ID: <20260108203004.3538449-5-andriy.shevchenko@linux.intel.com>
Date: Thu, 8 Jan 2026 21:23:41 +0100
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Mark Brown <broonie@...nel.org>,
Varshini Rajendran <varshini.rajendran@...rochip.com>,
Mikhail Kshevetskiy <mikhail.kshevetskiy@...sys.eu>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Sunny Luo <sunny.luo@...ogic.com>,
Janne Grunau <j@...nau.net>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
Chin-Ting Kuo <chin-ting_kuo@...eedtech.com>,
CL Wang <cl634@...estech.com>,
Manikandan Muralidharan <manikandan.m@...rochip.com>,
David Lechner <dlechner@...libre.com>,
Florian Fainelli <florian.fainelli@...adcom.com>,
Jonas Gorski <jonas.gorski@...il.com>,
Hang Zhou <929513338@...com>,
Jun Guo <jun.guo@...tech.com>,
Philipp Stanner <phasta@...nel.org>,
Charles Keepax <ckeepax@...nsource.cirrus.com>,
Bartosz Golaszewski <brgl@...nel.org>,
Benoît Monin <benoit.monin@...tlin.com>,
Shiji Yang <yangshiji66@...look.com>,
James Clark <james.clark@...aro.org>,
Jonathan Marek <jonathan@...ek.ca>,
Carlos Song <carlos.song@....com>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Huacai Chen <chenhuacai@...nel.org>,
Xianwei Zhao <xianwei.zhao@...ogic.com>,
Prajna Rajendra Kumar <prajna.rajendrakumar@...rochip.com>,
Sergio Perez Gonzalez <sperezglz@...il.com>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Qianfeng Rong <rongqianfeng@...o.com>,
Haibo Chen <haibo.chen@....com>,
Gabor Juhos <j4g8y7@...il.com>,
Md Sadre Alam <quic_mdalam@...cinc.com>,
Rosen Penev <rosenp@...il.com>,
Luis de Arquer <luis.dearquer@...rtim.com>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Cosmin Tanislav <cosmin-gabriel.tanislav.xa@...esas.com>,
Tudor Ambarus <tudor.ambarus@...aro.org>,
Krzysztof Kozlowski <krzk@...nel.org>,
Longbin Li <looong.bin@...il.com>,
Patrice Chotard <patrice.chotard@...s.st.com>,
Clément Le Goffic <clement.legoffic@...s.st.com>,
Alessandro Grassi <alessandro.grassi@...lbox.org>,
Chen-Yu Tsai <wens@...nel.org>,
Darshan R <rathod.darshan.0896@...il.com>,
Aaron Kling <webgeek1234@...il.com>,
Vishwaroop A <va@...dia.com>,
Haixu Cui <quic_haixcui@...cinc.com>,
Darshan Rathod <darshanrathod475@...il.com>,
linux-spi@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org,
linux-amlogic@...ts.infradead.org,
asahi@...ts.linux.dev,
linux-aspeed@...ts.ozlabs.org,
openbmc@...ts.ozlabs.org,
linux-rpi-kernel@...ts.infradead.org,
linux-sound@...r.kernel.org,
patches@...nsource.cirrus.com,
imx@...ts.linux.dev,
linux-arm-msm@...r.kernel.org,
linux-riscv@...ts.infradead.org,
linux-mediatek@...ts.infradead.org,
linux-rockchip@...ts.infradead.org,
linux-renesas-soc@...r.kernel.org,
linux-samsung-soc@...r.kernel.org,
linux-stm32@...md-mailman.stormreply.com,
linux-sunxi@...ts.linux.dev,
linux-tegra@...r.kernel.org,
virtualization@...ts.linux.dev
Cc: Nicolas Ferre <nicolas.ferre@...rochip.com>,
Alexandre Belloni <alexandre.belloni@...tlin.com>,
Claudiu Beznea <claudiu.beznea@...on.dev>,
Lorenzo Bianconi <lorenzo@...nel.org>,
Ray Liu <ray.liu@...oha.com>,
Sven Peter <sven@...nel.org>,
Neal Gompa <neal@...pa.dev>,
Cédric Le Goater <clg@...d.org>,
Joel Stanley <joel@....id.au>,
Andrew Jeffery <andrew@...econstruct.com.au>,
Ryan Wanner <ryan.wanner@...rochip.com>,
Michael Hennerich <michael.hennerich@...log.com>,
Nuno Sá <nuno.sa@...log.com>,
Kamal Dasu <kamal.dasu@...adcom.com>,
Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>,
Ray Jui <rjui@...adcom.com>,
Scott Branden <sbranden@...adcom.com>,
William Zhang <william.zhang@...adcom.com>,
Kursad Oney <kursad.oney@...adcom.com>,
Anand Gore <anand.gore@...adcom.com>,
Rafał Miłecki <rafal@...ecki.pl>,
David Rhodes <david.rhodes@...rus.com>,
Richard Fitzgerald <rf@...nsource.cirrus.com>,
Vladimir Oltean <olteanv@...il.com>,
Frank Li <Frank.Li@....com>,
Jean-Marie Verdun <verdun@....com>,
Nick Hawkins <nick.hawkins@....com>,
Yang Shen <shenyang39@...wei.com>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
Lixu Zhang <lixu.zhang@...el.com>,
Yinbo Zhu <zhuyinbo@...ngson.cn>,
Neil Armstrong <neil.armstrong@...aro.org>,
Kevin Hilman <khilman@...libre.com>,
Jerome Brunet <jbrunet@...libre.com>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
Conor Dooley <conor.dooley@...rochip.com>,
Daire McNamara <daire.mcnamara@...rochip.com>,
Matthias Brugger <matthias.bgg@...il.com>,
Avi Fishman <avifishman70@...il.com>,
Tomer Maimon <tmaimon77@...il.com>,
Tali Perry <tali.perry1@...il.com>,
Patrick Venture <venture@...gle.com>,
Nancy Yuen <yuenn@...gle.com>,
Benjamin Fair <benjaminfair@...gle.com>,
Han Xu <han.xu@....com>,
Yogesh Gaur <yogeshgaur.83@...il.com>,
Linus Walleij <linusw@...nel.org>,
Daniel Mack <daniel@...que.org>,
Haojian Zhuang <haojian.zhuang@...il.com>,
Robert Jarzmik <robert.jarzmik@...e.fr>,
Chris Packham <chris.packham@...iedtelesis.co.nz>,
Heiko Stuebner <heiko@...ech.de>,
Fabrizio Castro <fabrizio.castro.jz@...esas.com>,
Andi Shyti <andi.shyti@...nel.org>,
Alim Akhtar <alim.akhtar@...sung.com>,
Paul Walmsley <pjw@...nel.org>,
Samuel Holland <samuel.holland@...ive.com>,
Orson Zhai <orsonzhai@...il.com>,
Baolin Wang <baolin.wang@...ux.alibaba.com>,
Chunyan Zhang <zhang.lyra@...il.com>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>,
Alain Volmat <alain.volmat@...s.st.com>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Li-hao Kuo <lhjeff911@...il.com>,
Masahisa Kojima <masahisa.kojima@...aro.org>,
Jassi Brar <jaswinder.singh@...aro.org>,
Laxman Dewangan <ldewangan@...dia.com>,
Thierry Reding <thierry.reding@...il.com>,
Jonathan Hunter <jonathanh@...dia.com>,
Sowjanya Komatineni <skomatineni@...dia.com>,
Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>,
Masami Hiramatsu <mhiramat@...nel.org>,
Jonathan Neuschäfer <j.neuschaefer@....net>,
Michal Simek <michal.simek@....com>,
Max Filippov <jcmvbkbc@...il.com>
Subject: [PATCH v1 4/4] spi: Drop duplicate device_set_node() call
The SPI core provides the default fwnode for the controller,
assigned by device_set_node(). No need to repeat it in the driver.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
---
drivers/spi/spi-airoha-snfi.c | 1 -
drivers/spi/spi-dln2.c | 2 --
drivers/spi/spi-dw-core.c | 2 --
drivers/spi/spi-ep93xx.c | 1 -
drivers/spi/spi-gpio.c | 1 -
drivers/spi/spi-ljca.c | 1 -
drivers/spi/spi-loongson-core.c | 1 -
drivers/spi/spi-mpc512x-psc.c | 2 --
drivers/spi/spi-mpc52xx-psc.c | 2 --
drivers/spi/spi-nxp-fspi.c | 2 --
drivers/spi/spi-pxa2xx.c | 2 --
drivers/spi/spi-realtek-rtl-snand.c | 1 -
drivers/spi/spi-rzv2h-rspi.c | 2 --
drivers/spi/spi-rzv2m-csi.c | 2 --
drivers/spi/spi-sc18is602.c | 2 --
drivers/spi/spi-sunplus-sp7021.c | 1 -
drivers/spi/spi-virtio.c | 2 --
17 files changed, 27 deletions(-)
diff --git a/drivers/spi/spi-airoha-snfi.c b/drivers/spi/spi-airoha-snfi.c
index 70327aebc26b..7b6c09f91fef 100644
--- a/drivers/spi/spi-airoha-snfi.c
+++ b/drivers/spi/spi-airoha-snfi.c
@@ -1124,7 +1124,6 @@ static int airoha_snand_probe(struct platform_device *pdev)
ctrl->bits_per_word_mask = SPI_BPW_MASK(8);
ctrl->mode_bits = SPI_RX_DUAL;
ctrl->setup = airoha_snand_setup;
- device_set_node(&ctrl->dev, dev_fwnode(dev));
err = airoha_snand_nfi_init(as_ctrl);
if (err)
diff --git a/drivers/spi/spi-dln2.c b/drivers/spi/spi-dln2.c
index 2013bc56ded8..fac737e18ac5 100644
--- a/drivers/spi/spi-dln2.c
+++ b/drivers/spi/spi-dln2.c
@@ -689,8 +689,6 @@ static int dln2_spi_probe(struct platform_device *pdev)
if (!host)
return -ENOMEM;
- device_set_node(&host->dev, dev_fwnode(dev));
-
platform_set_drvdata(pdev, host);
dln2 = spi_controller_get_devdata(host);
diff --git a/drivers/spi/spi-dw-core.c b/drivers/spi/spi-dw-core.c
index 9ebf244294f8..0d59c141beb0 100644
--- a/drivers/spi/spi-dw-core.c
+++ b/drivers/spi/spi-dw-core.c
@@ -936,8 +936,6 @@ int dw_spi_add_controller(struct device *dev, struct dw_spi *dws)
if (!ctlr)
return -ENOMEM;
- device_set_node(&ctlr->dev, dev_fwnode(dev));
-
dws->ctlr = ctlr;
dws->dma_addr = (dma_addr_t)(dws->paddr + DW_SPI_DR);
diff --git a/drivers/spi/spi-ep93xx.c b/drivers/spi/spi-ep93xx.c
index e1d097091925..90d5f3ea6508 100644
--- a/drivers/spi/spi-ep93xx.c
+++ b/drivers/spi/spi-ep93xx.c
@@ -689,7 +689,6 @@ static int ep93xx_spi_probe(struct platform_device *pdev)
/* make sure that the hardware is disabled */
writel(0, espi->mmio + SSPCR1);
- device_set_node(&host->dev, dev_fwnode(&pdev->dev));
error = devm_spi_register_controller(&pdev->dev, host);
if (error) {
dev_err(&pdev->dev, "failed to register SPI host\n");
diff --git a/drivers/spi/spi-gpio.c b/drivers/spi/spi-gpio.c
index c8dadb532c40..072127a38fad 100644
--- a/drivers/spi/spi-gpio.c
+++ b/drivers/spi/spi-gpio.c
@@ -351,7 +351,6 @@ static int spi_gpio_probe(struct platform_device *pdev)
return -ENOMEM;
if (fwnode) {
- device_set_node(&host->dev, fwnode);
host->use_gpio_descriptors = true;
} else {
status = spi_gpio_probe_pdata(pdev, host);
diff --git a/drivers/spi/spi-ljca.c b/drivers/spi/spi-ljca.c
index 3f412cf8f1cd..0c6e6248d8ba 100644
--- a/drivers/spi/spi-ljca.c
+++ b/drivers/spi/spi-ljca.c
@@ -238,7 +238,6 @@ static int ljca_spi_probe(struct auxiliary_device *auxdev,
controller->auto_runtime_pm = false;
controller->max_speed_hz = LJCA_SPI_BUS_MAX_HZ;
- device_set_node(&ljca_spi->controller->dev, dev_fwnode(&auxdev->dev));
auxiliary_set_drvdata(auxdev, controller);
ret = spi_register_controller(controller);
diff --git a/drivers/spi/spi-loongson-core.c b/drivers/spi/spi-loongson-core.c
index b46f072a0387..f50423c3db4c 100644
--- a/drivers/spi/spi-loongson-core.c
+++ b/drivers/spi/spi-loongson-core.c
@@ -210,7 +210,6 @@ int loongson_spi_init_controller(struct device *dev, void __iomem *regs)
controller->unprepare_message = loongson_spi_unprepare_message;
controller->set_cs = loongson_spi_set_cs;
controller->num_chipselect = 4;
- device_set_node(&controller->dev, dev_fwnode(dev));
dev_set_drvdata(dev, controller);
spi = spi_controller_get_devdata(controller);
diff --git a/drivers/spi/spi-mpc512x-psc.c b/drivers/spi/spi-mpc512x-psc.c
index 3c1638ba5bee..a1aeb5403a74 100644
--- a/drivers/spi/spi-mpc512x-psc.c
+++ b/drivers/spi/spi-mpc512x-psc.c
@@ -480,8 +480,6 @@ static int mpc512x_psc_spi_of_probe(struct platform_device *pdev)
host->use_gpio_descriptors = true;
host->cleanup = mpc512x_psc_spi_cleanup;
- device_set_node(&host->dev, dev_fwnode(dev));
-
tempp = devm_platform_get_and_ioremap_resource(pdev, 0, NULL);
if (IS_ERR(tempp))
return dev_err_probe(dev, PTR_ERR(tempp), "could not ioremap I/O port range\n");
diff --git a/drivers/spi/spi-mpc52xx-psc.c b/drivers/spi/spi-mpc52xx-psc.c
index 3bbeb8d5bfb8..73d2383461ca 100644
--- a/drivers/spi/spi-mpc52xx-psc.c
+++ b/drivers/spi/spi-mpc52xx-psc.c
@@ -319,8 +319,6 @@ static int mpc52xx_psc_spi_of_probe(struct platform_device *pdev)
host->transfer_one_message = mpc52xx_psc_spi_transfer_one_message;
host->cleanup = mpc52xx_psc_spi_cleanup;
- device_set_node(&host->dev, dev_fwnode(dev));
-
mps->psc = devm_platform_get_and_ioremap_resource(pdev, 0, NULL);
if (IS_ERR(mps->psc))
return dev_err_probe(dev, PTR_ERR(mps->psc), "could not ioremap I/O port range\n");
diff --git a/drivers/spi/spi-nxp-fspi.c b/drivers/spi/spi-nxp-fspi.c
index 50a7e4916a60..320b3d93df57 100644
--- a/drivers/spi/spi-nxp-fspi.c
+++ b/drivers/spi/spi-nxp-fspi.c
@@ -1383,8 +1383,6 @@ static int nxp_fspi_probe(struct platform_device *pdev)
else
ctlr->mem_caps = &nxp_fspi_mem_caps;
- device_set_node(&ctlr->dev, fwnode);
-
ret = devm_add_action_or_reset(dev, nxp_fspi_cleanup, f);
if (ret)
return ret;
diff --git a/drivers/spi/spi-pxa2xx.c b/drivers/spi/spi-pxa2xx.c
index ec7117a94d5f..78c399e95ef2 100644
--- a/drivers/spi/spi-pxa2xx.c
+++ b/drivers/spi/spi-pxa2xx.c
@@ -1290,8 +1290,6 @@ int pxa2xx_spi_probe(struct device *dev, struct ssp_device *ssp,
drv_data->controller_info = platform_info;
drv_data->ssp = ssp;
- device_set_node(&controller->dev, dev_fwnode(dev));
-
/* The spi->mode bits understood by this driver: */
controller->mode_bits = SPI_CPOL | SPI_CPHA | SPI_CS_HIGH | SPI_LOOP;
diff --git a/drivers/spi/spi-realtek-rtl-snand.c b/drivers/spi/spi-realtek-rtl-snand.c
index 741cf2af3e91..7d5853d202c6 100644
--- a/drivers/spi/spi-realtek-rtl-snand.c
+++ b/drivers/spi/spi-realtek-rtl-snand.c
@@ -400,7 +400,6 @@ static int rtl_snand_probe(struct platform_device *pdev)
ctrl->mem_ops = &rtl_snand_mem_ops;
ctrl->bits_per_word_mask = SPI_BPW_MASK(8);
ctrl->mode_bits = SPI_RX_DUAL | SPI_RX_QUAD | SPI_TX_DUAL | SPI_TX_QUAD;
- device_set_node(&ctrl->dev, dev_fwnode(dev));
return devm_spi_register_controller(dev, ctrl);
}
diff --git a/drivers/spi/spi-rzv2h-rspi.c b/drivers/spi/spi-rzv2h-rspi.c
index aae916882915..23f0e92ae208 100644
--- a/drivers/spi/spi-rzv2h-rspi.c
+++ b/drivers/spi/spi-rzv2h-rspi.c
@@ -797,8 +797,6 @@ static int rzv2h_rspi_probe(struct platform_device *pdev)
controller->dma_rx = NULL;
}
- device_set_node(&controller->dev, dev_fwnode(dev));
-
ret = devm_spi_register_controller(dev, controller);
if (ret)
dev_err(dev, "register controller failed\n");
diff --git a/drivers/spi/spi-rzv2m-csi.c b/drivers/spi/spi-rzv2m-csi.c
index 7c0442883ac0..5d80939dddb5 100644
--- a/drivers/spi/spi-rzv2m-csi.c
+++ b/drivers/spi/spi-rzv2m-csi.c
@@ -634,8 +634,6 @@ static int rzv2m_csi_probe(struct platform_device *pdev)
controller->use_gpio_descriptors = true;
controller->target_abort = rzv2m_csi_target_abort;
- device_set_node(&controller->dev, dev_fwnode(dev));
-
ret = devm_request_irq(dev, irq, rzv2m_csi_irq_handler, 0,
dev_name(dev), csi);
if (ret)
diff --git a/drivers/spi/spi-sc18is602.c b/drivers/spi/spi-sc18is602.c
index 1627aa66c965..78c558e7228e 100644
--- a/drivers/spi/spi-sc18is602.c
+++ b/drivers/spi/spi-sc18is602.c
@@ -251,8 +251,6 @@ static int sc18is602_probe(struct i2c_client *client)
if (!host)
return -ENOMEM;
- device_set_node(&host->dev, dev_fwnode(dev));
-
hw = spi_controller_get_devdata(host);
/* assert reset and then release */
diff --git a/drivers/spi/spi-sunplus-sp7021.c b/drivers/spi/spi-sunplus-sp7021.c
index 256ae07db6be..789b092fe8c0 100644
--- a/drivers/spi/spi-sunplus-sp7021.c
+++ b/drivers/spi/spi-sunplus-sp7021.c
@@ -419,7 +419,6 @@ static int sp7021_spi_controller_probe(struct platform_device *pdev)
ctlr = devm_spi_alloc_host(dev, sizeof(*pspim));
if (!ctlr)
return -ENOMEM;
- device_set_node(&ctlr->dev, dev_fwnode(dev));
ctlr->bus_num = pdev->id;
ctlr->mode_bits = SPI_CPOL | SPI_CPHA | SPI_CS_HIGH | SPI_LSB_FIRST;
ctlr->auto_runtime_pm = true;
diff --git a/drivers/spi/spi-virtio.c b/drivers/spi/spi-virtio.c
index 6aad9f1fd016..9e66c917fb75 100644
--- a/drivers/spi/spi-virtio.c
+++ b/drivers/spi/spi-virtio.c
@@ -344,8 +344,6 @@ static int virtio_spi_probe(struct virtio_device *vdev)
priv->vdev = vdev;
vdev->priv = priv;
- device_set_node(&ctrl->dev, dev_fwnode(&vdev->dev));
-
dev_set_drvdata(&vdev->dev, ctrl);
virtio_spi_read_config(vdev);
--
2.50.1
Powered by blists - more mailing lists