Re: [PATCH v3] arm64: defconfig: Enable Panfrost and Lima drivers

[Krzysztof Kozlowski]

On Mon, 24 Jun 2019 at 01:36, Chanwoo Choi <cw00.choi@xxxxxxxxxxx> wrote:
>
> On 19. 6. 24. ìì 4:16, Krzysztof Kozlowski wrote:
> > On Sun, 23 Jun 2019 at 06:31, Chanwoo Choi <cwchoi00@xxxxxxxxx> wrote:
> >>
> >> Hi Krzysztof,
> >>
> >> 2019ë 6ì 23ì (ì) ìì 4:20, Krzysztof Kozlowski <krzk@xxxxxxxxxx>ëì ìì:
> >>>
> >>> Enable support for Mali GPU with Panfrost and Lima drivers for:
> >>> 1. Samsung Exynos5433 and Exynos7 (having Mali T760),
> >>> 2. Allwiner A64 and H5 (Mali 400/450).
> >>>
> >>> Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> >>>
> >>> ---
> >>>
> >>> Changes since v1:
> >>> 1. Enable Lima driver
> >>> ---
> >>>  arch/arm64/configs/defconfig | 3 ++-
> >>>  1 file changed, 2 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> >>> index fbbc065415d4..3d31611368af 100644
> >>> --- a/arch/arm64/configs/defconfig
> >>> +++ b/arch/arm64/configs/defconfig
> >>> @@ -518,6 +518,8 @@ CONFIG_DRM_HISI_HIBMC=m
> >>>  CONFIG_DRM_HISI_KIRIN=m
> >>>  CONFIG_DRM_MESON=m
> >>>  CONFIG_DRM_PL111=m
> >>> +CONFIG_DRM_LIMA=m
> >>> +CONFIG_DRM_PANFROST=m
> >>>  CONFIG_FB=y
> >>>  CONFIG_FB_MODE_HELPERS=y
> >>>  CONFIG_BACKLIGHT_GENERIC=m
> >>> @@ -718,7 +720,6 @@ CONFIG_ARCH_TEGRA_194_SOC=y
> >>>  CONFIG_ARCH_K3_AM6_SOC=y
> >>>  CONFIG_SOC_TI=y
> >>>  CONFIG_TI_SCI_PM_DOMAINS=y
> >>> -CONFIG_DEVFREQ_GOV_SIMPLE_ONDEMAND=y
> >>
> >> Exynos5433-tm2 board support the exynos-bus device which
> >> used the simple_ondmenad governor of devfreq.
> >> Why do you remove this configuration from the defconfig?
> >
> > It is selected by default by DRM_PANFROST. The difference is that
> > PANFROST selects it as module. The 'y' is chosen because of:
> >   SCSI_UFSHCD [=y] && SCSI_LOWLEVEL [=y] && SCSI [=y] && SCSI_DMA [=y]
>
> When I tried to find the history of CONFIG_DEVFREQ_GOV_SIMPLE_ONDEMAND
> for defconfig, the following patch[1] added this configuration.
> [1] b972ff75f2938c39e2205c23ea0e531d36b27f86
> - "arm64: defconfig: Enable UFS on msm8996"
>
> I think that this patch will affect the opration of 'USF on msm8998'.

Hi Chanwoo,

Thanks for the feedback. Unfortunately I do not see how removing a
dependency will affect that SCSI_UFSHCD. The
DEVFREQ_GOV_SIMPLE_ONDEMAND is still built in, exactly the same as
before...

Best regards,
Krzysztof