[PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands

Mason Chang posted 3 patches 6 months, 3 weeks ago
drivers/thermal/mediatek/lvts_thermal.c | 74 ++++++++++++++++++++-----
1 file changed, 61 insertions(+), 13 deletions(-)
[PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
Posted by Mason Chang 6 months, 3 weeks ago
Add the LVTS commands for Mediatek Filogic 880/MT7988.

This series fixes severely abnormal and inaccurate LVTS temperature
readings when using the default commands.

Signed-off-by: Mason Chang <mason-cw.chang@mediatek.com>

Mason Chang (3):
  thermal/drivers/mediatek/lvts_thermal: change lvts commands array to
    static const
  thermal/drivers/mediatek/lvts_thermal: add lvts commands and their
    sizes to driver data
  thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands

 drivers/thermal/mediatek/lvts_thermal.c | 74 ++++++++++++++++++++-----
 1 file changed, 61 insertions(+), 13 deletions(-)

-- 
2.45.2
Re: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
Posted by Daniel Lezcano 5 months ago
On Mon, May 26, 2025 at 06:26:56PM +0800, Mason Chang wrote:
> Add the LVTS commands for Mediatek Filogic 880/MT7988.
> 
> This series fixes severely abnormal and inaccurate LVTS temperature
> readings when using the default commands.
> 
> Signed-off-by: Mason Chang <mason-cw.chang@mediatek.com>
> 
> Mason Chang (3):
>   thermal/drivers/mediatek/lvts_thermal: change lvts commands array to
>     static const
>   thermal/drivers/mediatek/lvts_thermal: add lvts commands and their
>     sizes to driver data
>   thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
> 
>  drivers/thermal/mediatek/lvts_thermal.c | 74 ++++++++++++++++++++-----
>  1 file changed, 61 insertions(+), 13 deletions(-)

Changes applied, thanks

-- 

 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
Aw: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
Posted by Frank Wunderlich 6 months ago
> Gesendet: Montag, 26. Mai 2025 um 12:26
> Von: "Mason Chang" <mason-cw.chang@mediatek.com>
> Betreff: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
>
> Add the LVTS commands for Mediatek Filogic 880/MT7988.
> 
> This series fixes severely abnormal and inaccurate LVTS temperature
> readings when using the default commands.
> 
> Signed-off-by: Mason Chang <mason-cw.chang@mediatek.com>

Hi

just a friedly reminder for maintainers, is the patch series ok or does it need any changes?

regards Frank
Re: Aw: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
Posted by Mason Chang 5 months, 2 weeks ago
On Mon, 2025-06-16 at 07:01 +0000, Frank Wunderlich wrote:
> 
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> > Gesendet: Montag, 26. Mai 2025 um 12:26
> > Von: "Mason Chang" <mason-cw.chang@mediatek.com>
> > Betreff: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add
> > mt7988 lvts commands
> > 
> > Add the LVTS commands for Mediatek Filogic 880/MT7988.
> > 
> > This series fixes severely abnormal and inaccurate LVTS temperature
> > readings when using the default commands.
> > 
> > Signed-off-by: Mason Chang <mason-cw.chang@mediatek.com>
> 
> Hi
> 
> just a friedly reminder for maintainers, is the patch series ok or
> does it need any changes?
> 
> regards Frank

Hi all,

I submitted the patch a while ago, but haven't received any response
from the maintainer yet. I would really appreciate it if any maintainer
could take a look or provide some comments.

If there is a more appropriate maintainer or reviewer for this patch,
please kindly let me know or help forward it.

Thank you very much for your time and help!

Sincerely,
Mason
Re: Aw: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add mt7988 lvts commands
Posted by Frank Wunderlich 5 months, 1 week ago
Hi,

i finally found someone for testing this as it only affects ~2% the mt7988 users:

https://github.com/frank-w/BPI-Router-Images/issues/16#issuecomment-3061976373

so can you please review and add it to mainline?

regards Frank


Am 8. Juli 2025 um 13:56 schrieb "Mason Chang" <mason-cw.chang@mediatek.com>:

> 
> On Mon, 2025-06-16 at 07:01 +0000, Frank Wunderlich wrote:
> 
> > 
> > External email : Please do not click links or open attachments until
> > 
> >  you have verified the sender or the content.
> > 
> >  
> > 
> >  
> > 
> >  Gesendet: Montag, 26. Mai 2025 um 12:26
> > 
> >  Von: "Mason Chang" <mason-cw.chang@mediatek.com>
> > 
> >  Betreff: [PATCH 0/3] thermal/drivers/mediatek/lvts_thermal: add
> > 
> >  mt7988 lvts commands
> > 
> >  
> > 
> >  Add the LVTS commands for Mediatek Filogic 880/MT7988.
> > 
> >  
> > 
> >  This series fixes severely abnormal and inaccurate LVTS temperature
> > 
> >  readings when using the default commands.
> > 
> >  
> > 
> >  Signed-off-by: Mason Chang <mason-cw.chang@mediatek.com>
> > 
> >  
> > 
> >  Hi
> > 
> >  
> > 
> >  just a friedly reminder for maintainers, is the patch series ok or
> > 
> >  does it need any changes?
> > 
> >  
> > 
> >  regards Frank
> > 
> 
> Hi all,
> 
> I submitted the patch a while ago, but haven't received any response
> 
> from the maintainer yet. I would really appreciate it if any maintainer
> 
> could take a look or provide some comments.
> 
> If there is a more appropriate maintainer or reviewer for this patch,
> 
> please kindly let me know or help forward it.
> 
> Thank you very much for your time and help!
> 
> Sincerely,
> 
> Mason
>