Hi Sören On Mon, May 5, 2014 at 6:16 PM, Sören Brinkmann <soren.brinkmann@xxxxxxxxxx> wrote: > > Hi Thor, > > On Mon, 2014-05-05 at 05:52PM -0500, tthayer@xxxxxxxxxx wrote: > > From: Thor Thayer <tthayer@xxxxxxxxxx> > > > > Addition of the Altera SDRAM controller bindings and device > > tree changes to the Altera SoC project. The "syscon" parameter > > is included here because the SDRAM EDAC bits are shared with the SDRAM > > configuration bits. > > --- > > v2: Changes to SoC SDRAM EDAC code. > > > > V3: Implement code suggestions for SDRAM EDAC code. > > > > Signed-off-by: Thor Thayer <tthayer@xxxxxxxxxx> > > --- > > .../bindings/arm/altera/socfpga-sdram.txt | 14 ++++++++++++++ > > arch/arm/boot/dts/socfpga.dtsi | 5 +++++ > > 2 files changed, 19 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/arm/altera/socfpga-sdram.txt > > > > diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-sdram.txt b/Documentation/devicetree/bindings/arm/altera/socfpga-sdram.txt > > new file mode 100644 > > index 0000000..525cb76 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/arm/altera/socfpga-sdram.txt > > @@ -0,0 +1,14 @@ > > +Altera SOCFPGA SDRAM Controller > > + > > +Required properties: > > +- compatible : "altr,sdr-ctl", "syscon"; > > + Note that syscon is invoked for this device to support the FPGA > > + bridge driver, EDAC driver and other devices that share the > > + registers. > > This sounds like implementation specifics, which shouldn't be part of > the bindings. > > Sören > I see your point and will remove. Thanks! -- 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