diff for duplicates of <1471788247.2188.17.camel@intel.com> diff --git a/a/1.txt b/N1/1.txt index f60b084..52f92f2 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,11 +1,11 @@ -On 日, 2016-08-21 at 21:37 +0800, Zhang Rui wrote: -> On 五, 2016-08-19 at 11:59 -0700, Eric Anholt wrote: +On ?, 2016-08-21 at 21:37 +0800, Zhang Rui wrote: +> On ?, 2016-08-19 at 11:59 -0700, Eric Anholt wrote: > > > > Zhang Rui <rui.zhang@intel.com> writes: > > > > > > > > -> > > On 四, 2016-08-18 at 11:39 -0700, Eric Anholt wrote: +> > > On ?, 2016-08-18 at 11:39 -0700, Eric Anholt wrote: > > > > > > > > > > > > Eric Anholt <eric@anholt.net> writes: @@ -13,7 +13,7 @@ On 日, 2016-08-21 at 21:37 +0800, Zhang Rui wrote: > > > > > > > > > > > > > > > -> > > > > kernel@martin.sperl.org writes: +> > > > > kernel at martin.sperl.org writes: > > > > > > > > > > > > > > > > > @@ -28,14 +28,14 @@ On 日, 2016-08-21 at 21:37 +0800, Zhang Rui wrote: > > > > > > Signed-off-by: Martin Sperl <kernel@martin.sperl.org> > > > > > > > > > > > > ChangeLog: -> > > > > > V1 -> V2: added specific settings depending on +> > > > > > ?V1 -> V2: added specific settings depending on > > > > > > compatiblity -> > > > > > added trip point based on register -> > > > > > setting up ctrl-register if HW is not enabled by +> > > > > > ???added trip point based on register +> > > > > > ???setting up ctrl-register if HW is not enabled by > > > > > > firmware -> > > > > > as per recommendation of Eric (untested) -> > > > > > check that clock frequency is in range -> > > > > > (1.9 - 5MHz - as per comment in clk-bcm2835.c) +> > > > > > ?????as per recommendation of Eric (untested) +> > > > > > ???check that clock frequency is in range +> > > > > > ?????(1.9 - 5MHz - as per comment in clk-bcm2835.c) > > > > > > --- > > > > > > > > > > > > diff --git a/drivers/thermal/bcm/Makefile @@ -65,36 +65,36 @@ On 日, 2016-08-21 at 21:37 +0800, Zhang Rui wrote: > > > > > > + rate = clk_get_rate(data->clk); > > > > > > + if ((rate < 1920000) || (rate > 5000000)) { > > > > > > + dev_warn(&pdev->dev, -> > > > > > + "Clock %pCn is running at %pCr +> > > > > > + ?"Clock %pCn is running at %pCr > > > > > > Hz, > > > > > > which is outside the recommended range of 1.9 to 5.0 > > > > > > MHz\n", -> > > > > > + data->clk, data->clk); +> > > > > > + ?data->clk, data->clk); > > > > > > + } > > > > > > + > > > > > > + /* register it */ > > > > > > + tz = > > > > > > thermal_zone_device_register("bcm2835_thermal", -> > > > > > + 1, 0, data, -> > > > > > + &bcm2835_thermal +> > > > > > + ??1, 0, data, +> > > > > > + ??&bcm2835_thermal > > > > > > _o > > > > > > ps, -> > > > > > + NULL, 0, 0); +> > > > > > + ??NULL, 0, 0); > > > > > I notice that the polling_delay is set to 0, but we're not > > > > > using -> > > > > interrupts to trigger the trip. Is it valid to expose a trip +> > > > > interrupts to trigger the trip.??Is it valid to expose a trip > > > > > without a -> > > > > polling_delay or interrupts? Should passive_delay be set as +> > > > > polling_delay or interrupts???Should passive_delay be set as > > > > > well? > > > > > -> > > > > This is up to the thermal maintainers. As far as I'm +> > > > > This is up to the thermal maintainers.??As far as I'm > > > > > concerned, > > > > > it's: > > > > > > > > > > Acked-by: Eric Anholt <eric@anholt.net> > > > > > > > > > > One it lands I'll pull the defconfig and DT bits. -> > > > Ping on this question to thermal maintainers. I'd still love +> > > > Ping on this question to thermal maintainers.??I'd still love > > > > to > > > > see > > > > this driver land. @@ -105,7 +105,7 @@ On 日, 2016-08-21 at 21:37 +0800, Zhang Rui wrote: > > > interface, > > > but > > > kernel thermal control is a no-op here, right? -> > Yeah. It seems useful to be able to get the information. Once we +> > Yeah.??It seems useful to be able to get the information.??Once we > > land > > this, I hope someone can add interrupt support so that there are > > actual @@ -116,12 +116,12 @@ BTW, can you add a "TODO" comment for this gap? thanks, rui > Another question, why you need a separate directory for a single -> file, bcm2835_thermal.c? +> file,?bcm2835_thermal.c? > > thanks, > rui > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" > in -> the body of a message to majordomo@vger.kernel.org -> More majordomo info at http://vger.kernel.org/majordomo-info.html +> the body of a message to majordomo at vger.kernel.org +> More majordomo info at??http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 13c3ba6..f9e87ba 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -5,32 +5,20 @@ "ref\01471591694.2691.39.camel@intel.com\0" "ref\087inuwd1p4.fsf@eliezer.anholt.net\0" "ref\01471786649.2188.4.camel@intel.com\0" - "From\0Zhang Rui <rui.zhang@intel.com>\0" - "Subject\0Re: [PATCH V3 2/5] thermal: bcm2835: add thermal driver for bcm2835 soc\0" + "From\0rui.zhang@intel.com (Zhang Rui)\0" + "Subject\0[PATCH V3 2/5] thermal: bcm2835: add thermal driver for bcm2835 soc\0" "Date\0Sun, 21 Aug 2016 22:04:07 +0800\0" - "To\0Eric Anholt <eric@anholt.net>" - kernel@martin.sperl.org - Eduardo Valentin <edubezval@gmail.com> - Rob Herring <robh+dt@kernel.org> - Pawel Moll <pawel.moll@arm.com> - Mark Rutland <mark.rutland@arm.com> - Stephen Warren <swarren@wwwdotorg.org> - Lee Jones <lee@kernel.org> - Russell King <linux@arm.linux.org.uk> - linux-pm@vger.kernel.org - devicetree@vger.kernel.org - linux-rpi-kernel@lists.infradead.org - " linux-arm-kernel@lists.infradead.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" - "On \346\227\245, 2016-08-21 at 21:37 +0800, Zhang Rui wrote:\n" - "> On \344\272\224, 2016-08-19 at 11:59 -0700, Eric Anholt wrote:\n" + "On ?, 2016-08-21 at 21:37 +0800, Zhang Rui wrote:\n" + "> On ?, 2016-08-19 at 11:59 -0700, Eric Anholt wrote:\n" "> > \n" "> > Zhang Rui <rui.zhang@intel.com> writes:\n" "> > \n" "> > > \n" "> > > \n" - "> > > On \345\233\233, 2016-08-18 at 11:39 -0700, Eric Anholt wrote:\n" + "> > > On ?, 2016-08-18 at 11:39 -0700, Eric Anholt wrote:\n" "> > > > \n" "> > > > \n" "> > > > Eric Anholt <eric@anholt.net> writes:\n" @@ -38,7 +26,7 @@ "> > > > > \n" "> > > > > \n" "> > > > > \n" - "> > > > > kernel@martin.sperl.org writes:\n" + "> > > > > kernel at martin.sperl.org writes:\n" "> > > > > \n" "> > > > > > \n" "> > > > > > \n" @@ -53,14 +41,14 @@ "> > > > > > Signed-off-by: Martin Sperl <kernel@martin.sperl.org>\n" "> > > > > > \n" "> > > > > > ChangeLog:\n" - "> > > > > > \302\240V1 -> V2: added specific settings depending on\n" + "> > > > > > ?V1 -> V2: added specific settings depending on\n" "> > > > > > compatiblity\n" - "> > > > > > \t\302\240\302\240\302\240added trip point based on register\n" - "> > > > > > \t\302\240\302\240\302\240setting up ctrl-register if HW is not enabled by\n" + "> > > > > > \t???added trip point based on register\n" + "> > > > > > \t???setting up ctrl-register if HW is not enabled by\n" "> > > > > > firmware\n" - "> > > > > > \t\302\240\302\240\302\240\302\240\302\240as per recommendation of Eric (untested)\n" - "> > > > > > \t\302\240\302\240\302\240check that clock frequency is in range\n" - "> > > > > > \t\302\240\302\240\302\240\302\240\302\240(1.9 - 5MHz - as per comment in clk-bcm2835.c)\n" + "> > > > > > \t?????as per recommendation of Eric (untested)\n" + "> > > > > > \t???check that clock frequency is in range\n" + "> > > > > > \t?????(1.9 - 5MHz - as per comment in clk-bcm2835.c)\n" "> > > > > > ---\n" "> > > > > > \n" "> > > > > > diff --git a/drivers/thermal/bcm/Makefile\n" @@ -90,36 +78,36 @@ "> > > > > > +\trate = clk_get_rate(data->clk);\n" "> > > > > > +\tif ((rate < 1920000) || (rate > 5000000)) {\n" "> > > > > > +\t\tdev_warn(&pdev->dev,\n" - "> > > > > > +\t\t\t\302\240\"Clock %pCn is running at %pCr\n" + "> > > > > > +\t\t\t?\"Clock %pCn is running at %pCr\n" "> > > > > > Hz,\n" "> > > > > > which is outside the recommended range of 1.9 to 5.0\n" "> > > > > > MHz\\n\",\n" - "> > > > > > +\t\t\t\302\240data->clk, data->clk);\n" + "> > > > > > +\t\t\t?data->clk, data->clk);\n" "> > > > > > +\t}\n" "> > > > > > +\n" "> > > > > > +\t/* register it */\n" "> > > > > > +\ttz =\n" "> > > > > > thermal_zone_device_register(\"bcm2835_thermal\",\n" - "> > > > > > +\t\t\t\t\t\302\240\302\2401, 0, data,\n" - "> > > > > > +\t\t\t\t\t\302\240\302\240&bcm2835_thermal\n" + "> > > > > > +\t\t\t\t\t??1, 0, data,\n" + "> > > > > > +\t\t\t\t\t??&bcm2835_thermal\n" "> > > > > > _o\n" "> > > > > > ps,\n" - "> > > > > > +\t\t\t\t\t\302\240\302\240NULL, 0, 0);\n" + "> > > > > > +\t\t\t\t\t??NULL, 0, 0);\n" "> > > > > I notice that the polling_delay is set to 0, but we're not\n" "> > > > > using\n" - "> > > > > interrupts to trigger the trip.\302\240\302\240Is it valid to expose a trip\n" + "> > > > > interrupts to trigger the trip.??Is it valid to expose a trip\n" "> > > > > without a\n" - "> > > > > polling_delay or interrupts?\302\240\302\240Should passive_delay be set as\n" + "> > > > > polling_delay or interrupts???Should passive_delay be set as\n" "> > > > > well?\n" "> > > > > \n" - "> > > > > This is up to the thermal maintainers.\302\240\302\240As far as I'm\n" + "> > > > > This is up to the thermal maintainers.??As far as I'm\n" "> > > > > concerned,\n" "> > > > > it's:\n" "> > > > > \n" "> > > > > Acked-by: Eric Anholt <eric@anholt.net>\n" "> > > > > \n" "> > > > > One it lands I'll pull the defconfig and DT bits.\n" - "> > > > Ping on this question to thermal maintainers.\302\240\302\240I'd still love\n" + "> > > > Ping on this question to thermal maintainers.??I'd still love\n" "> > > > to\n" "> > > > see\n" "> > > > this driver land.\n" @@ -130,7 +118,7 @@ "> > > interface,\n" "> > > but\n" "> > > kernel thermal control is a no-op here, right?\n" - "> > Yeah.\302\240\302\240It seems useful to be able to get the information.\302\240\302\240Once we\n" + "> > Yeah.??It seems useful to be able to get the information.??Once we\n" "> > land\n" "> > this, I hope someone can add interrupt support so that there are\n" "> > actual\n" @@ -141,14 +129,14 @@ "thanks,\n" "rui\n" "> Another question, why you need a separate directory for a single\n" - "> file,\302\240bcm2835_thermal.c?\n" + "> file,?bcm2835_thermal.c?\n" "> \n" "> thanks,\n" "> rui\n" "> --\n" "> To unsubscribe from this list: send the line \"unsubscribe linux-pm\"\n" "> in\n" - "> the body of a message to majordomo@vger.kernel.org\n" - "> More majordomo info at\302\240\302\240http://vger.kernel.org/majordomo-info.html" + "> the body of a message to majordomo at vger.kernel.org\n" + > More majordomo info at??http://vger.kernel.org/majordomo-info.html -a7532b8e3322a0d55c24f23986ab9b7612da639e80f6e6d978767cfddc863c7f +4e8135dbd7494735fb91a07db7ffa833aaa79344ac8f5e4656f1bfa436e9b078
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.