linux-kernel - Re: [PATCH v2] arm64: defconfig: Enable configs for DisplayPort on J721e

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <632b3099-36ff-6141-a27b-95d920588a80@ideasonboard.com>
Date:   Fri, 11 Mar 2022 10:48:42 +0200
From:   Tomi Valkeinen <tomi.valkeinen@...asonboard.com>
To:     Rahul T R <r-ravikumar@...com>, bjorn.andersson@...aro.org,
        catalin.marinas@....com, will@...nel.org, shawnguo@...nel.org,
        krzk@...nel.org, geert+renesas@...der.be,
        marcel.ziswiler@...adex.com
Cc:     biju.das.jz@...renesas.com, vkoul@...nel.org,
        enric.balletbo@...labora.com, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org, laurent.pinchart@...asonboard.com,
        nm@...com
Subject: Re: [PATCH v2] arm64: defconfig: Enable configs for DisplayPort on
 J721e

On 23/02/2022 12:15, Rahul T R wrote:
> Enable DRM and PHY configs required for supporting
> DisplayPort on J721e
> 
> Signed-off-by: Rahul T R <r-ravikumar@...com>
> ---
> 
> Notes:
>      v2:
>      	Fixed the places using savedefconfig
>      	Added more info in notes
>      
>      No change in vmlinux:
>      
>      	add/remove: 0/0 grow/shrink: 0/0 up/down: 0/0 (0)
>      	Function                                     old     new   delta
>      	Total: Before=24042991, After=24042991, chg +0.00%
>      
>      Diffstat of bootlogs:
>      
>      	after_boot.log |   46 ++++++++++++++++++++++++++--------------------
>      	1 file changed, 26 insertions(+), 20 deletions(-)
> 
>   arch/arm64/configs/defconfig | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 9f23d7ec1232..56d4c126f6da 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -736,6 +736,7 @@ CONFIG_DRM_THINE_THC63LVD1024=m
>   CONFIG_DRM_TI_SN65DSI86=m
>   CONFIG_DRM_I2C_ADV7511=m
>   CONFIG_DRM_I2C_ADV7511_AUDIO=y
> +CONFIG_DRM_CDNS_MHDP8546=m
>   CONFIG_DRM_DW_HDMI_AHB_AUDIO=m
>   CONFIG_DRM_DW_HDMI_CEC=m
>   CONFIG_DRM_IMX_DCSS=m
> @@ -750,6 +751,7 @@ CONFIG_DRM_MESON=m
>   CONFIG_DRM_PL111=m
>   CONFIG_DRM_LIMA=m
>   CONFIG_DRM_PANFROST=m
> +CONFIG_DRM_TIDSS=m
>   CONFIG_FB=y
>   CONFIG_FB_MODE_HELPERS=y
>   CONFIG_FB_EFI=y
> @@ -1151,6 +1153,7 @@ CONFIG_RESET_RZG2L_USBPHY_CTRL=y
>   CONFIG_RESET_TI_SCI=y
>   CONFIG_PHY_XGENE=y
>   CONFIG_PHY_SUN4I_USB=y
> +CONFIG_PHY_CADENCE_TORRENT=m
>   CONFIG_PHY_MIXEL_MIPI_DPHY=m
>   CONFIG_PHY_FSL_IMX8M_PCIE=y
>   CONFIG_PHY_HI6220_USB=y
> @@ -1175,6 +1178,7 @@ CONFIG_PHY_SAMSUNG_UFS=y
>   CONFIG_PHY_UNIPHIER_USB2=y
>   CONFIG_PHY_UNIPHIER_USB3=y
>   CONFIG_PHY_TEGRA_XUSB=y
> +CONFIG_PHY_J721E_WIZ=m
>   CONFIG_ARM_SMMU_V3_PMU=m
>   CONFIG_FSL_IMX8_DDR_PMU=m
>   CONFIG_QCOM_L2_PMU=y

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>

Not directly related to this patch, but this is slightly annoying:

cdns-mhdp8546 a000000.dp-bridge: invalid resource
cdns-mhdp8546 a000000.dp-bridge: Failed to get SAPB memory resource, 
HDCP not supported

The first one is an error print and the second is a warning. I think a 
dev_info about HDCP not supported would be enough.

  Tomi