From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.steuer-voss.de (mail.steuer-voss.de [85.183.69.95]) (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 12D1A360 for ; Sat, 7 Dec 2024 06:30:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=85.183.69.95 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733553035; cv=none; b=FhDdZ8+ln81/ijm48tNQskwWHfJuJzLbEFV+1e+VC8ecTyfbqIp5HCrxrQfnFTH2oMfuhUU+7X+Y8bwCNQlmfTkUIYZ35gvpGt3fLnFKNwFryTrqZ78DD6uo/aCxuxcYAQjGxUXfmqwrx0pMMKrUooMMZeJEPOTQpVv5snY/ieg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733553035; c=relaxed/simple; bh=Vin7YR2E9FVuXmPE4I2eJ+mPqvNc6M0xZrQuwYAwbdA=; h=MIME-Version:Date:From:To:Cc:Subject:In-Reply-To:References: Message-ID:Content-Type; b=Wor5c5rQ+qAq6nCnUj+Ky1omIR6YLN72nLq1wGIQ7qLhLTNAAhBEnKgEyYaDAKzR+1HAcd4lMBCUAC1uGAnXqUaK0cSArkgxGXadaTzGJiODtvM6CT7TkfQKqDWondWWEmO1jR+FGjwQshrmoodMv6jYXhnr/VSCFZkXwwQoaxc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=vosn.de; spf=pass smtp.mailfrom=vosn.de; arc=none smtp.client-ip=85.183.69.95 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=vosn.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=vosn.de X-Virus-Scanned: Debian amavisd-new at mail.steuer-voss.de Received: from mail.steuer-voss.de (localhost [127.0.0.1]) by mail.steuer-voss.de (Postfix) with ESMTP id 5FDDF22D4; Sat, 7 Dec 2024 07:30:27 +0100 (CET) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Date: Sat, 07 Dec 2024 07:30:27 +0100 From: Nikolaus Voss To: Miquel Raynal Cc: Alexander Stein , Liu Ying , Luca Ceresoli , Fabio Estevam , Marek Vasut , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , nikolaus.voss@haag-streit.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] drm: bridge: fsl-ldb: fixup mode on freq mismatch In-Reply-To: <87zfl8oqeg.fsf@bootlin.com> References: <20241203191111.47B56F7@mail.steuer-voss.de> <87zfl8oqeg.fsf@bootlin.com> User-Agent: Roundcube Webmail/1.5.0 Message-ID: X-Sender: nv@vosn.de Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Hi Miquèl, On 06.12.2024 15:08, Miquel Raynal wrote: > On 03/12/2024 at 20:09:52 +01, Nikolaus Voss wrote: > >> LDB clock has to be a fixed multiple of the pixel clock. > > Not only, IIUC it also needs to be synchronized, ie. share the same > source. > >> As LDB and pixel clock are derived from different clock sources >> (at least on imx8mp), > > Wait, what? I am sorry but that is not at all recommended, both should > come from video_pll1 which the de-facto versatile PLL to use, no? Am I > missing something here? No, I was wrong :-). I've corrected the log text in the v3. -- Nikolaus Voss