diff for duplicates of <20140815000403.GA7070@sonymobile.com> diff --git a/a/1.txt b/N1/1.txt index f444fbd..9e69153 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -12,7 +12,7 @@ On Tue 12 Aug 10:43 PDT 2014, Kumar Gala wrote: > > + Definition: two entries specifying the physical address and size of the > > + RPM's message ram > -> I’m a little confused here, by ‘two entries’ do you mean two values or really two regions? < A B > or < A B C D >? +> I?m a little confused here, by ?two entries? do you mean two values or really two regions? < A B > or < A B C D >? > Hmm, I agree with you, but I'm not sure what the definition of an entry is in @@ -41,7 +41,7 @@ Makes sense > > + > > += SUBDEVICES > -> These should not be children of RPM, but in an RPM container outside of the SoC node with some phandle reference (if needed) to the RPM. The reason is there isn’t really a technical means to translate from the SoC / MMIO bus address space to the RPM address space that these nodes live in. +> These should not be children of RPM, but in an RPM container outside of the SoC node with some phandle reference (if needed) to the RPM. The reason is there isn?t really a technical means to translate from the SoC / MMIO bus address space to the RPM address space that these nodes live in. > I don't agree with this; the regulators, clocks and bus-scalers aren't @@ -88,7 +88,7 @@ Yes, sorry about not improving this part. > > + > > +- qcom,force-mode-none: > -> I think I asked this last time I took a look at this, but can we have multiple force-mode’s set? If no, maybe this should be an enum instead. +> I think I asked this last time I took a look at this, but can we have multiple force-mode?s set? If no, maybe this should be an enum instead. > No, they are mutually exclusive. I just like the boolean representation @@ -114,7 +114,7 @@ override it in the very few places it needs to be anything else. > > +- qcom,force-mode-auto: > > + Usage: optional (only available for 8960/8064) > -> can we say only available for "qcom,rpm-msm8960”, "qcom,rpm-apq8064" +> can we say only available for "qcom,rpm-msm8960?, "qcom,rpm-apq8064" > Indeed. @@ -126,7 +126,7 @@ Indeed. > > +- qcom,force-mode-hpm: > > + Usage: optional (only available for 8960/8064) > -> can we say only available for "qcom,rpm-msm8960”, "qcom,rpm-apq8064" +> can we say only available for "qcom,rpm-msm8960?, "qcom,rpm-apq8064" > Indeed. @@ -138,7 +138,7 @@ Indeed. > > +- qcom,force-mode-bypass: (only for 8960/8064) > > + Usage: optional (only available for 8960/8064) > -> can we say only available for "qcom,rpm-msm8960”, "qcom,rpm-apq8064" +> can we say only available for "qcom,rpm-msm8960?, "qcom,rpm-apq8064" > Indeed. diff --git a/a/content_digest b/N1/content_digest index 451a127..2140772 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,22 +1,10 @@ "ref\01407796988-25872-1-git-send-email-bjorn.andersson@sonymobile.com\0" "ref\01407796988-25872-2-git-send-email-bjorn.andersson@sonymobile.com\0" "ref\0AF6050C7-EF40-4865-9E41-B730FEF59D6C@codeaurora.org\0" - "From\0Bjorn Andersson <bjorn.andersson@sonymobile.com>\0" - "Subject\0Re: [PATCH v5 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding\0" + "From\0bjorn.andersson@sonymobile.com (Bjorn Andersson)\0" + "Subject\0[PATCH v5 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding\0" "Date\0Thu, 14 Aug 2014 17:04:05 -0700\0" - "To\0Kumar Gala <galak@codeaurora.org>\0" - "Cc\0Rob Herring <robh+dt@kernel.org>" - Mark Brown <broonie@kernel.org> - Lee Jones <lee.jones@linaro.org> - Pawel Moll <pawel.moll@arm.com> - Andy Gross <agross@codeaurora.org> - Mark Rutland <mark.rutland@arm.com> - Kevin Hilman <khilman@linaro.org> - Josh Cartwright <joshc@codeaurora.org> - devicetree@vger.kernel.org <devicetree@vger.kernel.org> - linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> - linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org> - " linux-arm-msm@vger.kernel.org <linux-arm-msm@vger.kernel.org>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Tue 12 Aug 10:43 PDT 2014, Kumar Gala wrote:\n" @@ -33,7 +21,7 @@ "> > + Definition: two entries specifying the physical address and size of the\n" "> > + RPM's message ram\n" "> \n" - "> I\342\200\231m a little confused here, by \342\200\230two entries\342\200\231 do you mean two values or really two regions? < A B > or < A B C D >?\n" + "> I?m a little confused here, by ?two entries? do you mean two values or really two regions? < A B > or < A B C D >?\n" "> \n" "\n" "Hmm, I agree with you, but I'm not sure what the definition of an entry is in\n" @@ -62,7 +50,7 @@ "> > +\n" "> > += SUBDEVICES\n" "> \n" - "> These should not be children of RPM, but in an RPM container outside of the SoC node with some phandle reference (if needed) to the RPM. The reason is there isn\342\200\231t really a technical means to translate from the SoC / MMIO bus address space to the RPM address space that these nodes live in.\n" + "> These should not be children of RPM, but in an RPM container outside of the SoC node with some phandle reference (if needed) to the RPM. The reason is there isn?t really a technical means to translate from the SoC / MMIO bus address space to the RPM address space that these nodes live in.\n" "> \n" "\n" "I don't agree with this; the regulators, clocks and bus-scalers aren't\n" @@ -109,7 +97,7 @@ "> > +\n" "> > +- qcom,force-mode-none:\n" "> \n" - "> I think I asked this last time I took a look at this, but can we have multiple force-mode\342\200\231s set? If no, maybe this should be an enum instead.\n" + "> I think I asked this last time I took a look at this, but can we have multiple force-mode?s set? If no, maybe this should be an enum instead.\n" "> \n" "\n" "No, they are mutually exclusive. I just like the boolean representation\n" @@ -135,7 +123,7 @@ "> > +- qcom,force-mode-auto:\n" "> > + Usage: optional (only available for 8960/8064)\n" "> \n" - "> can we say only available for \"qcom,rpm-msm8960\342\200\235, \"qcom,rpm-apq8064\"\n" + "> can we say only available for \"qcom,rpm-msm8960?, \"qcom,rpm-apq8064\"\n" "> \n" "\n" "Indeed.\n" @@ -147,7 +135,7 @@ "> > +- qcom,force-mode-hpm:\n" "> > + Usage: optional (only available for 8960/8064)\n" "> \n" - "> can we say only available for \"qcom,rpm-msm8960\342\200\235, \"qcom,rpm-apq8064\"\n" + "> can we say only available for \"qcom,rpm-msm8960?, \"qcom,rpm-apq8064\"\n" "> \n" "\n" "Indeed.\n" @@ -159,7 +147,7 @@ "> > +- qcom,force-mode-bypass: (only for 8960/8064)\n" "> > + Usage: optional (only available for 8960/8064)\n" "> \n" - "> can we say only available for \"qcom,rpm-msm8960\342\200\235, \"qcom,rpm-apq8064\"\n" + "> can we say only available for \"qcom,rpm-msm8960?, \"qcom,rpm-apq8064\"\n" "> \n" "\n" "Indeed.\n" @@ -199,4 +187,4 @@ "Regards,\n" Bjorn -d457382e6760702004ca5adec894a36da346cf966a15d357f5f746c18d2b0d1c +ad4404cda0163fac8d19f42ade664d6a4f79888d56efeadccd6388c32075ca39
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.