On 06/10/2021 17:41, Dmitry Osipenko wrote: > 06.10.2021 13:57, Krzysztof Kozlowski пишет: >>> + density: >>> + description: | >>> + Density in megabits of SDRAM chip. Obtained from device datasheet. >> You need here a type/ref, so uint32. >> > > The type is uint32 by default. I can add it, but it's not really necessary. Hmmm, maybe I missed some background here. Who sets uint32 as default? The schema does not define "density" so this is more like a vendor-type property which required therefore a type. Also Rob's pointed this to my patch here around "op_mode": https://lore.kernel.org/linux-samsung-soc/cdcd4eda-a7a9-2aa2-1316-e7184ff30bf3@xxxxxxxxxxxxx/T/#m17adea693a9cbcca75b0ba6f9d878f5fd1bf5d14 Best regards, Krzysztof
