Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1316530

Re: [PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323 regulator

From Mark Brown <broonie@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323 regulator
Date 2016-01-25 12:40 +0100
Message-ID <qUNF1-3cn-23@gated-at.bofh.it> (permalink)
References <qUMSD-2CU-31@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Mon, Jan 25, 2016 at 11:40:04AM +0100, John Crispin wrote:
> Based on the MT6397 binding documentation.
> 
> Signed-off-by: John Crispin <blogic@openwrt.org>
> Cc: devicetree@vger.kernel.org

In reply to your previous submission I said:

| important.  Please also use subject lines matching the style for the
| subsystem (for patch 1).

Please don't ignore review comments, people are generally making them
for a reason and are likely to have the same concerns if issues remain
unaddressed.  Having to repeat the same comments can get repetitive and
make people question the value of time spent reviewing.  If you disagree
with the review comments that's fine but you need to reply and discuss
your concerns so that the reviewer can understand your decisions.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323 regulator John Crispin <blogic@openwrt.org> - 2016-01-25 11:50 +0100
  [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator John Crispin <blogic@openwrt.org> - 2016-01-25 11:50 +0100
    Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 13:20 +0100
      Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 13:40 +0100
      Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator Mark Brown <broonie@kernel.org> - 2016-01-25 13:40 +0100
        Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 14:20 +0100
          Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 14:20 +0100
            Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 14:30 +0100
              Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 14:50 +0100
                Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 15:10 +0100
                Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 15:10 +0100
    Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 13:20 +0100
  Re: [PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323  regulator Mark Brown <broonie@kernel.org> - 2016-01-25 12:40 +0100
    Re: [PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 13:10 +0100

csiph-web