On Sun, Dec 17, 2023 at 09:16:39PM +0800, jingbao qiu wrote: > On Sun, Dec 17, 2023 at 8:26 PM Conor Dooley <conor@xxxxxxxxxx> wrote: > > > > On Sun, Dec 17, 2023 at 07:09:50PM +0800, Jingbao Qiu wrote: > > > > > + reg: > > > + items: > > > + - description: data register > > > + - description: control register > > > > > + rtc@5025000{ > > > + compatible = "sophgo,cv1800-rtc"; > > > + reg = <0x5025000 0x1000>, <0x5026000 0x1000>; > > > > Why are these two regions rather than just one, given they are located > > next to one another? > > Are they separate on one of the other devices in this family? > > > > Thanks, > > Conor. > > > > I think there are two reasons, the first one is to distinguish > different logical , > REG_ CTRL (base on 0x5025000) controls clock calibration, sleep,and other > functions, RTC_ CORE (base on 0x5026000) has basic RTC functionality, > The second is the maximum address used by RTC_CTRL (base on 0x5025000) > is 0x0ac,which is much smaller than 0x1000. Therefore, the datasheet divides > it into two parts for introduction, and I also divide it into two > parts based on this > introduction.So do you suggest that I merge them together? If all of the cv1800 series devices have them sequentially, I would just make them one region.
Attachment:
signature.asc
Description: PGP signature