From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from metis.whiteo.stw.pengutronix.de (metis.whiteo.stw.pengutronix.de [185.203.201.7]) (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 F2CC133F366 for ; Fri, 7 Nov 2025 21:07:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.203.201.7 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762549640; cv=none; b=dFg+SZx9jvNYE6kzmkl3lUCItdPx70Ol3Cv6CMQT0cbQiyoOXF+pfdF7BhwsID5l0yhrSbhBK/9gibAmYpGJgbmLDjmrbHywtflF6a9CLMK3/UoRAgJUpgAlKvywecRkhTK/pBH/C8/WL185+pHHixKxGK8bvfGn0AtMgno0Dys= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762549640; c=relaxed/simple; bh=WYD4+nVqmn8xdO6/m0G1Di/RyLfcikFbr473qTzxXm4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=jyG7pUllmjZfIZiTxLgSfLfPsutDBrwYqxPUx297CyKDsS2epOsp2yrIauv7ozUNxpipF6+eQEV688GYw6qBx0E0O/tgb0Ydn8RxKAnpSfdiJzmcBUZGYvPOE2At7cZwvE69+Re2+wrBGamQmRufJH2gndNtqwPRHitKCb6Kqnc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de; spf=pass smtp.mailfrom=pengutronix.de; arc=none smtp.client-ip=185.203.201.7 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pengutronix.de Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1vHTeG-0001aK-Mg; Fri, 07 Nov 2025 22:05:32 +0100 Message-ID: Date: Fri, 7 Nov 2025 22:05:30 +0100 Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RESEND v4 1/3] dt-bindings: lcdif: Document a imx6sx-lcdif fallback To: Fabio Estevam , Liu Ying Cc: marex@denx.de, stefan@agner.ch, airlied@gmail.com, simona@ffwll.ch, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, shawnguo@kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org References: <20251013205155.1187947-1-festevam@gmail.com> <4bd512fd-b3df-484a-8a04-a1ed066c42fa@nxp.com> <37b8d968-a725-404e-8fd4-84c2a0bb464c@nxp.com> Content-Language: en-US From: Ahmad Fatoum In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-SA-Exim-Connect-IP: 2a0a:edc0:0:900:1d::77 X-SA-Exim-Mail-From: a.fatoum@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: imx@lists.linux.dev Hi, On 04.11.25 21:44, Fabio Estevam wrote: > On Mon, Oct 20, 2025 at 11:53 PM Liu Ying wrote: >> On 10/20/2025, Fabio Estevam wrote: >>> On Tue, Oct 14, 2025 at 1:12 AM Liu Ying wrote: >>> >>>> Strictly speaking, I don't think i.MX6SX LCDIF is compatible with i.MX28 LCDIF >>>> because at least i.MX28 LCDIF has the version and debug{0,1,2} registers(at >>>> 0x1c0, 0x1d0, 0x1e0 and 0x1f0) while i.MX6SX LCDIF hasn't. Thanks for pointing this out. In my opinion, these registers are auxiliary and don't really change the compatibility situation as a functional driver can be written without their use, evidenced by the Linux driver doing just fine without using these registers. >>> There are some DT users, such as Barebox that matches against >>> fsl,imx28-lcdif, so we cannot remove it. >> >> Hmmm, it looks like software projects like Barebox don't really follow this DT >> binding. Is it possible to fix Barebox to avoid changing this DT binding by >> this patch? I'm assuming that Uboot has already been fixed. > > What do you think? I am sorry my prior feedback ended up stalling this series. There is a lot of regressions happening due to upstream DT changes and I am just trying to raise awareness. Another example I stumbled over today: https://lore.kernel.org/all/91f764ab-bec1-4791-b01b-3ba0803ce8f8@pengutronix.de/ I have submitted a patch[1] to barebox adding explicit i.MX6 SoloX support, so, having expressed my opinion above, please proceed as you see fit. [1]: https://lore.barebox.org/barebox/20251107210033.2229781-3-a.fatoum@barebox.org/ Cheers, Ahmad > > Thanks > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |