Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Liu Ying <victor.liu@nxp.com>
To:     dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Cc:     andrzej.hajda@intel.com, narmstrong@baylibre.com,
        robert.foss@linaro.org, Laurent.pinchart@ideasonboard.com,
        jonas@kwiboo.se, jernej.skrabec@gmail.com, airlied@linux.ie,
        daniel@ffwll.ch, sam@ravnborg.org, marex@denx.de, linux-imx@nxp.com
Subject: [PATCH 1/3] drm/bridge: fsl-ldb: Fix mode clock rate validation
Date:   Fri,  1 Jul 2022 14:56:32 +0800
Message-Id: <20220701065634.4027537-2-victor.liu@nxp.com>
In-Reply-To: <20220701065634.4027537-1-victor.liu@nxp.com>
References: <20220701065634.4027537-1-victor.liu@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?yyssFHdRQCQZXUkYG27Nfc8fVqCgG1msysXl5SmwfcP2yFhnLA7vc/aBhuSi?=
 =?us-ascii?Q?8BIComhEmdvxBcwTWw0upMctesJSnQ/wKkvC/WWPWNzFhAhaILWKVfhQQxD2?=
 =?us-ascii?Q?xnMtbieC9IJpNCcQnhd/pq/vlyV0/Y9apXkO2SSIsvETU99hbDg4LYHcWx0H?=
 =?us-ascii?Q?UaddWzKgWtOqVWMy7iFD8phMcQ5/tVG9KdXqOeoSywJCNtQDVX3ch+woKFDm?=
 =?us-ascii?Q?lfkGZHhVMKcPrK9dzxKCXhOPgVBn1v2y5oJzyJAlqPXXXkhXSP3W7NrEpIcA?=
 =?us-ascii?Q?vJlyks+/YisQ0S0qD44OcpdydIsovqWhiFHcb1AahkIDW2YZUBmhpTpnTDlo?=
 =?us-ascii?Q?SOia64KiPRqk3sQRrXAHzOOw2ug4ldKn5mjF1mxLb4CegkYLcw7U6S508ZXC?=
 =?us-ascii?Q?wwQ1czfd7DG4bpwjULxoJazPskpYuEQQ51U8jF/1NAtw3hNNLxPnuphhgAK2?=
 =?us-ascii?Q?Yn05fX6r4tXNM3zRxQOtwOCwX6FdZOvRI4s3qold3+ciUeNGsE0lziYCFJj6?=
 =?us-ascii?Q?0mxFjvNrAhPEd70nFhKGPKjBRxrnrAFOkjwIAoqjAh3SA9w3NhgKMZycfIt6?=
 =?us-ascii?Q?CboGPJuEyxI3ICmSHZK08xlOoVJpOgbfKgxaMM2gSa3UwCxYWxe1cAuCnltK?=
 =?us-ascii?Q?oMe1btU+Pxx5PCnmIeIVeIMoYjhYv6ZnMO1ZBk9sdqBxQWhssdckzUcNXFqW?=
 =?us-ascii?Q?3okTSMFLuuAWSMIM6EiKGDqxd2eeF3H5+khHkGCPvHECgAo5EpaT2tY475/G?=
 =?us-ascii?Q?BA7l46iYHz6UY38qZuFsPYSQayx4pY80QMO/peLO/CJIpXHHfGDhcdUZxlIG?=
 =?us-ascii?Q?/Vv1ciYoibOJiaTVn/6+iJA8XBye2Rl/f4OhaMku7XeyBxThT9M3E0zKBn0B?=
 =?us-ascii?Q?oqkDEeHcvfUk4o+MGaqYtQPjjAzLOmYKHy0K9ZBNeU3NdrVVsG2T/xwkKVPt?=
 =?us-ascii?Q?I8g9xBRC6TIwpWOpGUXBQIfEamS9UxtiM4b9mK84ohcgoMdwtgTN9MSAuXIV?=
 =?us-ascii?Q?7FWI5u/SQi8p1zM39aroJ2TsYlkf0O6kt7d3fxFrrLq/+tiYmkpc7Qib8KS2?=
 =?us-ascii?Q?vwS56ogOdoSKMMTM3uHdIdJLBHOIECSy5Jsl3yJ5kVeGgQcC1irl3grdQvBv?=
 =?us-ascii?Q?Za2arhUPfKKdZR/Jc23XbC2cZkx6veJXw/bIv2yqYANrSJkU8O4pr3yiB2s9?=
 =?us-ascii?Q?xb6WhwTlr8KRNrgmKG/I3BceZTFVD+KUijZsZ2FBSvG6fdUxjtsGIxGlzerT?=
 =?us-ascii?Q?P9osLTN2YafRPuu9qXbiMbsjqzbCJdnCXoC82kGkMoksWaJkr9/xZBGjP4tt?=
 =?us-ascii?Q?9MToKGU7/+XaZQDyGl0Z6roaInJb0ZanyuDbyOQ2G1zjrQs3wBDzR/bpzq9L?=
 =?us-ascii?Q?m5xv7FII991m7mcQJ8XhBOfeUDQsbGzhjklukauMXLgV2McVO3WJyaAxGIFs?=
 =?us-ascii?Q?gyCe4Na9nzE43OGNZAZk5hKr46cz11fGojTXpL+7XmyDT9lXq0sJ+Q4qiWug?=
 =?us-ascii?Q?F/PmmQuTh2BNbwBD2cp8ejXa4/cmR7q4L/LwzFIs0EJ9/TEumJ1xTJEggjQJ?=
 =?us-ascii?Q?Jyo/SL2oMEv+fEE2QmQDh7VypaZNBgm8I7o7QJPV?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a2001636-0320-4bfd-9143-08da5b2e9c06
X-MS-Exchange-CrossTenant-AuthSource: AM7PR04MB7046.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Jul 2022 06:54:57.2364
 (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: yU5CXkLL10NnPkdNNbq27X7i8bXiF8F0HaQDA2/kjPe/PN98DgzU7b8YFkNkNpIPohgkRlc4fkR7P5D5mO9YVw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB5762
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

With LVDS dual link, up to 160MHz mode clock rate is supported.
With LVDS single link, up to 80MHz mode clock rate is supported.
Fix mode clock rate validation by swapping the maximum mode clock
rates of the two link modes.

Fixes: 463db5c2ed4a ("drm: bridge: ldb: Implement simple Freescale i.MX8MP LDB bridge")
Cc: Andrzej Hajda <andrzej.hajda@intel.com>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Robert Foss <robert.foss@linaro.org>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Jernej Skrabec <jernej.skrabec@gmail.com>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Marek Vasut <marex@denx.de>
Cc: NXP Linux Team <linux-imx@nxp.com>
Signed-off-by: Liu Ying <victor.liu@nxp.com>
---
 drivers/gpu/drm/bridge/fsl-ldb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/fsl-ldb.c b/drivers/gpu/drm/bridge/fsl-ldb.c
index b2675c769a55..3cb3b310e283 100644
--- a/drivers/gpu/drm/bridge/fsl-ldb.c
+++ b/drivers/gpu/drm/bridge/fsl-ldb.c
@@ -233,7 +233,7 @@ fsl_ldb_mode_valid(struct drm_bridge *bridge,
 {
 	struct fsl_ldb *fsl_ldb = to_fsl_ldb(bridge);
 
-	if (mode->clock > (fsl_ldb->lvds_dual_link ? 80000 : 160000))
+	if (mode->clock > (fsl_ldb->lvds_dual_link ? 160000 : 80000))
 		return MODE_CLOCK_HIGH;
 
 	return MODE_OK;
-- 
2.25.1