* [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties
@ 2023-06-09 19:33 Rob Herring
2023-06-09 20:21 ` Florian Fainelli
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Rob Herring @ 2023-06-09 19:33 UTC (permalink / raw)
To: Lorenzo Pieralisi, Krzysztof Wilczyński, Bjorn Helgaas,
Ray Jui, Scott Branden, Broadcom internal kernel review list
Cc: linux-pci, linux-arm-kernel, linux-kernel
It is preferred to use typed property access functions (i.e.
of_property_read_<type> functions) rather than low-level
of_get_property/of_find_property functions for reading properties.
Convert reading boolean properties to of_property_read_bool().
Signed-off-by: Rob Herring <robh@kernel.org>
---
v2:
- Commit msg: s/to to/to/
---
drivers/pci/controller/pcie-iproc-msi.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/pci/controller/pcie-iproc-msi.c b/drivers/pci/controller/pcie-iproc-msi.c
index fee036b07cd4..649fcb449f34 100644
--- a/drivers/pci/controller/pcie-iproc-msi.c
+++ b/drivers/pci/controller/pcie-iproc-msi.c
@@ -525,7 +525,7 @@ int iproc_msi_init(struct iproc_pcie *pcie, struct device_node *node)
if (!of_device_is_compatible(node, "brcm,iproc-msi"))
return -ENODEV;
- if (!of_find_property(node, "msi-controller", NULL))
+ if (!of_property_read_bool(node, "msi-controller"))
return -ENODEV;
if (pcie->msi)
@@ -585,8 +585,7 @@ int iproc_msi_init(struct iproc_pcie *pcie, struct device_node *node)
return -EINVAL;
}
- if (of_find_property(node, "brcm,pcie-msi-inten", NULL))
- msi->has_inten_reg = true;
+ msi->has_inten_reg = of_property_read_bool(node, "brcm,pcie-msi-inten");
msi->nr_msi_vecs = msi->nr_irqs * EQ_LEN;
msi->bitmap = devm_bitmap_zalloc(pcie->dev, msi->nr_msi_vecs,
--
2.39.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties
2023-06-09 19:33 [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties Rob Herring
@ 2023-06-09 20:21 ` Florian Fainelli
2023-06-12 16:18 ` Ray Jui
2023-06-24 9:02 ` Krzysztof Wilczyński
2 siblings, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2023-06-09 20:21 UTC (permalink / raw)
To: Rob Herring, Lorenzo Pieralisi, Krzysztof Wilczyński,
Bjorn Helgaas, Ray Jui, Scott Branden,
Broadcom internal kernel review list
Cc: linux-pci, linux-arm-kernel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 418 bytes --]
On 6/9/23 12:33, Rob Herring wrote:
> It is preferred to use typed property access functions (i.e.
> of_property_read_<type> functions) rather than low-level
> of_get_property/of_find_property functions for reading properties.
> Convert reading boolean properties to of_property_read_bool().
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
--
Florian
[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4221 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties
2023-06-09 19:33 [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties Rob Herring
2023-06-09 20:21 ` Florian Fainelli
@ 2023-06-12 16:18 ` Ray Jui
2023-06-24 9:02 ` Krzysztof Wilczyński
2 siblings, 0 replies; 4+ messages in thread
From: Ray Jui @ 2023-06-12 16:18 UTC (permalink / raw)
To: Rob Herring, Lorenzo Pieralisi, Krzysztof Wilczyński,
Bjorn Helgaas, Ray Jui, Scott Branden,
Broadcom internal kernel review list
Cc: linux-pci, linux-arm-kernel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 400 bytes --]
On 6/9/2023 12:33 PM, Rob Herring wrote:
> It is preferred to use typed property access functions (i.e.
> of_property_read_<type> functions) rather than low-level
> of_get_property/of_find_property functions for reading properties.
> Convert reading boolean properties to of_property_read_bool().
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Ray Jui <ray.jui@broadcom.com>
Thanks!
[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4194 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties
2023-06-09 19:33 [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties Rob Herring
2023-06-09 20:21 ` Florian Fainelli
2023-06-12 16:18 ` Ray Jui
@ 2023-06-24 9:02 ` Krzysztof Wilczyński
2 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Wilczyński @ 2023-06-24 9:02 UTC (permalink / raw)
To: Rob Herring
Cc: Lorenzo Pieralisi, Bjorn Helgaas, Ray Jui, Scott Branden,
Broadcom internal kernel review list, linux-pci, linux-arm-kernel,
linux-kernel
Hello,
> It is preferred to use typed property access functions (i.e.
> of_property_read_<type> functions) rather than low-level
> of_get_property/of_find_property functions for reading properties.
> Convert reading boolean properties to of_property_read_bool().
Applied to controller/iproc, thank you!
[1/1] PCI: iproc: Use of_property_read_bool() for boolean properties
https://git.kernel.org/pci/pci/c/4ce7d88e7ad9
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-06-24 9:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-09 19:33 [PATCH v2] PCI: iproc: Use of_property_read_bool() for boolean properties Rob Herring
2023-06-09 20:21 ` Florian Fainelli
2023-06-12 16:18 ` Ray Jui
2023-06-24 9:02 ` Krzysztof Wilczyński
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).