All of lore.kernel.org
 help / color / mirror / Atom feed
From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/7] dt: describe base reset signal binding
Date: Wed, 16 Jan 2013 15:06:00 -0700	[thread overview]
Message-ID: <50F72448.8070807@wwwdotorg.org> (raw)
In-Reply-To: <1358352787-15441-2-git-send-email-p.zabel@pengutronix.de>

On 01/16/2013 09:13 AM, Philipp Zabel wrote:
> From: Stephen Warren <swarren@nvidia.com>
> 
> This binding is intended to represent the hardware reset signals present
> internally in most IC (SoC, FPGA, ...) designs.
> 
> Such a binding would allow the creation of a "reset subsystem", which
> could replace APIs such as the following Tegra-specific API:
> 
> void tegra_periph_reset_deassert(struct clk *c);
> void tegra_periph_reset_assert(struct clk *c);
> 
> (Note that at present, Tegra couples reset assertion with the clock for
> the affected peripheral module. However, reset and clocking are two
> separate, yet admittedly related, concepts).
> 
> Signed-off-by: Stephen Warren <swarren@nvidia.com>

You probably want to update the patch description; it was somewhat
appropriate when I sent the patch as an RFC to see what people thought
about the idea, but now you're actually providing the whole thing a more
specific description would be better.

Your S-o-b line should be present in all patches you send; add it after
mine. See Documentation/SubmittingPatches section "Sign your work" item
(c) (or perhaps (b) if you modified it).

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
Cc: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>,
	Fabio Estevam
	<fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	Mike Turquette
	<mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Sascha Hauer <s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org,
	Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH 1/7] dt: describe base reset signal binding
Date: Wed, 16 Jan 2013 15:06:00 -0700	[thread overview]
Message-ID: <50F72448.8070807@wwwdotorg.org> (raw)
In-Reply-To: <1358352787-15441-2-git-send-email-p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>

On 01/16/2013 09:13 AM, Philipp Zabel wrote:
> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> This binding is intended to represent the hardware reset signals present
> internally in most IC (SoC, FPGA, ...) designs.
> 
> Such a binding would allow the creation of a "reset subsystem", which
> could replace APIs such as the following Tegra-specific API:
> 
> void tegra_periph_reset_deassert(struct clk *c);
> void tegra_periph_reset_assert(struct clk *c);
> 
> (Note that at present, Tegra couples reset assertion with the clock for
> the affected peripheral module. However, reset and clocking are two
> separate, yet admittedly related, concepts).
> 
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

You probably want to update the patch description; it was somewhat
appropriate when I sent the patch as an RFC to see what people thought
about the idea, but now you're actually providing the whole thing a more
specific description would be better.

Your S-o-b line should be present in all patches you send; add it after
mine. See Documentation/SubmittingPatches section "Sign your work" item
(c) (or perhaps (b) if you modified it).

  reply	other threads:[~2013-01-16 22:06 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-16 16:13 [RFC PATCH 0/5] Reset controller API to reset IP modules on i.MX5 and i.MX6 Philipp Zabel
2013-01-16 16:13 ` Philipp Zabel
2013-01-16 16:13 ` [PATCH 1/7] dt: describe base reset signal binding Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-16 22:06   ` Stephen Warren [this message]
2013-01-16 22:06     ` Stephen Warren
2013-01-16 16:13 ` [PATCH 2/7] reset: Add reset controller API Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-16 20:15   ` Sascha Hauer
2013-01-16 20:15     ` Sascha Hauer
2013-01-16 22:15   ` Stephen Warren
2013-01-16 22:15     ` Stephen Warren
2013-01-17 10:45     ` Philipp Zabel
2013-01-17 10:45       ` Philipp Zabel
2013-01-17 16:55       ` Stephen Warren
2013-01-17 16:55         ` Stephen Warren
2013-01-17  5:16   ` Shawn Guo
2013-01-17  5:16     ` Shawn Guo
2013-01-16 16:13 ` [PATCH 3/7] ARM i.MX6q: Add GPU, VPU, IPU, and OpenVG resets to System Reset Controller (SRC) Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-17  5:31   ` Shawn Guo
2013-01-17  5:31     ` Shawn Guo
2013-01-18 19:57   ` Matt Sealey
2013-01-18 19:57     ` Matt Sealey
2013-01-21  9:52     ` Philipp Zabel
2013-01-21  9:52       ` Philipp Zabel
2013-01-21 17:47       ` Matt Sealey
2013-01-21 17:47         ` Matt Sealey
2013-01-22  7:50         ` Shawn Guo
2013-01-22  7:50           ` Shawn Guo
2013-01-16 16:13 ` [PATCH 4/7] ARM i.MX6q: Link system reset controller (SRC) to IPU in DT Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-17  5:34   ` Shawn Guo
2013-01-17  5:34     ` Shawn Guo
2013-01-16 16:13 ` [PATCH 5/7] staging: drm/imx: Use SRC to reset IPU Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-16 16:13 ` [PATCH 6/7] ARM i.MX5: Add System Reset Controller (SRC) support for i.MX51 and i.MX53 Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-17  6:12   ` Shawn Guo
2013-01-17  6:12     ` Shawn Guo
2013-01-17 10:45     ` Philipp Zabel
2013-01-17 10:45       ` Philipp Zabel
2013-01-16 16:13 ` [PATCH 7/7] ARM i.MX5: Add system reset controller (SRC) to i.MX51 and i.MX53 device tree Philipp Zabel
2013-01-16 16:13   ` Philipp Zabel
2013-01-16 22:19   ` Stephen Warren
2013-01-16 22:19     ` Stephen Warren
2013-01-17  6:37     ` Shawn Guo
2013-01-17  6:37       ` Shawn Guo
2013-01-17 10:45     ` Philipp Zabel
2013-01-17 10:45       ` Philipp Zabel
2013-01-16 18:46 ` [RFC PATCH 0/5] Reset controller API to reset IP modules on i.MX5 and i.MX6 Marek Vasut
2013-01-16 18:46   ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50F72448.8070807@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.