[RFC PATCH 1/5] ARM i.MX6q: Add GPU, VPU, IPU, and OpenVG resets to system reset controller

On 01/10/2013 06:51 AM, Philipp Zabel wrote:

Hi Stephen,

Am Mittwoch, den 09.01.2013, 11:15 -0700 schrieb Stephen Warren:

quoted

On 01/09/2013 10:17 AM, Philipp Zabel wrote:

quoted

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 .../devicetree/bindings/reset/fsl,imx-src.txt      |   45 ++++++++++++++++++++

I proposed something very similar a while back; it may be useful to look
at the previous discussion there:

http://www.spinics.net/lists/arm-kernel/msg202451.html

Thanks, I had started playing around with some "reset subsystem"
patches, but so far shied away from what feels like a whole lot of code
for little effect. I came to the same conclusion that replacing gpio
resets would probably be overkill, but I was only peripherally aware of
the tegra reset controller.

So a simple common struct and some oftree parsing code in drivers/reset
that can replace both the proposed imx patch and what
tegra_periph_reset_* do would be considered useful?

Yes, I didn't really imagine the "reset subsystem" would do more than
parsing the DT property that represented the reset and look up the
registered reset controller and reset ID. Plus, provide some central
function to pass the parsed reset ID to the reset controller driver. Of
course, if it expanded to cover non-DT cases it might get more complex,
but that can always be added later.