Linux-mediatek Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Henry Chen <HenryC.Chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
To: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org,
	Liam Girdwood <lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	Matthias Brugger
	<matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	eddie.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH 1/2] dt-bindings: Add document for MT6311 regulator
Date: Tue, 21 Jul 2015 17:46:53 +0800	[thread overview]
Message-ID: <1437472013.30329.53.camel@mtksdaap41> (raw)
In-Reply-To: <20150720172828.GG11162-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>

On Mon, 2015-07-20 at 18:28 +0100, Mark Brown wrote:
> On Mon, Jul 20, 2015 at 03:36:32PM +0800, Henry Chen wrote:
> 
> > +Required properties:
> > +- compatible: "mediatek,mt6311-regulator"
> > +- mt6311regulator: List of regulators provided by this controller. It is named
> > +  to VDVFS and VBIASN.
> 
> Why this weird property name?  It looks like it might just be a typo and
> should just be "regulator"?
Yes, it should be "regulators".

> 
> Please use subject lines reflecting the style for the subsystem, it
> helps people find patches that are relevant to them.

Ok.

Thanks,
Henry

  parent reply	other threads:[~2015-07-21  9:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-20  7:36 regulator: MT6311: Add support for MT6311 regulator Henry Chen
2015-07-20  7:36 ` [PATCH 1/2] dt-bindings: Add document " Henry Chen
2015-07-20 17:28   ` Mark Brown
     [not found]     ` <20150720172828.GG11162-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2015-07-21  9:46       ` Henry Chen [this message]
2015-07-20  7:36 ` [PATCH 2/2] regulator: MT6311: Add support " Henry Chen
2015-07-20 17:31   ` Mark Brown
2015-07-21  9:52     ` Henry Chen

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=1437472013.30329.53.camel@mtksdaap41 \
    --to=henryc.chen-nus5lvnupcjwk0htik3j/w@public.gmane.org \
    --cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=eddie.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
    --cc=kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
    --cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox