From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2070.outbound.protection.outlook.com [40.107.22.70]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3E6C641C6E; Sat, 12 Oct 2024 10:01:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.22.70 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728727275; cv=fail; b=gKko2SVjIXn49zWcBpmkGyaE99vlQEC56nTqkG17khNBp3jXNwK8PMA1aUO8cgtt1QuS5hbUy3yvzydsPlPi1QIgjacy90OIk9neQchVsm5T/GAwIaoKHliRJIVJ+HmUyaBD+tzuXxpJpoHuwesFtOyKAQsnADOKGvTQ+BK7dgQ= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728727275; c=relaxed/simple; bh=qir5Eoc0Yla0TiXr/zglZ4hfH2y2Tx1erbYqbkoBxzE=; h=Message-ID:Date:Subject:To:Cc:References:From:In-Reply-To: Content-Type:MIME-Version; b=GjgXBVqM+1evKhR2NRhkIxGjCd1YLGOrcosrLoSHGR1pbTYT2c3kLs/xyQrSTdKFWdWv/kVzLRNdbbKXlcibv3YdSM9xJHRFX+GcbqEbU6w8xYCrCZKqQpL3Wd/0QyChT41213JLHxmL/sQMA6fcU+FTTkkkYMaOoNoi8keqF8o= ARC-Authentication-Results:i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=Lep7XixM; arc=fail smtp.client-ip=40.107.22.70 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="Lep7XixM" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=iSoRvxTcJDjDU701sE6tubPmmx0+5w7CY/IzgdE0uqia7BO6iSJ3i4RCXZ2ZgKQnk3OISsKEyHa/p7azahnE5cuZ9/yAvmD65NCeY1rVuDIWfXAVRlvAjiHl+A9yu0PpwXhl+cm2ib9nHx2HwcyRPWgRdo/Je86h2D+ynXnGiSCuFBO6fPPU2qAAFBHHbDmogB1dnyqyzQuc0+bsQuGWnul56AsYcahmIav+MzxpDWjhVLjpEZFuQxcnvK52USY4FTb5lTYlJBkEC+41TxLk2zp0EMaZi8sbgWsBWfxjpg39rOBOpqyD+B+peqePDOBdq07Syaw/n5dMW1QUwHXhAQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ZrqFt1xihqVPEiWtf5t23Pj/NKXLfajZxcQjrG6NkDk=; b=Q6vGTeHtEhKAMhdMQ4r1yTwegUxATox981TC2leNBp9PqaF/f3D7qglMvS3NcRTJFCqctjeOQtRcUfXgrJVZewZNDOC6Q6AX6thKLLslX7zKgCKpSs+Wyt94cB6Drtqe2M+Trwo5kRYw5Lp4kchSL3I00KoydI9M5rEdIvFNVWAXKpn3LZZzevtVOynu9dFM/qzfzxk/LB0CUpMAEJwK3JXKtKF8/GAVrIleAiksXuutBxZ2lU6IYxgmJ5y5DOWrv9VYJy/RDwdK1RkYh3qeD58AMx3KzFi5vBLsE7D0EjZKQlOtCW6Zk8BVXal9HN0ROCfZjTKKJiy9ip66eIUWDg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ZrqFt1xihqVPEiWtf5t23Pj/NKXLfajZxcQjrG6NkDk=; b=Lep7XixMdkTJIvX4uw2bVp2Ro1L5BOEBJN9MmS7BLldMKlQ56BCCGGMmxltH1DFT5flSlXR1OEMd4zOb2NtX3ZOdS8/K46ZqWhmXWMrer+09Q/Qs0kzV/wM6/1zO/d4BobSHBa1ovmmJAhF4PzluYJBidt9gDFxQaAr6OWbTH7Jplw/TqOjscsB+L7uraHI1whU2BJAK/oMkkPHrHXThVHOmM4JMSUNDcvXJvP/UoZHxa4Rp4/qXQQEm92I0ljjj0yrnIjXjLGkQBiv1iKgDdWRUkrXSd+TDb6B492gykH/bRqDvk6s0eC2WQwa7KCc2yIhpYOOJrxOk/9f2/48ZLw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM7PR04MB7046.eurprd04.prod.outlook.com (2603:10a6:20b:113::22) by AS8PR04MB8484.eurprd04.prod.outlook.com (2603:10a6:20b:34c::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8026.15; Sat, 12 Oct 2024 10:01:08 +0000 Received: from AM7PR04MB7046.eurprd04.prod.outlook.com ([fe80::d1ce:ea15:6648:6f90]) by AM7PR04MB7046.eurprd04.prod.outlook.com ([fe80::d1ce:ea15:6648:6f90%4]) with mapi id 15.20.8048.020; Sat, 12 Oct 2024 10:01:07 +0000 Message-ID: <78fb51c5-4568-4f91-a25f-e4ea4bfbd7ee@nxp.com> Date: Sat, 12 Oct 2024 18:01:30 +0800 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 6/9] drm/bridge: Add ITE IT6263 LVDS to HDMI converter To: Dmitry Baryshkov Cc: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, andrzej.hajda@intel.com, neil.armstrong@linaro.org, rfoss@kernel.org, Laurent.pinchart@ideasonboard.com, jonas@kwiboo.se, jernej.skrabec@gmail.com, airlied@gmail.com, simona@ffwll.ch, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, catalin.marinas@arm.com, will@kernel.org, quic_bjorande@quicinc.com, geert+renesas@glider.be, arnd@arndb.de, nfraprado@collabora.com, o.rempel@pengutronix.de, y.moog@phytec.de, marex@denx.de, isaac.scott@ideasonboard.com, biju.das.jz@bp.renesas.com References: <20241012073543.1388069-1-victor.liu@nxp.com> <20241012073543.1388069-7-victor.liu@nxp.com> From: Liu Ying Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8 X-ClientProxiedBy: SI2PR06CA0010.apcprd06.prod.outlook.com (2603:1096:4:186::6) To AM7PR04MB7046.eurprd04.prod.outlook.com (2603:10a6:20b:113::22) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM7PR04MB7046:EE_|AS8PR04MB8484:EE_ X-MS-Office365-Filtering-Correlation-Id: 36cebdd2-6f2a-4a6d-932d-08dceaa4ca2f X-LD-Processed: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|1800799024; X-Microsoft-Antispam-Message-Info: =?utf-8?B?NDE5Y3pLeVJ5RkdEdlVtTEsxbmw0UFdQdDNPYkh2NXViZkNEekN1ak41SlhI?= =?utf-8?B?RXpGb0p3NzVZZ2gvaUd2bjNYZEpoazg4bWdzRjkwc3U5ZWZvdGdCNVZabmFl?= =?utf-8?B?b3RaTGRxZzQ3aFExRzB5M3JVWjVDdkpnMWQ0b3V1WmhWZmdoN1BMN1Ric1VL?= =?utf-8?B?YkRyMlVaM0FEYXBkdlJQVlcrS0d1S0F5QUlNWmRiQzE0eGlGUnB1K1BzSEta?= =?utf-8?B?KzVkNGViZFVjcFprdUppL3pUdUx5ZDVBOWxtUXQrS1NXV25vbUZVODhaa2M0?= =?utf-8?B?TmhOaGxXcGV6aVR4M2kzdktVaStTWkdBem9aWHIrVDNpdDFCLzhwQ05rMGlQ?= =?utf-8?B?S1pyTFRvdmxtb0pXVEVpR0ttK0JBSkxTVjlqUjZVU1YwT3NpcU5iZW5TeW02?= =?utf-8?B?ZXBHWWRkWlhDZE9Cd1d0RkxwMmVlNitFT2s0OG5TbUFFRzM0ajdGTyt4VXk1?= =?utf-8?B?dlBNY0M5Um9Ga0tNTCs4ZUhOVmRNQU8yNTliSXZ5aFBsYW1nNncxMStDV1FF?= =?utf-8?B?bnAzV3dSYjNtaFd0dVdnaEFZb3lwem9EMmZXZnRZQUtKZGVSNDdaNCtPelRp?= =?utf-8?B?NTMzM0RhMENlOFVrMlhyU0hnNUpkVjdyaEoyakQ0dUpYYi9vODd6dmlDSE5z?= =?utf-8?B?aXVxMmRtOTUwbnc5ZitRd2lkSG1sUWJaSnAyRFM3cG9GT3NxQzBXL242RXNG?= =?utf-8?B?d2lvOTduN0ZYWXNoUnhYdXJ1M1ZYcGVoY3dmbGZKb090QmFsT1UrNFRaMTdN?= =?utf-8?B?ZUF6ZytNa2o3WE9QcGNuM282b2pueDVhejJ3YytQTEZ4dHBVZUgxN09HRytl?= =?utf-8?B?MzFkSWRsamNJYXRnc3k1aEhMM0dwTEl6cDlWbVBqVUFXd1plN2luUzVRQWYr?= =?utf-8?B?YUhKblhGeGdpblhtTGRjd2srcmMrRjQ4cUlBaW5GN3VKNzlmdkhmczFmNjBu?= =?utf-8?B?ZnhuYVZaT3pYaWJQUjZkdks3UEJETW1SV0V2d1Ywc3RCdVBUMVIvOGhRS2Ix?= =?utf-8?B?emRIM1ZYWlo2dGhSbitucW1vaGk3NDlqbWdMSzd2blFmMVNtRlIrQXVKRFQz?= =?utf-8?B?b3cwMGpTMFFDcFNBRHFidERhR2JoOTNnVXphV1gyRElvWGRFREk3V1VveTR1?= =?utf-8?B?MlVKdVZSOUhOa3krK2VFVGRLeVdCamcxRTR2TmpMOVU3Wkw5M1gvdlRxRkFR?= =?utf-8?B?MlRFWGx0b2JFZkFZSHVOMXJBWEl5LzhZMVRSaUlpeWpvYkdkWmZncWY3Skkx?= =?utf-8?B?d2xsbEpCaEFRc1BxejNGVHFBUlBhN1B1Z08wc1dPTXR6UnNuenVnTzFMRFJw?= =?utf-8?B?aFJvQ1lNZlBKZ29uS1ZyblU4b01wOTkvcjh5TjF5ZHI2QWRUcEwzaDFJVG9W?= =?utf-8?B?a1hpT3ZCK0ZXT2FZMXVFWTBJVFFLaHRreUVSbjh5ckcyY0QxSnJOdTdGSnhH?= =?utf-8?B?VTRWbmZpb3V3ZVhydEU2VHdDTS9RRnkyM3M1Wm4zQTR2bytHSW01UXpZbzZW?= =?utf-8?B?Rk5LWjBIck9iTWFOQXU0ZFdhbjFuak9zeGFscU15aHkwZnpTV3BaQkthWkJB?= =?utf-8?B?bGxwUkVtRFZJVmxtajZhSWw4UU9ZNDRWNWs4QUpFWVRvc3A1MGlnUXZVanRo?= =?utf-8?B?MXM3VklMamNYdkV0T0lkQ3RHVDA3dWFkeWRyVUVoTHVSWFl1U0JYNXUwcmZk?= =?utf-8?B?ZEZ4Z3U2ZkcydituUVgvSjVmVEp1TkNtenlCV2tKKzdMTDhvRTFQZGl3PT0=?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM7PR04MB7046.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MkJnVXlsWmdtSi9LUkRCRHExelYzT01pVWFPdWY0S2RhWFIvYnVpREIvM0pw?= =?utf-8?B?ck5NRnYyL3hDOHIrTU5TSnRCNW5odU1JeHlSbW5tVjhOY2IxZHR5M1ZHNnhi?= =?utf-8?B?ZE9aakxpWVBvenlyRE9HeXQ1cnlTM3hxeEVVK3JES2JPbFBwbkhXWUdQemZk?= =?utf-8?B?WFlTcnJMVEJKWGdRcGNNTnNFYWttRnJiTFo1ZU5mNFVIcXRWM0lLS3M5ajJY?= =?utf-8?B?RFhXMndDY1k4UFNDOXNkMlg0QlphVDg3WGg5cnRpMklFNHJLZFQ0TGtPNmh0?= =?utf-8?B?R2d6T0x6K2RDa2FmdW44T1FZaWU3QzBqNXJUZy9RZFNnVit4bEZXUjAveENa?= =?utf-8?B?dWpncjJyM3hTSGgyTGNGRXhzYU5OYWlUQU1qMnFyaGRGL256Y3VTS09aTmNq?= =?utf-8?B?YnE3bE9kSVdPRkpURitpdWYyWXRtVW1jOFBNOC9JZXNCVmk4MitiTS9DOEY1?= =?utf-8?B?d2d4emRHKy9ZZCt1MFJYYURaYlRpTnNrRmdlS1VVOHorOGVCS3pGK3lVWisr?= =?utf-8?B?VEtvL01wTVVPNVBXZmJBMGR4eVhQdXhhcGtrZXZ0RWIxWHZzYWVqL0RiUlFT?= =?utf-8?B?bHBpUHhVVDVuN3ljL0pTMEw3SjhQK21kYVh0VEpIMnRSZi9FK2t1UVFrdUYx?= =?utf-8?B?cVBkSnE3M3RBUTNUaGdwd09DNlh2WDZOcWphY3BKK0QzTEswTDVPclBDVy82?= =?utf-8?B?T2NMZndFUmZNbytVTlhHK25tN2tIdjRjU09zRnRKS2lCQW5DMnFTTUFHRTVW?= =?utf-8?B?ZEJ2WjRUamE0SEc5RWI2anZIT0RIdEllZ21FbDNZTWNGbkhCWXdrc3p0SmZi?= =?utf-8?B?TVllRUdDZFhxZWN6R3ZWZFIyR1RXMFRNM1JOVVJFeEMwY0VVZU81NlNIaFhy?= =?utf-8?B?Y0s2eVVGQzlHUHgxUmlRcytLbXJteEVBQkNNZVhORUFxdFgxUis0NFlOT3pI?= =?utf-8?B?SE13Y0plcTZFWWpZWTZVWUZVUW9xS2xOMjl0dktudyt0Vkc1Vko4SWhWR0pm?= =?utf-8?B?cm1YUmlzWTFhTVpoWU5JU3JVcWcxNXZTempGRnpibm9FL2RWWGs4QW5tcUpi?= =?utf-8?B?QWFMbzF6SVhGZHU1Vi9Da1pxSUxVTXMrdHNlUDk5ZkpOL3B6TVR5MlN1NkdL?= =?utf-8?B?QTV0OTF2bENQb1Y1bXU4TVpUd2VCS1hYZE13Rkh0ZUdRMkllZGRtbDlETDdC?= =?utf-8?B?TEZvTklXT2NPMnVJZWEvTVREcFlFTE4xSHB2UTdoYkQwQXV0aTRGVGpyZlhC?= =?utf-8?B?Z3RjTW40QjZ5SjdNWkttanhCUHZsdkhKSjdORE00c1dNdEhvYTZvU2locnE2?= =?utf-8?B?Q2hCak8xdklBOENxME1tN2VPOE1mQ0lyditKSWI0Z2VaeEhiSFUyRmJKbG1h?= =?utf-8?B?Wm5QeVpSbUw0NnRKazJMMUNkaUEvaUxueVZ1dGNXY2Y1Nmp1VWZ6Ykg3d1BO?= =?utf-8?B?Zmt3b3pNOUhBajBoOW52eWloSitodlBneTR1MDljWVZlZFZDalNwZmcrTTYv?= =?utf-8?B?Rlk0MU9VUDY3ZmQvbkZVQ2tGdFlwVTFtTkJGTHcvUzMvM09wOU1mUHRXaEow?= =?utf-8?B?Vy9pTWF1L3htcjg4L2tDMFZLT3hZdkptS2JzY2poaVRMVXRuTUkveGFUWTlW?= =?utf-8?B?YzZUWk9lVTAzWWhsZG5BanA3dVA2R2ZLSzArWHdaODlYRFNpRjErZkFYY3lj?= =?utf-8?B?MGlQVk8wVXA4Sm9jQyt4S0FNZTdNZWpqS3M2ZzBtaCtOWllsaGxjN2dPYWpN?= =?utf-8?B?cHZ2MGFhQ21WSFUvSnBrbzFyNVNTTUNiczZhYnI1T3dEdWo2VnQrTGU0L0pj?= =?utf-8?B?ckoxNHZxeGNBVUU0S2xxdFVYMHdTTWNzT01SeFM3Z2owaExxN3g1M29RYXZW?= =?utf-8?B?cEczK0p0YjlvdjhLOUR6WG5rOTBVZlA0M0F3ZWtYdzhNWmh0U2JtRmxPcVM3?= =?utf-8?B?alZuK05qMjlHL3JucHJ3czdnNUw0dkx6Q25BRFo2L1lPQjAyTGtLYzZudDhl?= =?utf-8?B?c1VWUmMrSEl2RTFwVzVEMlhnd1U3L0VpRmUwcm82UzdyMnVNTG1ma3FJd1Y2?= =?utf-8?B?WmtRQlhQb2xvZysvalNnSGE3ejFiQTduYlFvbHQza3hoMEVDbU1UVnlOc2hx?= =?utf-8?Q?HCKUfMfvZTNFnRq4kcipvWe9B?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 36cebdd2-6f2a-4a6d-932d-08dceaa4ca2f X-MS-Exchange-CrossTenant-AuthSource: AM7PR04MB7046.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Oct 2024 10:01:07.0274 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: xwcr5Wj+rfgMGCFkNmtgLZQcKERIy3lnqWSlInDFACuDsAW62iVxFlRWQVTdDXyrrseuHKYqbPsq8TdmjPLbZQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8484 On 10/12/2024, Dmitry Baryshkov wrote: > On Sat, Oct 12, 2024 at 03:35:40PM +0800, Liu Ying wrote: >> Add basic HDMI video output support. Currently, only RGB888 output >> pixel format is supported. At the LVDS input side, the driver >> supports single LVDS link and dual LVDS links with "jeida-24" LVDS >> mapping. >> >> Product link: >> https://www.ite.com.tw/en/product/cate1/IT6263 >> >> Signed-off-by: Liu Ying >> --- >> v2: >> * Add AVI inforframe support. (Maxime) > > Maxime asked to wire up the HDMI Connector framework. I have pushed the > patch that converts lt9611.c driver. Please implement similar changes in > your driver. Hmm, didn't fully get the meaning of "the new HDMI bridge infrastructure" maxime asked. Ok, will take a look at lt9611.c. > >> * Add DRM_MODE_CONNECTOR_HDMIA. (Biju) >> * Rename it6263_reset() to it6263_hw_reset(). (Biju) >> * Check number of LVDS link data lanes. (Biju) >> >> drivers/gpu/drm/bridge/Kconfig | 8 + >> drivers/gpu/drm/bridge/Makefile | 1 + >> drivers/gpu/drm/bridge/ite-it6263.c | 919 ++++++++++++++++++++++++++++ >> 3 files changed, 928 insertions(+) >> create mode 100644 drivers/gpu/drm/bridge/ite-it6263.c >> >> diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig >> index 3eb955333c80..93f99682a090 100644 >> --- a/drivers/gpu/drm/bridge/Kconfig >> +++ b/drivers/gpu/drm/bridge/Kconfig >> @@ -90,6 +90,14 @@ config DRM_FSL_LDB >> help >> Support for i.MX8MP DPI-to-LVDS on-SoC encoder. >> >> +config DRM_ITE_IT6263 >> + tristate "ITE IT6263 LVDS/HDMI bridge" >> + depends on OF >> + select DRM_KMS_HELPER >> + select REGMAP_I2C >> + help >> + ITE IT6263 LVDS to HDMI bridge chip driver. >> + >> config DRM_ITE_IT6505 >> tristate "ITE IT6505 DisplayPort bridge" >> depends on OF >> diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile >> index 7df87b582dca..f3776dd473fd 100644 >> --- a/drivers/gpu/drm/bridge/Makefile >> +++ b/drivers/gpu/drm/bridge/Makefile >> @@ -6,6 +6,7 @@ obj-$(CONFIG_DRM_CHRONTEL_CH7033) += chrontel-ch7033.o >> obj-$(CONFIG_DRM_CROS_EC_ANX7688) += cros-ec-anx7688.o >> obj-$(CONFIG_DRM_DISPLAY_CONNECTOR) += display-connector.o >> obj-$(CONFIG_DRM_FSL_LDB) += fsl-ldb.o >> +obj-$(CONFIG_DRM_ITE_IT6263) += ite-it6263.o >> obj-$(CONFIG_DRM_ITE_IT6505) += ite-it6505.o >> obj-$(CONFIG_DRM_LONTIUM_LT8912B) += lontium-lt8912b.o >> obj-$(CONFIG_DRM_LONTIUM_LT9211) += lontium-lt9211.o >> diff --git a/drivers/gpu/drm/bridge/ite-it6263.c b/drivers/gpu/drm/bridge/ite-it6263.c >> new file mode 100644 >> index 000000000000..9b3318792f15 >> --- /dev/null >> +++ b/drivers/gpu/drm/bridge/ite-it6263.c >> @@ -0,0 +1,919 @@ >> +// SPDX-License-Identifier: GPL-2.0 >> +/* >> + * Copyright 2024 NXP >> + */ >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> + >> +/* ----------------------------------------------------------------------------- >> + * LVDS registers >> + */ >> + >> +/* LVDS software reset registers */ >> +#define LVDS_REG_05 0x05 >> +#define REG_SOFT_P_RST BIT(1) >> + >> +/* LVDS system configuration registers */ >> +/* 0x0b */ >> +#define LVDS_REG_0B 0x0b >> +#define REG_SSC_PCLK_RF BIT(0) >> +#define REG_LVDS_IN_SWAP BIT(1) >> + >> +/* LVDS test pattern gen control registers */ >> +/* 0x2c */ >> +#define LVDS_REG_2C 0x2c >> +#define REG_COL_DEP GENMASK(1, 0) >> +#define BIT8 FIELD_PREP(REG_COL_DEP, 2) >> +#define OUT_MAP BIT(4) >> +#define JEIDA 0 >> +#define REG_DESSC_ENB BIT(6) >> +#define DMODE BIT(7) >> +#define DISO BIT(7) >> +#define SISO 0 >> + >> +#define LVDS_REG_3C 0x3c >> +#define LVDS_REG_3F 0x3f >> +#define LVDS_REG_47 0x47 >> +#define LVDS_REG_48 0x48 >> +#define LVDS_REG_4F 0x4f >> +#define LVDS_REG_52 0x52 >> + >> +/* ----------------------------------------------------------------------------- >> + * HDMI registers are separated into three banks: >> + * 1) HDMI register common bank: 0x00 ~ 0x2f >> + */ >> + >> +/* HDMI genernal registers */ >> +#define HDMI_REG_SW_RST 0x04 >> +#define SOFTREF_RST BIT(5) >> +#define SOFTA_RST BIT(4) >> +#define SOFTV_RST BIT(3) >> +#define AUD_RST BIT(2) >> +#define HDCP_RST BIT(0) >> +#define HDMI_RST_ALL (SOFTREF_RST | SOFTA_RST | SOFTV_RST | \ >> + AUD_RST | HDCP_RST) >> + >> +#define HDMI_REG_SYS_STATUS 0x0e >> +#define HPDETECT BIT(6) >> +#define TXVIDSTABLE BIT(4) >> + >> +#define HDMI_REG_BANK_CTRL 0x0f >> +#define REG_BANK_SEL BIT(0) >> + >> +/* HDMI System DDC control registers */ >> +#define HDMI_REG_DDC_MASTER_CTRL 0x10 >> +#define MASTER_SEL_HOST BIT(0) >> + >> +#define HDMI_REG_DDC_HEADER 0x11 >> + >> +#define HDMI_REG_DDC_REQOFF 0x12 >> +#define HDMI_REG_DDC_REQCOUNT 0x13 >> +#define HDMI_REG_DDC_EDIDSEG 0x14 >> + >> +#define HDMI_REG_DDC_CMD 0x15 >> +#define DDC_CMD_EDID_READ 0x3 >> +#define DDC_CMD_FIFO_CLR 0x9 >> + >> +#define HDMI_REG_DDC_STATUS 0x16 >> +#define DDC_DONE BIT(7) >> +#define DDC_NOACK BIT(5) >> +#define DDC_WAITBUS BIT(4) >> +#define DDC_ARBILOSE BIT(3) >> +#define DDC_ERROR (DDC_NOACK | DDC_WAITBUS | DDC_ARBILOSE) >> + >> +#define HDMI_DDC_FIFO_BYTES 32 >> +#define HDMI_REG_DDC_READFIFO 0x17 >> +#define HDMI_REG_LVDS_PORT 0x1d /* LVDS input control I2C addr */ >> +#define HDMI_REG_LVDS_PORT_EN 0x1e >> +#define LVDS_INPUT_CTRL_I2C_ADDR 0x33 >> + >> +/* ----------------------------------------------------------------------------- >> + * 2) HDMI register bank0: 0x30 ~ 0xff >> + */ >> + >> +/* HDMI AFE registers */ >> +#define HDMI_REG_AFE_DRV_CTRL 0x61 >> +#define AFE_DRV_PWD BIT(5) >> +#define AFE_DRV_RST BIT(4) >> + >> +#define HDMI_REG_AFE_XP_CTRL 0x62 >> +#define AFE_XP_GAINBIT BIT(7) >> +#define AFE_XP_ER0 BIT(4) >> +#define AFE_XP_RESETB BIT(3) >> + >> +#define HDMI_REG_AFE_ISW_CTRL 0x63 >> + >> +#define HDMI_REG_AFE_IP_CTRL 0x64 >> +#define AFE_IP_GAINBIT BIT(7) >> +#define AFE_IP_ER0 BIT(3) >> +#define AFE_IP_RESETB BIT(2) >> + >> +/* HDMI input data format registers */ >> +#define HDMI_REG_INPUT_MODE 0x70 >> +#define IN_RGB 0x00 >> + >> +/* HDMI general control registers */ >> +#define HDMI_REG_HDMI_MODE 0xc0 >> +#define TX_HDMI_MODE BIT(0) >> + >> +#define HDMI_REG_GCP 0xc1 >> +#define AVMUTE BIT(0) >> +#define HDMI_COLOR_DEPTH GENMASK(6, 4) >> +#define HDMI_COLOR_DEPTH_24 FIELD_PREP(HDMI_COLOR_DEPTH, 4) >> + >> +#define HDMI_REG_PKT_GENERAL_CTRL 0xc6 >> +#define ENABLE_PKT BIT(0) >> +#define REPEAT_PKT BIT(1) >> + >> +/* ----------------------------------------------------------------------------- >> + * 3) HDMI register bank1: 0x130 ~ 0x1ff (HDMI packet registers) >> + */ >> + >> +/* AVI packet registers */ >> +#define HDMI_REG_AVI_DB1 0x158 >> +#define HDMI_REG_AVI_DB2 0x159 >> +#define HDMI_REG_AVI_DB3 0x15a >> +#define HDMI_REG_AVI_DB4 0x15b >> +#define HDMI_REG_AVI_DB5 0x15c >> +#define HDMI_REG_AVI_CSUM 0x15d >> +#define HDMI_REG_AVI_DB6 0x15e >> +#define HDMI_REG_AVI_DB7 0x15f >> +#define HDMI_REG_AVI_DB8 0x160 >> +#define HDMI_REG_AVI_DB9 0x161 >> +#define HDMI_REG_AVI_DB10 0x162 >> +#define HDMI_REG_AVI_DB11 0x163 >> +#define HDMI_REG_AVI_DB12 0x164 >> +#define HDMI_REG_AVI_DB13 0x165 >> + >> +#define HDMI_AVI_DB_CHUNK1_SIZE (HDMI_REG_AVI_DB5 - HDMI_REG_AVI_DB1 + 1) >> +#define HDMI_AVI_DB_CHUNK2_SIZE (HDMI_REG_AVI_DB13 - HDMI_REG_AVI_DB6 + 1) >> + >> +#define MAX_PIXEL_CLOCK_KHZ 150000 >> +#define HIGH_PIXEL_CLOCK_KHZ 80000 >> + >> +struct it6263 { >> + struct device *dev; >> + struct i2c_client *hdmi_i2c; >> + struct i2c_client *lvds_i2c; >> + struct regmap *hdmi_regmap; >> + struct regmap *lvds_regmap; >> + struct drm_bridge bridge; >> + struct drm_bridge *next_bridge; >> + struct drm_connector connector; >> + struct gpio_desc *reset_gpio; >> + u8 lvds_link_num_dlanes; >> + bool lvds_dual_link; >> + bool lvds_link12_swap; >> +}; >> + >> +static inline struct it6263 *bridge_to_it6263(struct drm_bridge *bridge) >> +{ >> + return container_of(bridge, struct it6263, bridge); >> +} >> + >> +static inline struct it6263 *connector_to_it6263(struct drm_connector *conn) >> +{ >> + return container_of(conn, struct it6263, connector); >> +} >> + >> +static bool it6263_hdmi_writeable_reg(struct device *dev, unsigned int reg) >> +{ >> + switch (reg) { >> + case HDMI_REG_SW_RST: >> + case HDMI_REG_BANK_CTRL: >> + case HDMI_REG_DDC_MASTER_CTRL: >> + case HDMI_REG_DDC_HEADER: >> + case HDMI_REG_DDC_REQOFF: >> + case HDMI_REG_DDC_REQCOUNT: >> + case HDMI_REG_DDC_EDIDSEG: >> + case HDMI_REG_DDC_CMD: >> + case HDMI_REG_LVDS_PORT: >> + case HDMI_REG_LVDS_PORT_EN: >> + case HDMI_REG_AFE_DRV_CTRL: >> + case HDMI_REG_AFE_XP_CTRL: >> + case HDMI_REG_AFE_ISW_CTRL: >> + case HDMI_REG_AFE_IP_CTRL: >> + case HDMI_REG_INPUT_MODE: >> + case HDMI_REG_HDMI_MODE: >> + case HDMI_REG_GCP: >> + case HDMI_REG_PKT_GENERAL_CTRL: >> + case HDMI_REG_AVI_DB1: >> + case HDMI_REG_AVI_DB2: >> + case HDMI_REG_AVI_DB3: >> + case HDMI_REG_AVI_DB4: >> + case HDMI_REG_AVI_DB5: >> + case HDMI_REG_AVI_CSUM: >> + case HDMI_REG_AVI_DB6: >> + case HDMI_REG_AVI_DB7: >> + case HDMI_REG_AVI_DB8: >> + case HDMI_REG_AVI_DB9: >> + case HDMI_REG_AVI_DB10: >> + case HDMI_REG_AVI_DB11: >> + case HDMI_REG_AVI_DB12: >> + case HDMI_REG_AVI_DB13: >> + return true; >> + default: >> + return false; >> + } >> +} >> + >> +static bool it6263_hdmi_readable_reg(struct device *dev, unsigned int reg) >> +{ >> + if (it6263_hdmi_writeable_reg(dev, reg)) >> + return true; >> + >> + switch (reg) { >> + case HDMI_REG_SYS_STATUS: >> + case HDMI_REG_DDC_STATUS: >> + case HDMI_REG_DDC_READFIFO: >> + return true; >> + default: >> + return false; >> + } >> +} >> + >> +static bool it6263_hdmi_volatile_reg(struct device *dev, unsigned int reg) >> +{ >> + switch (reg) { >> + case HDMI_REG_SW_RST: >> + case HDMI_REG_SYS_STATUS: >> + case HDMI_REG_DDC_STATUS: >> + case HDMI_REG_DDC_READFIFO: >> + return true; >> + default: >> + return false; >> + } >> +} >> + >> +static const struct regmap_range_cfg it6263_hdmi_range_cfg = { >> + .range_min = 0x00, >> + .range_max = HDMI_REG_AVI_DB13, >> + .selector_reg = HDMI_REG_BANK_CTRL, >> + .selector_mask = REG_BANK_SEL, >> + .selector_shift = 0, >> + .window_start = 0x00, >> + .window_len = 0x100, >> +}; >> + >> +static const struct regmap_config it6263_hdmi_regmap_config = { >> + .name = "it6263-hdmi", >> + .reg_bits = 8, >> + .val_bits = 8, >> + .writeable_reg = it6263_hdmi_writeable_reg, >> + .readable_reg = it6263_hdmi_readable_reg, >> + .volatile_reg = it6263_hdmi_volatile_reg, >> + .max_register = HDMI_REG_AVI_DB13, >> + .ranges = &it6263_hdmi_range_cfg, >> + .num_ranges = 1, >> + .cache_type = REGCACHE_MAPLE, >> +}; >> + >> +static bool it6263_lvds_writeable_reg(struct device *dev, unsigned int reg) >> +{ >> + switch (reg) { >> + case LVDS_REG_05: >> + case LVDS_REG_0B: >> + case LVDS_REG_2C: >> + case LVDS_REG_3C: >> + case LVDS_REG_3F: >> + case LVDS_REG_47: >> + case LVDS_REG_48: >> + case LVDS_REG_4F: >> + case LVDS_REG_52: >> + return true; >> + default: >> + return false; >> + } >> +} >> + >> +static bool it6263_lvds_readable_reg(struct device *dev, unsigned int reg) >> +{ >> + return it6263_lvds_writeable_reg(dev, reg); >> +} >> + >> +static bool it6263_lvds_volatile_reg(struct device *dev, unsigned int reg) >> +{ >> + return reg == LVDS_REG_05; >> +} >> + >> +static const struct regmap_config it6263_lvds_regmap_config = { >> + .name = "it6263-lvds", >> + .reg_bits = 8, >> + .val_bits = 8, >> + .writeable_reg = it6263_lvds_writeable_reg, >> + .readable_reg = it6263_lvds_readable_reg, >> + .volatile_reg = it6263_lvds_volatile_reg, >> + .max_register = LVDS_REG_52, >> + .cache_type = REGCACHE_MAPLE, >> +}; >> + >> +static const char * const it6263_supplies[] = { >> + "ivdd", "ovdd", "txavcc18", "txavcc33", "pvcc1", "pvcc2", >> + "avcc", "anvdd", "apvdd" >> +}; >> + >> +static int it6263_parse_dt(struct it6263 *it) >> +{ >> + struct device *dev = it->dev; >> + struct device_node *port0, *port1; >> + int ret; >> + >> + ret = of_property_read_u8(dev->of_node, "ite,lvds-link-num-data-lanes", >> + &it->lvds_link_num_dlanes); >> + if (ret) { >> + dev_err(dev, "failed to get LVDS link number of data lanes: %d\n", >> + ret); >> + return ret; >> + } >> + >> + it->next_bridge = devm_drm_of_get_bridge(dev, dev->of_node, 2, 0); >> + if (IS_ERR(it->next_bridge)) >> + return dev_err_probe(dev, PTR_ERR(it->next_bridge), >> + "failed to get next bridge\n"); >> + >> + port0 = of_graph_get_port_by_id(dev->of_node, 0); >> + port1 = of_graph_get_port_by_id(dev->of_node, 1); >> + if (port0 && port1) { >> + if (of_property_read_bool(port0, "dual-lvds-even-pixels") && >> + of_property_read_bool(port1, "dual-lvds-odd-pixels")) { >> + it->lvds_dual_link = true; >> + it->lvds_link12_swap = true; > > This is a nice name for the propery in the Dual Link LVDS case: > > ite,lvds-link-swap Swapping LVDS link1&2 is implied by the dual-lvds-{even,odd}-pixels properties that DT writers specify in port@0 and port@1. The DT writers don't need to use a dedicated property like ite,lvds-link-swap to explicitly indicate the swap. > > >> + } else if (of_property_read_bool(port0, "dual-lvds-odd-pixels") && >> + of_property_read_bool(port1, "dual-lvds-even-pixels")) { >> + it->lvds_dual_link = true; >> + } >> + >> + if (!it->lvds_dual_link) { >> + dev_err(dev, >> + "failed to get LVDS dual link pixel order\n"); >> + ret = -EINVAL; >> + } >> + } else if (port1) { >> + ret = -EINVAL; >> + dev_err(dev, "single input LVDS port1 is not supported\n"); >> + } else if (!port0) { >> + ret = -EINVAL; >> + dev_err(dev, "no input LVDS port\n"); >> + } >> + >> + of_node_put(port0); >> + of_node_put(port1); >> + >> + return ret; >> +} >> + >> +static inline void it6263_hw_reset(struct it6263 *it) >> +{ >> + if (!it->reset_gpio) >> + return; >> + >> + gpiod_set_value_cansleep(it->reset_gpio, 0); >> + fsleep(1000); >> + gpiod_set_value_cansleep(it->reset_gpio, 1); >> + /* The chip maker says the low pulse should be at least 40ms. */ >> + fsleep(40000); >> + gpiod_set_value_cansleep(it->reset_gpio, 0); >> + /* addtional time to wait the high voltage to be stable */ >> + fsleep(5000); >> +} >> + >> +static inline int it6263_lvds_set_i2c_addr(struct it6263 *it) >> +{ >> + int ret; >> + >> + ret = regmap_write(it->hdmi_regmap, HDMI_REG_LVDS_PORT, >> + LVDS_INPUT_CTRL_I2C_ADDR << 1); >> + if (ret) >> + return ret; >> + >> + return regmap_write(it->hdmi_regmap, HDMI_REG_LVDS_PORT_EN, BIT(0)); >> +} >> + >> +static inline void it6263_lvds_reset(struct it6263 *it) >> +{ >> + /* AFE PLL reset */ >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_3C, BIT(0), 0x0); >> + fsleep(1000); >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_3C, BIT(0), BIT(0)); >> + >> + /* software pixel clock domain reset */ >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_05, REG_SOFT_P_RST, >> + REG_SOFT_P_RST); >> + fsleep(1000); >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_05, REG_SOFT_P_RST, 0x0); >> + fsleep(10000); >> +} >> + >> +static inline void it6263_lvds_set_interface(struct it6263 *it) >> +{ >> + /* color depth */ >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_2C, REG_COL_DEP, BIT8); >> + /* output mapping */ >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_2C, OUT_MAP, JEIDA); >> + >> + if (it->lvds_dual_link) { >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_2C, DMODE, DISO); >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_52, BIT(1), BIT(1)); >> + } else { >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_2C, DMODE, SISO); >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_52, BIT(1), 0); >> + } >> +} >> + >> +static inline void it6263_lvds_set_afe(struct it6263 *it) >> +{ >> + regmap_write(it->lvds_regmap, LVDS_REG_3C, 0xaa); >> + regmap_write(it->lvds_regmap, LVDS_REG_3F, 0x02); >> + regmap_write(it->lvds_regmap, LVDS_REG_47, 0xaa); >> + regmap_write(it->lvds_regmap, LVDS_REG_48, 0x02); >> + regmap_write(it->lvds_regmap, LVDS_REG_4F, 0x11); >> + >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_0B, REG_SSC_PCLK_RF, >> + REG_SSC_PCLK_RF); >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_3C, 0x07, 0); >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_2C, REG_DESSC_ENB, >> + REG_DESSC_ENB); >> +} >> + >> +static inline void it6263_lvds_sys_cfg(struct it6263 *it) >> +{ >> + regmap_write_bits(it->lvds_regmap, LVDS_REG_0B, REG_LVDS_IN_SWAP, >> + it->lvds_link12_swap ? REG_LVDS_IN_SWAP : 0); >> +} >> + >> +static inline void it6263_lvds_config(struct it6263 *it) >> +{ >> + it6263_lvds_reset(it); >> + it6263_lvds_set_interface(it); >> + it6263_lvds_set_afe(it); >> + it6263_lvds_sys_cfg(it); >> +} >> + >> +static inline void it6263_hdmi_config(struct it6263 *it) >> +{ >> + regmap_write(it->hdmi_regmap, HDMI_REG_SW_RST, HDMI_RST_ALL); >> + regmap_write(it->hdmi_regmap, HDMI_REG_INPUT_MODE, IN_RGB); >> + regmap_write_bits(it->hdmi_regmap, HDMI_REG_GCP, HDMI_COLOR_DEPTH, >> + HDMI_COLOR_DEPTH_24); >> +} >> + >> +static enum drm_connector_status it6263_detect(struct it6263 *it) >> +{ >> + unsigned int val; >> + >> + regmap_read(it->hdmi_regmap, HDMI_REG_SYS_STATUS, &val); >> + if (val & HPDETECT) >> + return connector_status_connected; >> + else >> + return connector_status_disconnected; >> +} >> + >> +static enum drm_connector_status >> +it6263_connector_detect(struct drm_connector *connector, bool force) >> +{ >> + struct it6263 *it = connector_to_it6263(connector); >> + >> + return it6263_detect(it); >> +} >> + >> +static const struct drm_connector_funcs it6263_connector_funcs = { >> + .detect = it6263_connector_detect, >> + .fill_modes = drm_helper_probe_single_connector_modes, >> + .destroy = drm_connector_cleanup, >> + .reset = drm_atomic_helper_connector_reset, >> + .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state, >> + .atomic_destroy_state = drm_atomic_helper_connector_destroy_state, >> +}; >> + >> +static int it6263_read_edid(void *data, u8 *buf, unsigned int block, size_t len) >> +{ >> + struct it6263 *it = data; >> + struct regmap *regmap = it->hdmi_regmap; >> + unsigned int start = (block % 2) * EDID_LENGTH; >> + unsigned int segment = block >> 1; >> + unsigned int count, val; >> + int ret; >> + >> + regmap_write(regmap, HDMI_REG_DDC_MASTER_CTRL, MASTER_SEL_HOST); >> + regmap_write(regmap, HDMI_REG_DDC_HEADER, DDC_ADDR << 1); >> + regmap_write(regmap, HDMI_REG_DDC_EDIDSEG, segment); >> + >> + while (len) { >> + /* clear DDC FIFO */ >> + regmap_write(regmap, HDMI_REG_DDC_CMD, DDC_CMD_FIFO_CLR); >> + >> + ret = regmap_read_poll_timeout(regmap, HDMI_REG_DDC_STATUS, >> + val, val & DDC_DONE, >> + 2000, 10000); >> + if (ret) { >> + dev_err(it->dev, "failed to clear DDC FIFO:%d\n", ret); >> + return ret; >> + } >> + >> + count = len > HDMI_DDC_FIFO_BYTES ? HDMI_DDC_FIFO_BYTES : len; >> + >> + /* fire the read command */ >> + regmap_write(regmap, HDMI_REG_DDC_REQOFF, start); >> + regmap_write(regmap, HDMI_REG_DDC_REQCOUNT, count); >> + regmap_write(regmap, HDMI_REG_DDC_CMD, DDC_CMD_EDID_READ); >> + >> + start += count; >> + len -= count; >> + >> + ret = regmap_read_poll_timeout(regmap, HDMI_REG_DDC_STATUS, val, >> + val & (DDC_DONE | DDC_ERROR), >> + 20000, 250000); >> + if (ret && !(val & DDC_ERROR)) { >> + dev_err(it->dev, "failed to read EDID:%d\n", ret); >> + return ret; >> + } >> + >> + if (val & DDC_ERROR) { >> + dev_err(it->dev, "DDC error\n"); >> + return -EIO; >> + } >> + >> + /* cache to buffer */ >> + for (; count > 0; count--) { >> + regmap_read(regmap, HDMI_REG_DDC_READFIFO, &val); >> + *(buf++) = val; >> + } >> + } >> + >> + return 0; >> +} >> + >> +static int it6263_connector_get_modes(struct drm_connector *connector) >> +{ >> + struct it6263 *it = connector_to_it6263(connector); >> + const struct drm_edid *drm_edid; >> + int count; >> + >> + drm_edid = drm_edid_read_custom(connector, it6263_read_edid, it); >> + >> + drm_edid_connector_update(connector, drm_edid); >> + count = drm_edid_connector_add_modes(connector); >> + >> + drm_edid_free(drm_edid); >> + >> + return count; >> +} >> + >> +static const struct drm_connector_helper_funcs it6263_connector_helper_funcs = { >> + .get_modes = it6263_connector_get_modes, >> +}; >> + >> +static int it6263_bridge_atomic_check(struct drm_bridge *bridge, >> + struct drm_bridge_state *bridge_state, >> + struct drm_crtc_state *crtc_state, >> + struct drm_connector_state *conn_state) >> +{ >> + struct drm_display_mode *mode = &crtc_state->adjusted_mode; > > Use drm_atomic_helper_connector_hdmi_check(). Will take a look at this. > > Implement .hdmi_tmds_char_rate_valid(). Also, I think, single and dual Will take a look at this. > link LVDS have different max clock rates. Please correct me if I'm > wrong. Yes, I think so too. But, I don't know the exact max clock rates. IT6263 data sheet only says LVDS RX supports clock rate up to 150MHz. > >> + >> + return mode->clock > MAX_PIXEL_CLOCK_KHZ ? -EINVAL : 0; >> +} >> + >> +static void >> +it6263_bridge_atomic_disable(struct drm_bridge *bridge, >> + struct drm_bridge_state *old_bridge_state) >> +{ >> + struct it6263 *it = bridge_to_it6263(bridge); >> + >> + regmap_write_bits(it->hdmi_regmap, HDMI_REG_GCP, AVMUTE, AVMUTE); >> + regmap_write(it->hdmi_regmap, HDMI_REG_PKT_GENERAL_CTRL, 0); >> + regmap_write(it->hdmi_regmap, HDMI_REG_AFE_DRV_CTRL, >> + AFE_DRV_RST | AFE_DRV_PWD); >> +} >> + >> +static void >> +it6263_hdmi_avi_infoframe_configure(struct it6263 *it, >> + struct drm_connector *connector, >> + const struct drm_display_mode *mode) > > -> .hdmi_write_infoframe() , .hdmi_clear_infoframe(). Will take a look at them. > >> +{ >> + struct hdmi_avi_infoframe frame; >> + u8 buf[HDMI_INFOFRAME_SIZE(AVI)]; >> + int ret; >> + >> + ret = drm_hdmi_avi_infoframe_from_display_mode(&frame, connector, mode); >> + if (ret) { >> + dev_err(it->dev, "failed to setup AVI infoframe: %d\n", ret); >> + return; >> + } >> + >> + ret = hdmi_avi_infoframe_pack(&frame, buf, sizeof(buf)); >> + if (ret < 0) { >> + dev_err(it->dev, "failed to pack infoframe: %d\n", ret); >> + return; >> + } >> + >> + /* write the first AVI infoframe data byte chunk(DB1-DB5) */ >> + ret = regmap_bulk_write(it->hdmi_regmap, HDMI_REG_AVI_DB1, >> + &buf[HDMI_INFOFRAME_HEADER_SIZE], >> + HDMI_AVI_DB_CHUNK1_SIZE); >> + if (ret) { >> + dev_err(it->dev, "failed to write the 1st AVI infoframe data byte chunk: %d\n", >> + ret); >> + return; >> + } >> + >> + /* write the second AVI infoframe data byte chunk(DB6-DB13) */ >> + ret = regmap_bulk_write(it->hdmi_regmap, HDMI_REG_AVI_DB6, >> + &buf[HDMI_INFOFRAME_HEADER_SIZE + >> + HDMI_AVI_DB_CHUNK1_SIZE], >> + HDMI_AVI_DB_CHUNK2_SIZE); >> + if (ret) { >> + dev_err(it->dev, "failed to write the 2nd AVI infoframe data byte chunk: %d\n", >> + ret); >> + return; >> + } >> + >> + ret = regmap_write(it->hdmi_regmap, HDMI_REG_AVI_CSUM, buf[3]); >> + if (ret) >> + dev_err(it->dev, "failed to set AVI infoframe checksum: %d\n", >> + ret); >> +} >> + >> +static void >> +it6263_bridge_atomic_enable(struct drm_bridge *bridge, >> + struct drm_bridge_state *old_bridge_state) >> +{ >> + struct drm_atomic_state *state = old_bridge_state->base.state; >> + struct it6263 *it = bridge_to_it6263(bridge); >> + const struct drm_crtc_state *crtc_state; >> + struct regmap *regmap = it->hdmi_regmap; >> + const struct drm_display_mode *mode; >> + struct drm_connector *connector; >> + bool is_stable = false; >> + struct drm_crtc *crtc; >> + unsigned int val; >> + bool pclk_high; >> + int i, ret; >> + >> + connector = drm_atomic_get_new_connector_for_encoder(state, >> + bridge->encoder); >> + crtc = drm_atomic_get_new_connector_state(state, connector)->crtc; >> + crtc_state = drm_atomic_get_new_crtc_state(state, crtc); >> + mode = &crtc_state->adjusted_mode; >> + >> + regmap_write(regmap, HDMI_REG_HDMI_MODE, TX_HDMI_MODE); >> + >> + it6263_hdmi_avi_infoframe_configure(it, connector, mode); > > drm_atomic_helper_connector_hdmi_update_infoframes Will take a look at this. > >> + >> + /* HDMI AFE setup */ >> + pclk_high = mode->clock > HIGH_PIXEL_CLOCK_KHZ ? true : false; >> + regmap_write(regmap, HDMI_REG_AFE_DRV_CTRL, AFE_DRV_RST); >> + if (pclk_high) >> + regmap_write(regmap, HDMI_REG_AFE_XP_CTRL, >> + AFE_XP_GAINBIT | AFE_XP_RESETB); >> + else >> + regmap_write(regmap, HDMI_REG_AFE_XP_CTRL, >> + AFE_XP_ER0 | AFE_XP_RESETB); >> + regmap_write(regmap, HDMI_REG_AFE_ISW_CTRL, 0x10); >> + if (pclk_high) >> + regmap_write(regmap, HDMI_REG_AFE_IP_CTRL, >> + AFE_IP_GAINBIT | AFE_IP_RESETB); >> + else >> + regmap_write(regmap, HDMI_REG_AFE_IP_CTRL, >> + AFE_IP_ER0 | AFE_IP_RESETB); >> + >> + /* HDMI software video reset */ >> + regmap_write_bits(regmap, HDMI_REG_SW_RST, SOFTV_RST, SOFTV_RST); >> + fsleep(1000); >> + regmap_write_bits(regmap, HDMI_REG_SW_RST, SOFTV_RST, 0); >> + >> + /* reconfigure LVDS and retry several times in case video is instable */ >> + for (i = 0; i < 3; i++) { >> + ret = regmap_read_poll_timeout(regmap, HDMI_REG_SYS_STATUS, val, >> + val & TXVIDSTABLE, >> + 20000, 500000); >> + if (!ret) { >> + is_stable = true; >> + break; >> + } >> + >> + it6263_lvds_config(it); >> + } >> + >> + if (!is_stable) >> + dev_warn(it->dev, "failed to wait for video stable\n"); >> + >> + /* HDMI AFE reset release and power up */ >> + regmap_write(regmap, HDMI_REG_AFE_DRV_CTRL, 0); >> + >> + regmap_write_bits(regmap, HDMI_REG_GCP, AVMUTE, 0); >> + >> + regmap_write(regmap, HDMI_REG_PKT_GENERAL_CTRL, ENABLE_PKT | REPEAT_PKT); >> +} >> + >> +static enum drm_mode_status >> +it6263_bridge_mode_valid(struct drm_bridge *bridge, >> + const struct drm_display_info *info, >> + const struct drm_display_mode *mode) >> +{ >> + return mode->clock > MAX_PIXEL_CLOCK_KHZ ? MODE_CLOCK_HIGH : MODE_OK; > > rate = drm_hdmi_compute_mode_clock(mode, 8, HDMI_COLORSPACE_RGB); > return bridge->funcs->hdmi_tmds_char_rate_valid(bridge, mode, rate); Will look into the two lines. > >> +} >> + >> +static int it6263_bridge_attach(struct drm_bridge *bridge, >> + enum drm_bridge_attach_flags flags) >> +{ >> + struct it6263 *it = bridge_to_it6263(bridge); >> + int ret; >> + >> + ret = drm_bridge_attach(bridge->encoder, it->next_bridge, bridge, >> + flags | DRM_BRIDGE_ATTACH_NO_CONNECTOR); >> + if (ret < 0) >> + return ret; >> + >> + if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) >> + return 0; >> + >> + it->connector.polled = DRM_CONNECTOR_POLL_CONNECT | >> + DRM_CONNECTOR_POLL_DISCONNECT; >> + > > Please strongly consider dropping this and using drm_bridge_connector in > the host driver. I can't afford to make i.MX8MP imx-lcdif KMS use drm_bridge_connector currently. Maybe the Renesas RZ/G3E SMARC EVK Biju tested v1 patch set with is also not using drm_bridge_connector. I hope we can leave it as-is for now. > >> + ret = drm_connector_init(bridge->dev, &it->connector, >> + &it6263_connector_funcs, >> + DRM_MODE_CONNECTOR_HDMIA); >> + if (ret) >> + return ret; >> + >> + drm_connector_helper_add(&it->connector, >> + &it6263_connector_helper_funcs); >> + drm_connector_attach_encoder(&it->connector, bridge->encoder); >> + >> + return 0; >> +} >> + >> +static enum drm_connector_status it6263_bridge_detect(struct drm_bridge *bridge) >> +{ >> + struct it6263 *it = bridge_to_it6263(bridge); >> + >> + return it6263_detect(it); >> +} >> + >> +static const struct drm_edid * >> +it6263_bridge_edid_read(struct drm_bridge *bridge, >> + struct drm_connector *connector) >> +{ >> + struct it6263 *it = bridge_to_it6263(bridge); >> + >> + return drm_edid_read_custom(connector, it6263_read_edid, it); >> +} >> + >> +static u32 * >> +it6263_bridge_atomic_get_input_bus_fmts(struct drm_bridge *bridge, >> + struct drm_bridge_state *bridge_state, >> + struct drm_crtc_state *crtc_state, >> + struct drm_connector_state *conn_state, >> + u32 output_fmt, >> + unsigned int *num_input_fmts) >> +{ >> + struct it6263 *it = bridge_to_it6263(bridge); >> + u32 *input_fmts; >> + >> + *num_input_fmts = 0; >> + >> + if (it->lvds_link_num_dlanes == 3) >> + return NULL; >> + >> + input_fmts = kmalloc(sizeof(*input_fmts), GFP_KERNEL); >> + if (!input_fmts) >> + return NULL; >> + >> + input_fmts[0] = MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA; >> + *num_input_fmts = 1; >> + >> + return input_fmts; >> +} >> + >> +static const struct drm_bridge_funcs it6263_bridge_funcs = { >> + .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, >> + .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, >> + .atomic_reset = drm_atomic_helper_bridge_reset, >> + .attach = it6263_bridge_attach, >> + .mode_valid = it6263_bridge_mode_valid, >> + .atomic_disable = it6263_bridge_atomic_disable, >> + .atomic_enable = it6263_bridge_atomic_enable, >> + .atomic_check = it6263_bridge_atomic_check, >> + .detect = it6263_bridge_detect, >> + .edid_read = it6263_bridge_edid_read, >> + .atomic_get_input_bus_fmts = it6263_bridge_atomic_get_input_bus_fmts, >> +}; >> + >> +static int it6263_probe(struct i2c_client *client) >> +{ >> + struct device *dev = &client->dev; >> + struct it6263 *it; >> + int ret; >> + >> + it = devm_kzalloc(dev, sizeof(*it), GFP_KERNEL); >> + if (!it) >> + return -ENOMEM; >> + >> + it->dev = dev; >> + it->hdmi_i2c = client; >> + >> + it->hdmi_regmap = devm_regmap_init_i2c(client, >> + &it6263_hdmi_regmap_config); >> + if (IS_ERR(it->hdmi_regmap)) >> + return dev_err_probe(dev, PTR_ERR(it->hdmi_regmap), >> + "failed to init I2C regmap for HDMI\n"); >> + >> + it->reset_gpio = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_LOW); >> + if (IS_ERR(it->reset_gpio)) >> + return dev_err_probe(dev, PTR_ERR(it->reset_gpio), >> + "failed to get reset gpio\n"); >> + >> + ret = devm_regulator_bulk_get_enable(dev, ARRAY_SIZE(it6263_supplies), >> + it6263_supplies); >> + if (ret) >> + return dev_err_probe(dev, ret, "failed to get power supplies\n"); >> + >> + ret = it6263_parse_dt(it); >> + if (ret) >> + return ret; >> + >> + it6263_hw_reset(it); >> + >> + ret = it6263_lvds_set_i2c_addr(it); >> + if (ret) >> + return dev_err_probe(dev, ret, "failed to set I2C addr\n"); >> + >> + it->lvds_i2c = devm_i2c_new_dummy_device(dev, client->adapter, >> + LVDS_INPUT_CTRL_I2C_ADDR); >> + if (IS_ERR(it->lvds_i2c)) >> + dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), >> + "failed to allocate I2C device for LVDS\n"); >> + >> + it->lvds_regmap = devm_regmap_init_i2c(it->lvds_i2c, >> + &it6263_lvds_regmap_config); >> + if (IS_ERR(it->lvds_regmap)) >> + return dev_err_probe(dev, PTR_ERR(it->lvds_regmap), >> + "failed to init I2C regmap for LVDS\n"); >> + >> + it6263_lvds_config(it); >> + it6263_hdmi_config(it); >> + >> + i2c_set_clientdata(client, it); >> + >> + it->bridge.funcs = &it6263_bridge_funcs; >> + it->bridge.of_node = dev->of_node; >> + it->bridge.ops = DRM_BRIDGE_OP_EDID | DRM_BRIDGE_OP_DETECT; > > | DRM_BRIDGE_OP_HDMI Will take a look at this. > > BTW: No HPD IRQ support? IT6263 chip doesn't support HPD IRQ. > >> + it->bridge.type = DRM_MODE_CONNECTOR_HDMIA; >> + drm_bridge_add(&it->bridge); > > devm_drm_bridge_add() Will try to use this. Thanks. > >> + >> + return 0; >> +} >> + >> +static void it6263_remove(struct i2c_client *client) >> +{ >> + struct it6263 *it = i2c_get_clientdata(client); >> + >> + drm_bridge_remove(&it->bridge); > > Remove, it becomes unused with the devm_ function. > >> +} >> + >> +static const struct of_device_id it6263_of_match[] = { >> + { .compatible = "ite,it6263", }, >> + { } >> +}; >> +MODULE_DEVICE_TABLE(of, it6263_of_match); >> + >> +static const struct i2c_device_id it6263_i2c_ids[] = { >> + { "it6263", 0 }, >> + { } >> +}; >> +MODULE_DEVICE_TABLE(i2c, it6263_i2c_ids); >> + >> +static struct i2c_driver it6263_driver = { >> + .probe = it6263_probe, >> + .remove = it6263_remove, >> + .driver = { >> + .name = "it6263", >> + .of_match_table = it6263_of_match, >> + }, >> + .id_table = it6263_i2c_ids, >> +}; >> +module_i2c_driver(it6263_driver); >> + >> +MODULE_DESCRIPTION("ITE Tech. Inc. IT6263 LVDS/HDMI bridge"); >> +MODULE_AUTHOR("Liu Ying "); >> +MODULE_LICENSE("GPL"); >> -- >> 2.34.1 >> > -- Regards, Liu Ying