Hello! > > +Bank configurations can be defined as an optional subnodes. They can have the > > +following properties: > > +- bank : bank number (0 - 3) > > +- width : data width in bytes (1 or 2). If omitted, default of 1 is used. > > +- srom-timing : array of 7 integers: Tacp, Tcah, Tcoh, Tacc, Tcos, Tacs > > + > > I think these will be optional properties as all Exynos SoC may not be > having these. So better to add them under "Optional Properties" heading, > and move this above the "-Example" heading Not really. Actually 'bank' is not optional, because it actually identifies to which bank properties are applied, and srom-timings are settings by themselves. Without both of them the definition just doesn't make sense. The only really optional specifier is 'width', which has a default. Could you give me an example of how to write this correctly? Kind regards, Pavel Fedin Expert Engineer Samsung Electronics Research center Russia -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html