[PATCH 1/2] drivers: drm/msm/a6xx_catalog: Add A642L speedbin (0x81)

Danila Tikhonov posted 2 patches 1 year, 4 months ago
There is a newer version of this series
[PATCH 1/2] drivers: drm/msm/a6xx_catalog: Add A642L speedbin (0x81)
Posted by Danila Tikhonov 1 year, 4 months ago
From: Eugene Lepshy <fekz115@gmail.com>

According to downstream, A642L's speedbin is 129 and uses 4 as index

Signed-off-by: Eugene Lepshy <fekz115@gmail.com>
Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
---
 drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
index 68ba9aed5506e..99f0ee1a2edea 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
+++ b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
@@ -869,6 +869,7 @@ static const struct adreno_info a6xx_gpus[] = {
 		.speedbins = ADRENO_SPEEDBINS(
 			{ 0,   0 },
 			{ 117, 0 },
+			{ 129, 4 },
 			{ 172, 2 }, /* Called speedbin 1 downstream, but let's not break things! */
 			{ 190, 1 },
 		),
-- 
2.45.2
Re: [PATCH 1/2] drivers: drm/msm/a6xx_catalog: Add A642L speedbin (0x81)
Posted by Bjorn Andersson 1 year, 4 months ago
On Mon, Jul 22, 2024 at 09:43:13PM GMT, Danila Tikhonov wrote:
> From: Eugene Lepshy <fekz115@gmail.com>
> 

Please make sure the subject prefix matches other changes in the same
driver/files.

Regards,
Bjorn

> According to downstream, A642L's speedbin is 129 and uses 4 as index
> 
> Signed-off-by: Eugene Lepshy <fekz115@gmail.com>
> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
> ---
>  drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
> index 68ba9aed5506e..99f0ee1a2edea 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
> @@ -869,6 +869,7 @@ static const struct adreno_info a6xx_gpus[] = {
>  		.speedbins = ADRENO_SPEEDBINS(
>  			{ 0,   0 },
>  			{ 117, 0 },
> +			{ 129, 4 },
>  			{ 172, 2 }, /* Called speedbin 1 downstream, but let's not break things! */
>  			{ 190, 1 },
>  		),
> -- 
> 2.45.2
>
Re: [PATCH 1/2] drivers: drm/msm/a6xx_catalog: Add A642L speedbin (0x81)
Posted by Danila Tikhonov 1 year, 4 months ago
On 7/29/24 06:09, Bjorn Andersson wrote:
> On Mon, Jul 22, 2024 at 09:43:13PM GMT, Danila Tikhonov wrote:
>> From: Eugene Lepshy <fekz115@gmail.com>
>>
> Please make sure the subject prefix matches other changes in the same
> driver/files.
>
> Regards,
> Bjorn
>

Thanks for the advice

"drm/msm/a6xx: --//--" will be better?

Best wishes,
Danila
Re: [PATCH 1/2] drivers: drm/msm/a6xx_catalog: Add A642L speedbin (0x81)
Posted by Konrad Dybcio 1 year, 4 months ago
On 22.07.2024 8:43 PM, Danila Tikhonov wrote:
> From: Eugene Lepshy <fekz115@gmail.com>
> 
> According to downstream, A642L's speedbin is 129 and uses 4 as index
> 
> Signed-off-by: Eugene Lepshy <fekz115@gmail.com>
> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad