Hi Yue,
On 26/08/2020 04:13, Yue Hu wrote: > On Mon, 24 Aug 2020 12:40:35 +0200 > Daniel Lezcano <daniel.lezc...@linaro.org> wrote: > >> On 18/08/2020 08:30, Yue Hu wrote: >>> From: Yue Hu <huy...@yulong.com> >>> >>> We observed warning about kzalloc() when register thermal cooling device >>> in backlight_device_register(). backlight display can be a cooling device >>> since reducing screen brightness will can help reduce temperature. >>> >>> However, ->get_max_state of backlight will assign max brightness of 1024 >>> to states. The memory size can be getting 1MB+ due to states * states. >> >> What are the benefits of a 1024 states cooling device ? Is the >> difference noticeable with a such small step ? > > Okay, this issue is happened under MSM/Android platform. QCOM spmi wled driver > will define the max brightness. We needs to fix the issue to get thermal > statistics. Let me rephrase my questions: Don't you think there is something wrong in creating a 1024 x 1024 matrix to show transitions ? What is the benefit of such stats ? What is the benefit of having a 1024 states cooling device ? -- <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
