diff for duplicates of <1501489667.6834.1.camel@baylibre.com> diff --git a/a/1.txt b/N1/1.txt index e235a1c..d4c3524 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -20,24 +20,24 @@ On Fri, 2017-07-28 at 23:13 +0200, Martin Blumenstingl wrote: > > Changes since v2 at [3]: > - move the reset line preprocessor defines into a separate file -> ? (affects patch #1 as this header file is part of the dt-binding) +> (affects patch #1 as this header file is part of the dt-binding) > - rename the reset line preprocessor macros from RESETID_ to -> ? CLKC_RESET_ to clearly indicate that these are provided by the -> ? clock controller (unlike the preprocessor macros for the reset -> ? lines in the standlone reset controller, which start with RESET_) +> CLKC_RESET_ to clearly indicate that these are provided by the +> clock controller (unlike the preprocessor macros for the reset +> lines in the standlone reset controller, which start with RESET_) > - updated patch #2 due to the changes mentioned above > - added Neil's Reviewed-by to all patches > > Changes since v1 at [0]: > - updated cover letter description as we are now registering more than -> ? four reset lines +> four reset lines > - split patch #1 into a dt-binding and clk driver patch > - slightly reworded the dt-binding documentation so it's now clear that -> ? the reset identifiers are preprocessor macros in -> ? dt-bindings/clock/meson8b-clkc.h (v1 of this series didn't have these -> ? macros at all) +> the reset identifiers are preprocessor macros in +> dt-bindings/clock/meson8b-clkc.h (v1 of this series didn't have these +> macros at all) > - patch #2 (previously part of patch #1) now registers all known reset -> ? lines (see [1] and [2] for the results of my detective work) +> lines (see [1] and [2] for the results of my detective work) > - patch #3 is untouched > > @@ -48,20 +48,20 @@ On Fri, 2017-07-28 at 23:13 +0200, Martin Blumenstingl wrote: > > > Martin Blumenstingl (3): -> ? dt-bindings: clock: meson8b: describe the embedded reset controller -> ? clk: meson: meson8b: register the built-in reset controller -> ? ARM: dts: meson: mark the clock controller also as reset controller +> dt-bindings: clock: meson8b: describe the embedded reset controller +> clk: meson: meson8b: register the built-in reset controller +> ARM: dts: meson: mark the clock controller also as reset controller > -> ?.../bindings/clock/amlogic,meson8b-clkc.txt????????|???9 +- -> ?arch/arm/boot/dts/meson8.dtsi??????????????????????|???1 + -> ?arch/arm/boot/dts/meson8b.dtsi?????????????????????|???1 + -> ?drivers/clk/meson/Kconfig??????????????????????????|???1 + -> ?drivers/clk/meson/meson8b.c????????????????????????| 159 +++++++++++++++++++- +> .../bindings/clock/amlogic,meson8b-clkc.txt | 9 +- +> arch/arm/boot/dts/meson8.dtsi | 1 + +> arch/arm/boot/dts/meson8b.dtsi | 1 + +> drivers/clk/meson/Kconfig | 1 + +> drivers/clk/meson/meson8b.c | 159 +++++++++++++++++++- > - -> ?drivers/clk/meson/meson8b.h????????????????????????|???9 +- -> ?.../dt-bindings/reset/amlogic,meson8b-clkc-reset.h |??27 ++++ -> ?7 files changed, 193 insertions(+), 14 deletions(-) -> ?create mode 100644 include/dt-bindings/reset/amlogic,meson8b-clkc-reset.h +> drivers/clk/meson/meson8b.h | 9 +- +> .../dt-bindings/reset/amlogic,meson8b-clkc-reset.h | 27 ++++ +> 7 files changed, 193 insertions(+), 14 deletions(-) +> create mode 100644 include/dt-bindings/reset/amlogic,meson8b-clkc-reset.h > Series looks good to me overall. diff --git a/a/content_digest b/N1/content_digest index dc4d3dd..cf3706a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,8 +1,19 @@ "ref\020170728211313.10603-1-martin.blumenstingl@googlemail.com\0" - "From\0jbrunet@baylibre.com (Jerome Brunet)\0" - "Subject\0[PATCH v3 0/3] add the reset controller to the Meson8b clkc\0" + "From\0Jerome Brunet <jbrunet@baylibre.com>\0" + "Subject\0Re: [PATCH v3 0/3] add the reset controller to the Meson8b clkc\0" "Date\0Mon, 31 Jul 2017 10:27:47 +0200\0" - "To\0linus-amlogic@lists.infradead.org\0" + "To\0Martin Blumenstingl <martin.blumenstingl@googlemail.com>" + linux-amlogic@lists.infradead.org + khilman@baylibre.com + carlo@caione.org + narmstrong@baylibre.com + linux-clk@vger.kernel.org + " robh+dt@kernel.org\0" + "Cc\0devicetree@vger.kernel.org" + linux@armlinux.org.uk + mark.rutland@arm.com + mturquette@baylibre.com + " sboyd@codeaurora.org\0" "\00:1\0" "b\0" "On Fri, 2017-07-28 at 23:13 +0200, Martin Blumenstingl wrote:\n" @@ -27,24 +38,24 @@ "> \n" "> Changes since v2 at [3]:\n" "> - move the reset line preprocessor defines into a separate file\n" - "> ? (affects patch #1 as this header file is part of the dt-binding)\n" + "> \302\240 (affects patch #1 as this header file is part of the dt-binding)\n" "> - rename the reset line preprocessor macros from RESETID_ to\n" - "> ? CLKC_RESET_ to clearly indicate that these are provided by the\n" - "> ? clock controller (unlike the preprocessor macros for the reset\n" - "> ? lines in the standlone reset controller, which start with RESET_)\n" + "> \302\240 CLKC_RESET_ to clearly indicate that these are provided by the\n" + "> \302\240 clock controller (unlike the preprocessor macros for the reset\n" + "> \302\240 lines in the standlone reset controller, which start with RESET_)\n" "> - updated patch #2 due to the changes mentioned above\n" "> - added Neil's Reviewed-by to all patches\n" "> \n" "> Changes since v1 at [0]:\n" "> - updated cover letter description as we are now registering more than\n" - "> ? four reset lines\n" + "> \302\240 four reset lines\n" "> - split patch #1 into a dt-binding and clk driver patch\n" "> - slightly reworded the dt-binding documentation so it's now clear that\n" - "> ? the reset identifiers are preprocessor macros in\n" - "> ? dt-bindings/clock/meson8b-clkc.h (v1 of this series didn't have these\n" - "> ? macros at all)\n" + "> \302\240 the reset identifiers are preprocessor macros in\n" + "> \302\240 dt-bindings/clock/meson8b-clkc.h (v1 of this series didn't have these\n" + "> \302\240 macros at all)\n" "> - patch #2 (previously part of patch #1) now registers all known reset\n" - "> ? lines (see [1] and [2] for the results of my detective work)\n" + "> \302\240 lines (see [1] and [2] for the results of my detective work)\n" "> - patch #3 is untouched\n" "> \n" "> \n" @@ -55,20 +66,20 @@ "> \n" "> \n" "> Martin Blumenstingl (3):\n" - "> ? dt-bindings: clock: meson8b: describe the embedded reset controller\n" - "> ? clk: meson: meson8b: register the built-in reset controller\n" - "> ? ARM: dts: meson: mark the clock controller also as reset controller\n" + "> \302\240 dt-bindings: clock: meson8b: describe the embedded reset controller\n" + "> \302\240 clk: meson: meson8b: register the built-in reset controller\n" + "> \302\240 ARM: dts: meson: mark the clock controller also as reset controller\n" "> \n" - "> ?.../bindings/clock/amlogic,meson8b-clkc.txt????????|???9 +-\n" - "> ?arch/arm/boot/dts/meson8.dtsi??????????????????????|???1 +\n" - "> ?arch/arm/boot/dts/meson8b.dtsi?????????????????????|???1 +\n" - "> ?drivers/clk/meson/Kconfig??????????????????????????|???1 +\n" - "> ?drivers/clk/meson/meson8b.c????????????????????????| 159 +++++++++++++++++++-\n" + "> \302\240.../bindings/clock/amlogic,meson8b-clkc.txt\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\302\240\302\240\302\2409 +-\n" + "> \302\240arch/arm/boot/dts/meson8.dtsi\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\302\240\302\240\302\2401 +\n" + "> \302\240arch/arm/boot/dts/meson8b.dtsi\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\302\240\302\240\302\2401 +\n" + "> \302\240drivers/clk/meson/Kconfig\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\302\240\302\240\302\2401 +\n" + "> \302\240drivers/clk/meson/meson8b.c\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 159 +++++++++++++++++++-\n" "> -\n" - "> ?drivers/clk/meson/meson8b.h????????????????????????|???9 +-\n" - "> ?.../dt-bindings/reset/amlogic,meson8b-clkc-reset.h |??27 ++++\n" - "> ?7 files changed, 193 insertions(+), 14 deletions(-)\n" - "> ?create mode 100644 include/dt-bindings/reset/amlogic,meson8b-clkc-reset.h\n" + "> \302\240drivers/clk/meson/meson8b.h\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\302\240\302\240\302\2409 +-\n" + "> \302\240.../dt-bindings/reset/amlogic,meson8b-clkc-reset.h |\302\240\302\24027 ++++\n" + "> \302\2407 files changed, 193 insertions(+), 14 deletions(-)\n" + "> \302\240create mode 100644 include/dt-bindings/reset/amlogic,meson8b-clkc-reset.h\n" "> \n" "\n" "Series looks good to me overall.\n" @@ -78,4 +89,4 @@ "Just one thing, which is not an issue really, I wonder if the dt-bindings\n" documentation and the bindings them-self should be in separate patches ? -a988153faa54969d6095bbf31b92d4d1fd13090434f5e34f7c5b00c5bd156c7e +c1827e7a37f8e11c73da53c971d3c391164ab7772d6bef4516fc4248aac365a1
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.