Address review comments.

src/darwin/Framework/CHIP/MTRDeviceController.mm

@@ -141,6 +141,8 @@ @implementation MTRDeviceController {
     os_unfair_lock _assertionLock;
 }
 
+@property (readwrite, nonatomic) NSUUID * uniqueIdentifier;
+
 @synthesize uniqueIdentifier = _uniqueIdentifier;
 
 - (os_unfair_lock_t)deviceMapLock
@@ -154,7 +156,7 @@ - (instancetype)initForSubclasses:(BOOL)startSuspended uniqueIdentifier:(NSUUID
         // nothing, as superclass of MTRDeviceController is NSObject
     }
 
-    _uniqueIdentifier = uniqueIdentifier;
+    self.uniqueIdentifier = uniqueIdentifier;
 
     _underlyingDeviceMapLock = OS_UNFAIR_LOCK_INIT;
 
@@ -165,7 +167,8 @@ - (instancetype)initForSubclasses:(BOOL)startSuspended uniqueIdentifier:(NSUUID
 
     // All synchronous suspend/resume activity has to be protected by
     // @synchronized(self), so that parts of suspend/resume can't
-    // interleave with each other.
+    // interleave with each other. Using @synchronized here because
+    // MTRDevice may call isSuspended.
     _suspended = startSuspended;
 
     _nodeIDToDeviceMap = [NSMapTable strongToWeakObjectsMapTable];
@@ -354,7 +357,6 @@ - (BOOL)isRunning
 
 - (BOOL)isSuspended
 {
-    // Using @synchronized here because MTRDevice may call isSuspended.
     @synchronized(self) {
         return _suspended;
     }