On Tue, 2014-04-15 at 22:04 -0700, Brian Norris wrote: > Hi Sergei, > > On Tue, Dec 17, 2013 at 11:37:40PM +0400, Sergei Ianovich wrote: > > This provides an MTD device driver for 512kB of battery backed up SRAM > > on ICPDAS LP-8X4X programmable automation controllers. > > > > SRAM chip is connected via FPGA and is not accessible without a driver, > > unlike flash memory which is wired to CPU MMU. > > > > This SRAM becomes an excellent persisent storage of volatile process > > data like counter values and sensor statuses. Storing those data in > > flash or mmc card is not a viable solution. > > > > Signed-off-by: Sergei Ianovich <ynvich@xxxxxxxxx> > > --- > > v2..v3 > > * no changes (except number 08/16 -> 10/21) > > > > v0..v2 > > * use device tree > > * use devm helpers where possible > > What's the status of this series? Is the rest of this platform support > going in via other trees yet? I have a few trivial comments, but this > driver mostly looks good as-is. (...) > You'll need a (trivial) DT binding doc in > Documentation/devicetree/bindings/ to go with this. You might also need > to split the arch/arm/boot/dts/ stuff out to a separate patch, so the > MTD driver can go in the MTD tree separate from the ARM tree bits. (...) > With that, you have my: > > Reviewed-by: Brian Norris <computersforpeace@xxxxxxxxx> > > and I can merge this to the MTD tree if/when you'd like. Thanks for response, the rest of the series is stuck on a DMA driver with no development activity. I'll fix the comments, separate ARM staff and refile the patch. -- 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
