From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EF3D0C77B76 for ; Sun, 16 Apr 2023 22:13:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229991AbjDPWNn (ORCPT ); Sun, 16 Apr 2023 18:13:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37050 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229446AbjDPWNm (ORCPT ); Sun, 16 Apr 2023 18:13:42 -0400 Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8CFA1211C; Sun, 16 Apr 2023 15:13:41 -0700 (PDT) Received: from [127.0.0.1] (p578adb1c.dip0.t-ipconnect.de [87.138.219.28]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: marex@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id A167C85D46; Mon, 17 Apr 2023 00:13:38 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1681683220; bh=7PbI+KkaZZTLDdNEdsw5NBsMDYvFroQp2MXTg73mOGg=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=ZhDuNUxN0+2P/RHekLgAnF5qlsp1A87kFpzFMpmbmp41IS9MMEMGyOqojc8s748ZP XGjFD+cIhvShS2Zk7a2HI2lQ27ZhrQsQCe98sE0ojdnwZwdnFTqGL/jzWBC0OujrjK NAeJlQmIom2f1mo4IY0l3URQef1vvTAyIbCIZxMgO1m7XPQ9HK0nSH61dfkfiIOJLN E/T7WImkN94KuKPwH/o2tSQrlN2As1EzxtzyHvv0fQbUfeHtKDQKIPdjDiSRv6iSBA W4OSgbe3R+wBRnfSHlIwHBFTUnpMnTZq7omK4NJ1Pdron0TF8WTvBlGt+Btw/mPciC je+xh+pCueseg== Message-ID: <807aa6c6-bbea-abcc-172d-17e22d1a3988@denx.de> Date: Mon, 17 Apr 2023 00:13:38 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [PATCH 5/6] drm: bridge: samsung-dsim: Support non-burst mode Content-Language: en-US To: Adam Ford , dri-devel@lists.freedesktop.org Cc: m.szyprowski@samsung.com, aford@beaconembedded.com, Rob Herring , Krzysztof Kozlowski , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Inki Dae , Jagan Teki , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Frieder Schrempf , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20230415104104.5537-1-aford173@gmail.com> <20230415104104.5537-5-aford173@gmail.com> From: Marek Vasut In-Reply-To: <20230415104104.5537-5-aford173@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 4/15/23 12:41, Adam Ford wrote: > The high-speed clock is hard-coded to the burst-clock > frequency specified in the device tree. However, when > using devices like certain bridge chips without burst mode > and varying resolutions and refresh rates, it may be > necessary to set the high-speed clock dynamically based > on the desired pixel clock for the connected device. The link rate negotiation should happen internally between the nearest bridge and DSIM, so please add that to DRM core instead of hacking around it by tweaking the HS clock again.