Hi Enric, Am Donnerstag, 19. April 2018, 12:40:15 CEST schrieb Enric Balletbo i Serra: > DDR3 SDRAM Standard (JESD79-3F) defines some standard speed bins for > DDR3 memories. The devfreq/rk3399_dmc.txt binding refers to this file > which does not exist, so add a ddr.h file with the standard speed bins > for DDR3. > > Fixes: c1ceb8f7c167 (Documentation: bindings: add dt documentation for rk3399 dmc) > Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx> > --- > > include/dt-bindings/clock/ddr.h | 34 +++++++++++++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 include/dt-bindings/clock/ddr.h > > diff --git a/include/dt-bindings/clock/ddr.h b/include/dt-bindings/clock/ddr.h > new file mode 100644 > index 000000000000..506aef7e609e > --- /dev/null > +++ b/include/dt-bindings/clock/ddr.h > @@ -0,0 +1,34 @@ > +/* SPDX-License-Identifier: (GPL-2.0+ OR MIT) */ > + > +#ifndef DT_BINDINGS_DDR_H > +#define DT_BINDINGS_DDR_H > + > +/* DDR3-800 Standard Speed Bins */ > +#define DDR3_800D 15 > +#define DDR3_800E 18 > +/* DDR3-1066 Standard Speed Bins */ > +#define DDR3_1066E 18 > +#define DDR3_1066F 21 > +#define DDR3_1066G 24 looking at the mentioned jedec standard, I don't see where these numerical values are defined in the standard itself. [I may be blind though] Could you explain a bit more where these numerical values are coming from? Thanks Heiko -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html