From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtpout-02.galae.net (smtpout-02.galae.net [185.246.84.56]) (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 B19FE443315; Tue, 1 Sep 2026 07:09:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.246.84.56 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788246566; cv=none; b=DIMPbHPGPKjFoH3NUcFlwLHz4VzAt00rdq4jmMiFUwxUpN+JBxWalZSr8S3TQjJSy60pS5JK5lmB/9keFxOknnGssRCCjNvgYLgTRHujz+ACwvaju9RHjznQuAKwwmnfE8vdWDQjcSkun186S6phqpcF9pxVzwQ8AxqimT8KIKA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788246566; c=relaxed/simple; bh=vtIrC8TakK09zNf74g0t4AkIV0EtG0GErTao6iHQiyg=; h=Mime-Version:Content-Type:Date:Message-Id:Subject:Cc:To:From: References:In-Reply-To; b=m6SfPg2yphgG6rILcMOkudWAKrsbPY/muxyxM2QjpOQz4q37Qeo5QmGcgj/gcYUhI+tvctluip4bahT+t4cm6ulOeGL+amPrRCA11hOkGKDzXEDhBeM1rVT3lrlXXtnotQPiJA4CYIJVJqr/RbLwq3wxgEqFOoROZszAspeuKv8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=S46tbBRs; arc=none smtp.client-ip=185.246.84.56 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="S46tbBRs" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-02.galae.net (Postfix) with ESMTPS id E03E61A1851; Tue, 1 Sep 2026 07:09:20 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id A96676053C; Tue, 1 Sep 2026 07:09:20 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 5A05911C78A73; Tue, 1 Sep 2026 09:09:11 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1788246559; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=azbA4EriyIuSSgA9MK6fPCdkyDzmm+8sSS/z2y87Z5U=; b=S46tbBRspM5s0aKsHNokZfOfNb8cl+p0i1dapuhgbhX4RFRQMvUxoA/OAEfbFKnAZcy9Tw uMaannzJwpY0e2mWLI2p/xJZvIYWw3AumCHdFxIYH/hj9/1gtmOO0I0/FQTJ8T1uAUxJ4M 1b84GsLHAKA0HJZUbC5mC6I5YxtIb5szVmtzIUHnhjy0O+X5yf/06Iz08gkClHtM0mAQMT g/q42mKkHayvwWJEa1m0GOAFR5Z+XApPMDPXglD81DMNBYcE+H81NQvIsQdUrCQbGxMRay 0u2b73EcCt/VW/d35Wf4J1AHj7+JfSFQQGtiL645y4oPLA1cAJ7wlJ/qnXgdcg== Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 01 Sep 2026 09:09:11 +0200 Message-Id: Subject: Re: [PATCH v11 2/2] drm/bridge: Add Lontium LT9611C(EX/UXD) MIPI DSI to HDMI driver Cc: "Sunyun Yang" , "Andrzej Hajda" , "Neil Armstrong" , "Robert Foss" , "Laurent Pinchart" , "Jonas Karlman" , "Jernej Skrabec" , "Maarten Lankhorst" , "Maxime Ripard" , "Thomas Zimmermann" , "David Airlie" , "Simona Vetter" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Vinod Koul" , , , , , "Jessica Zhang" , "Dmitry Baryshkov" To: "Mohit Dsor" , "Luca Ceresoli" From: "Luca Ceresoli" X-Mailer: aerc 0.22.0 References: <20260824-lt9611c-v7-v11-0-eb4a79cb597c@oss.qualcomm.com> <20260824-lt9611c-v7-v11-2-eb4a79cb597c@oss.qualcomm.com> <178766696083.117435.16339828039068931624.b4-review@b4> In-Reply-To: X-Last-TLS-Session-Version: TLSv1.3 Hello Mohit, On Mon Aug 31, 2026 at 8:38 PM CEST, Mohit Dsor wrote: [...] >> > +#include >> >> Unused include? > Using this header for of_drm_get_bridge_by_endpoint of_drm_get_bridge_by_endpoint() is declared in drm_bridge.h. >> > +static enum drm_mode_status >> > +lt9611c_hdmi_tmds_char_rate_valid(const struct drm_bridge *bridge, >> ^^^^ >> Out of curiosity, what does "char" means here? > "char" =3D character =E2=80=94 TMDS (Transition Minimized Differential Si= gnaling) transmits data using 8b/10b encoding, where each > encoded 10-bit symbol is called a "character". The TMDS character rate = is the number of these symbols transmitted per second > per lane, which equals the TMDS clock frequency. Ah, OK, thanks for the explanation. >> Also, I _think_ lt9611c_regulator_init() could just >> devm_regulator_bulk_get_enable() to do both things at once, but I'm not >> sure that would be compatible with PM. If it's safe it would simplify th= e >> code quite a lot, and also allow using devm_drm_bridge_add() below, maki= ng >> the remove function almost empty. > regulator_bulk_disable/enable are called in suspend/resume and in remove = =E2=80=94 so devm_regulator_bulk_get_enable is not suitable > here OK, I was not sure about that, thanks for checking. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com