[PATCH 0/7] of: Constify DT structs

Rob Herring (Arm) posted 7 patches 1 year, 2 months ago
drivers/of/address.c       | 22 +++++++++++-----------
drivers/of/base.c          | 20 ++++++++++----------
drivers/of/cpu.c           |  2 +-
drivers/of/dynamic.c       |  4 ++--
drivers/of/irq.c           |  4 ++--
drivers/of/kobj.c          |  8 ++++----
drivers/of/of_private.h    | 12 ++++++------
drivers/of/overlay.c       | 19 ++++++++++---------
drivers/of/property.c      | 10 +++++-----
drivers/of/resolver.c      | 12 ++++++------
drivers/pci/pci.c          |  2 +-
include/linux/logic_pio.h  |  6 +++---
include/linux/of.h         | 28 ++++++++++++++--------------
include/linux/of_address.h |  6 +++---
include/linux/of_irq.h     |  4 ++--
include/linux/pci.h        |  2 +-
lib/logic_pio.c            |  4 ++--
17 files changed, 83 insertions(+), 82 deletions(-)
[PATCH 0/7] of: Constify DT structs
Posted by Rob Herring (Arm) 1 year, 2 months ago
This series constifies many usages of DT structs in the DT core code. 
Many uses of struct device_node where the node refcount is not 
changed can be const. Most uses of struct property can also be const.

The first 2 patches are dependencies. The functions called by the 
DT core where the fwnode_handle needs to be const to make the containing 
device_node const.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
Rob Herring (Arm) (7):
      PCI: Constify pci_register_io_range() fwnode_handle
      logic_pio: Constify fwnode_handle
      of: Constify struct device_node function arguments
      of: Constify struct property pointers
      of: Constify of_changeset_entry function arguments
      of: Constify safe_name() kobject arg
      of/address: Constify of_busses[] array and pointers

 drivers/of/address.c       | 22 +++++++++++-----------
 drivers/of/base.c          | 20 ++++++++++----------
 drivers/of/cpu.c           |  2 +-
 drivers/of/dynamic.c       |  4 ++--
 drivers/of/irq.c           |  4 ++--
 drivers/of/kobj.c          |  8 ++++----
 drivers/of/of_private.h    | 12 ++++++------
 drivers/of/overlay.c       | 19 ++++++++++---------
 drivers/of/property.c      | 10 +++++-----
 drivers/of/resolver.c      | 12 ++++++------
 drivers/pci/pci.c          |  2 +-
 include/linux/logic_pio.h  |  6 +++---
 include/linux/of.h         | 28 ++++++++++++++--------------
 include/linux/of_address.h |  6 +++---
 include/linux/of_irq.h     |  4 ++--
 include/linux/pci.h        |  2 +-
 lib/logic_pio.c            |  4 ++--
 17 files changed, 83 insertions(+), 82 deletions(-)
---
base-commit: 9852d85ec9d492ebef56dc5f229416c925758edc
change-id: 20241010-dt-const-7ceef73df29c

Best regards,
-- 
Rob Herring (Arm) <robh@kernel.org>
Re: [PATCH 0/7] of: Constify DT structs
Posted by Krzysztof Kozlowski 1 year, 2 months ago
On Thu, Oct 10, 2024 at 11:27:13AM -0500, Rob Herring (Arm) wrote:
> This series constifies many usages of DT structs in the DT core code. 
> Many uses of struct device_node where the node refcount is not 
> changed can be const. Most uses of struct property can also be const.
> 
> The first 2 patches are dependencies. The functions called by the 
> DT core where the fwnode_handle needs to be const to make the containing 
> device_node const.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---

I gave reviewes for individual patches, but all look correct except
order between #1 and #2.

Therefore with the order fixed:

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof