mbox series

[0/2] Add msm8939 tsens support

Message ID 20200629144926.665-1-shawn.guo@linaro.org (mailing list archive)
Headers show
Series Add msm8939 tsens support | expand

Message

Shawn Guo June 29, 2020, 2:49 p.m. UTC 
It's my version of msm8939 tsens series that supersides Konrad's [1].
The bindings one is taken from Konrad though.

Shawn

[1] https://lkml.org/lkml/2020/5/1/1213

Konrad Dybcio (1):
  dt-bindings: tsens: qcom: Document MSM8939 compatible

Shawn Guo (1):
  thermal: qcom: tsens-v0_1: Add support for MSM8939

 .../bindings/thermal/qcom-tsens.yaml          |   1 +
 drivers/thermal/qcom/tsens-v0_1.c             | 144 +++++++++++++++++-
 drivers/thermal/qcom/tsens.c                  |   3 +
 drivers/thermal/qcom/tsens.h                  |   2 +-
 4 files changed, 148 insertions(+), 2 deletions(-)

Comments

Daniel Lezcano July 21, 2020, 1:29 p.m. UTC | #1 
On 29/06/2020 16:49, Shawn Guo wrote:
> It's my version of msm8939 tsens series that supersides Konrad's [1].
> The bindings one is taken from Konrad though.

I would like to see Konrad's ack.

Thanks

  -- Daniel
Konrad Dybcio July 25, 2020, 12:25 p.m. UTC | #2 
Tested-by: Konrad Dybcio <konradybcio@gmail.com> /* on Asus Z00T smartphone */
Acked-by: Konrad Dybcio <konradybcio@gmail.com>

Sorry for being (very) late with that, but I've been doing some
630/36/60 stuff. In case anyone's interested in the progress, I'll
leave it there. [1]


[1] https://github.com/konradybcio/linux/commits/ninges_labs

Regards
Konrad
Daniel Lezcano July 27, 2020, 8:34 a.m. UTC | #3 
On 25/07/2020 14:25, Konrad Dybcio wrote:
> Tested-by: Konrad Dybcio <konradybcio@gmail.com> /* on Asus Z00T smartphone */
> Acked-by: Konrad Dybcio <konradybcio@gmail.com>
> 
> Sorry for being (very) late with that, but I've been doing some
> 630/36/60 stuff. In case anyone's interested in the progress, I'll
> leave it there. [1]

Applied, thanks
Daniel Lezcano July 27, 2020, 8:44 a.m. UTC | #4 
On 25/07/2020 14:25, Konrad Dybcio wrote:
> Tested-by: Konrad Dybcio <konradybcio@gmail.com> /* on Asus Z00T smartphone */
> Acked-by: Konrad Dybcio <konradybcio@gmail.com>
> 
> Sorry for being (very) late with that, but I've been doing some
> 630/36/60 stuff. In case anyone's interested in the progress, I'll
> leave it there. [1]

In the future, when resending the same patches, please add a new
revision and on the same mailing lists.

Otherwise when using patchwork, it can be source of confusion.

eg.

1. With default project 'lkml':

pwclient list "thermal: qcom: tsens-v0_1: Add support for MSM8939"

1234899 New          [1/2] thermal: qcom: tsens-v0_1: Add support for
MSM8939

2. With 'linux-pm' project:

pwclient list "thermal: qcom: tsens-v0_1: Add support for MSM8939" -p
linux-pm

11523211 New          [1/2] thermal: qcom: tsens-v0_1: Add support for
MSM8939
11631843 New          [2/2] thermal: qcom: tsens-v0_1: Add support for
MSM8939
Shawn Guo July 27, 2020, 8:48 a.m. UTC | #5 
On Mon, Jul 27, 2020 at 4:44 PM Daniel Lezcano
<daniel.lezcano@linaro.org> wrote:
>
> On 25/07/2020 14:25, Konrad Dybcio wrote:
> > Tested-by: Konrad Dybcio <konradybcio@gmail.com> /* on Asus Z00T smartphone */
> > Acked-by: Konrad Dybcio <konradybcio@gmail.com>
> >
> > Sorry for being (very) late with that, but I've been doing some
> > 630/36/60 stuff. In case anyone's interested in the progress, I'll
> > leave it there. [1]
>
> In the future, when resending the same patches, please add a new
> revision and on the same mailing lists.

Noted.  Thanks, Daniel.

Shawn