diff options
| author | Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> | 2025-12-24 14:02:41 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-01-13 14:43:00 +0300 |
| commit | a014c203b54d9013ad52ad8a531cf46e71028f2b (patch) | |
| tree | 7a9eed60e025285a53dda14cb0c587108a323b77 | |
| parent | 8672e4b51adfc57150f3862b1665faff0acf1bad (diff) | |
| download | linux-a014c203b54d9013ad52ad8a531cf46e71028f2b.tar.xz | |
spi: fsi: Simplify with scoped for each OF child loop
Use scoped for_each_available_child_of_node_scoped() when iterating over
device nodes to make code a bit simpler.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Link: https://patch.msgid.link/20251224110240.56803-2-krzysztof.kozlowski@oss.qualcomm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/spi/spi-fsi.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/spi/spi-fsi.c b/drivers/spi/spi-fsi.c index e01c63d23b64..68276d195917 100644 --- a/drivers/spi/spi-fsi.c +++ b/drivers/spi/spi-fsi.c @@ -531,7 +531,6 @@ static size_t fsi_spi_max_transfer_size(struct spi_device *spi) static int fsi_spi_probe(struct device *dev) { int rc; - struct device_node *np; int num_controllers_registered = 0; struct fsi2spi *bridge; struct fsi_device *fsi = to_fsi_dev(dev); @@ -547,7 +546,7 @@ static int fsi_spi_probe(struct device *dev) bridge->fsi = fsi; mutex_init(&bridge->lock); - for_each_available_child_of_node(dev->of_node, np) { + for_each_available_child_of_node_scoped(dev->of_node, np) { u32 base; struct fsi_spi *ctx; struct spi_controller *ctlr; @@ -556,10 +555,8 @@ static int fsi_spi_probe(struct device *dev) continue; ctlr = spi_alloc_host(dev, sizeof(*ctx)); - if (!ctlr) { - of_node_put(np); + if (!ctlr) break; - } ctlr->dev.of_node = np; ctlr->num_chipselect = of_get_available_child_count(np) ?: 1; |