Received-SPF: pass (google.com: domain of linux-kernel+bounces-208375-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Robert Foss <rfoss@kernel.org>
To: dri-devel@lists.freedesktop.org, Adam Ford <aford173@gmail.com>
Cc: Jagan Teki <jagan@amarulasolutions.com>, Michael Tretter <m.tretter@pengutronix.de>,
 Thomas Zimmermann <tzimmermann@suse.de>, Marek Szyprowski <m.szyprowski@samsung.com>, Marek Vasut <marex@denx.de>,
 Daniel Vetter <daniel@ffwll.ch>, aford@beaconembedded.com, Jonas Karlman <jonas@kwiboo.se>,
 Frieder Schrempf <frieder.schrempf@kontron.de>, Andrzej Hajda <andrzej.hajda@intel.com>,
 Inki Dae <inki.dae@samsung.com>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, linux-kernel@vger.kernel.org,
 Neil Armstrong <neil.armstrong@linaro.org>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
 David Airlie <airlied@gmail.com>, Maxime Ripard <mripard@kernel.org>, Marco Felsch <m.felsch@pengutronix.de>,
 Jernej Skrabec <jernej.skrabec@gmail.com>
In-Reply-To: <20240601144103.198299-1-aford173@gmail.com>
References: <20240601144103.198299-1-aford173@gmail.com>
Subject: Re: [PATCH V3 1/2] drm/bridge: samsung-dsim: Set P divider based on min/max of fin pll
Message-Id: <171803008513.465712.14703974733678728383.b4-ty@kernel.org>
Date: Mon, 10 Jun 2024 16:34:45 +0200
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

On Sat, 1 Jun 2024 09:41:01 -0500, Adam Ford wrote:
> The P divider should be set based on the min and max values of
> the fin pll which may vary between different platforms.
> These ranges are defined per platform, but hard-coded values
> were used instead which resulted in a smaller range available
> on the i.MX8M[MNP] than what was possible.
> 
> As noted by Frieder, there are descripencies between the reference
> manuals of the Mini, Nano and Plus, so I reached out to my NXP
> rep and got the following response regarding the varing notes
> in the documentation.
> 
> [...]

Applied, thanks!

[1/2] drm/bridge: samsung-dsim: Set P divider based on min/max of fin pll
      (no commit info)
[2/2] drm/bridge: samsung-dsim: Fix porch calcalcuation rounding
      (no commit info)


Rob