Thread (19 messages) 19 messages, 6 authors, 2016-04-01

Re: [PATCH v4 1/2] watchdog: imx2_wdt: add external reset support via 'ext-reset-output' dt prop

From: Guenter Roeck <hidden>
Date: 2016-03-30 21:09:20
Also in: linux-arm-kernel, linux-watchdog, lkml

On Wed, Mar 30, 2016 at 09:22:33AM +0800, Shawn Guo wrote:
On Mon, Mar 28, 2016 at 04:19:15PM -0400, Akshay Bhat wrote:
quoted
Hi Shawn,

On 02/28/2016 08:56 AM, Fabio Estevam wrote:
quoted
Rob,

On Thu, Jan 28, 2016 at 6:28 PM, Akshay Bhat [off-list ref] wrote:
quoted
quoted
I have no objections against the idea and the code itself.
But as Guenter pointed out: it would be handy to get feedback from the
devicetree maintainers on the above discussion.

Kind regards,
Wim.
Any suggestions on whether a vendor specific prefix is necessary?
Any comments?
Is this something you can help with, since you are the iMX
architecture/devicetree maintainer? Appreciate any feedback.
FWIW,

Acked-by: Shawn Guo <redacted>

Guenter,

I think it's reasonable to pick up the patch, if we have it copied to DT
maintainers and on the list for a long period of time, and do not see
any objection from anyone.
The question was if the property name should be ext-reset-output or
fsl,ext-reset-output. In my opinion, it should be fsl,ext-reset-output
because it is not a generic property. Tim disagrees.

So we have two options: Change the property name to fsl,ext-reset-output,
which I would accept, or wait for a devicetree maintainer to make a decision.

Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help