2019-09-03 10:37:18

by Vitor Soares

Subject: [PATCH v2 3/5] dt-bindings: i3c: Make 'assigned-address' valid if static address == 0

The I3C devices without a static address can require a specific dynamic
address for priority reasons.

Let's update the binding document to make the 'assigned-address' property
valid if static address == 0 and add an example with this use case.

Signed-off-by: Vitor Soares <[email protected]>
---
Documentation/devicetree/bindings/i3c/i3c.txt | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/i3c/i3c.txt b/Documentation/devicetree/bindings/i3c/i3c.txt
index ab729a0..c851e75 100644
--- a/Documentation/devicetree/bindings/i3c/i3c.txt
+++ b/Documentation/devicetree/bindings/i3c/i3c.txt
@@ -98,9 +98,7 @@ Required properties

Optional properties
-------------------
-- assigned-address: dynamic address to be assigned to this device. This
- property is only valid if the I3C device has a static
- address (first cell of the reg property != 0).
+- assigned-address: dynamic address to be assigned to this device.


Example:
@@ -129,6 +127,15 @@ Example:

/*
* I3C device without a static I2C address but requiring
+ * specific dynamic address.
+ */
+ sensor@0,39200154004 {
+ reg = <0x0 0x6072 0x303904d2>;
+ assigned-address = <0xb>;
+ };
+
+ /*
+ * I3C device without a static I2C address but requiring
* resources described in the DT.
*/
sensor@0,39200154004 {
--
2.7.4

2019-09-03 17:42:19

by Rob Herring (Arm)

Subject: Re: [PATCH v2 3/5] dt-bindings: i3c: Make 'assigned-address' valid if static address == 0

On Tue, 3 Sep 2019 12:35:52 +0200, Vitor Soares wrote:
> The I3C devices without a static address can require a specific dynamic
> address for priority reasons.
>
> Let's update the binding document to make the 'assigned-address' property
> valid if static address == 0 and add an example with this use case.
>
> Signed-off-by: Vitor Soares <[email protected]>
> ---
> Documentation/devicetree/bindings/i3c/i3c.txt | 13 ++++++++++---
> 1 file changed, 10 insertions(+), 3 deletions(-)
>

Reviewed-by: Rob Herring <[email protected]>