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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1B0F1C87FCF for ; Wed, 13 Aug 2025 05:12:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:Message-ID:Date:References:In-Reply-To:Subject:Cc: To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=78z1kuRSWutMg/Hy75bF++4O7lZJqDUQoV9YvjD+Wtw=; b=vO0wAj+d/nPhle12ySExDKJ/W9 zMzlV5IaKPvzwk8pdfzL61gazJ5J4RwR7ygIpwr1rCg51qcGxHkbWCwY2twdhYDc/Ag+imsAx402k JZ5mOUlp4rEFOR/ymp4lhSvBYj9LgsmmcerVK8dvRCYvWEjvfCQdhbJIXp7NMv2Nglb7bOSwCicoS NVRFlRSOB1Sd4j2ZaEeEDvbUdLqCewsVOaSG5UEW8xkNZNTApMIhcxaz7I1AnWBJ1aNhoJBpl69Vu d10UG3mf8avhYIPEwOeSCnoC6n3fdb2aOje02290/knpYjReclG1yaSLq7KsEb0MzDId1SZ7lwe7j c2a/ed6w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1um3n5-0000000CdjQ-2Zri; Wed, 13 Aug 2025 05:12:47 +0000 Received: from ni.piap.pl ([195.187.100.5]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1um3kX-0000000Cdcx-0ykL; Wed, 13 Aug 2025 05:10:11 +0000 Received: from t19.piap.pl (OSB1819.piap.pl [10.0.9.19]) by ni.piap.pl (Postfix) with ESMTPS id C17F5C3E4DE7; Wed, 13 Aug 2025 07:10:03 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 ni.piap.pl C17F5C3E4DE7 From: =?utf-8?Q?Krzysztof_Ha=C5=82asa?= To: Adam Ford Cc: Stefan Klug , Laurent Pinchart , Dafna Hirschfeld , Heiko Stuebner , Paul Elder , Jacopo Mondi , Ondrej Jirman , linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: FYI: i.MX8MP ISP (RKISP1) MI registers corruption: resolved In-Reply-To: (Adam Ford's message of "Tue, 12 Aug 2025 13:22:54 -0500") References: <175308758352.3134829.9472501038683860006@localhost> <175326599663.2811177.16620980968274114885@localhost> <175344176070.2811177.10693943493658922992@localhost> <20250812103243.GK30054@pendragon.ideasonboard.com> <175501095338.74722.11604545949710100799@localhost> Date: Wed, 13 Aug 2025 07:10:03 +0200 Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250812_221009_437430_E317181C X-CRM114-Status: GOOD ( 11.47 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Adam, Adam Ford writes: > I was reading through the data sheet (not the reference manual), and > it lists a few limitations for the clocks: > > For single Camera, MIPI CSI 1 can support up to 400/500 MHz pixel > clock in the Nominal/Overdrive mode. > For single Camera, MIPI CSI 2 can support up to 277 MHz pixel clock. > For dual Camera, both MIPI CSI can support up to 266 MHz pixel clock. > > If you're running dual cameras, it sounds like you're capped at 266 > MHz regardless of whether or not you're in overdrive or nominal. Right. But these are pixel clocks, not ISP clock. At least theoretically entirely different stuff. For example, I'm using (at most) two IMX290/462 1920x1080p60 sensors in 12-bit (Bayer) mode, resulting in: - 148.5 MHz pixel clocks - 1782 Mb/s total bandwidth (for each sensor - 12 bits) - 445.5 Mb/s per lane (each sensor uses 4 MIPI lanes) - 222.75 MHz MIPI clocks (MIPI uses DDR, the double data rate) So I'm well within specs. Though I don't know if the people writing the datasheets did really mean what they wrote, or maybe they meant ISP core clocks as well. Why? Because ISP blocks (and all such logic blocks) require a certain number of their core clocks for a signal rate (in this case, a pixel rate). And maybe what they specified in the datasheets was a calculation of this (unpublished?) required pixel/ISP clock ratio and the max ISP clock rate (specified incorrectly at 400/500 MHz). Anyway, this is only a possibility. Another one is that ISP2 simply cannot reliably run at 500 MHz, for some "analog" reason, design bug etc. > My understanding is that the imx8mp.dtsi is pre-configured for > overdrive mode, so if you need to run the ISP in nominal, the clock > updates should go into imx8mp-nominial.dtsi. It seems my modules are setup for overdrive mode. --=20 Krzysztof "Chris" Ha=C5=82asa Sie=C4=87 Badawcza =C5=81ukasiewicz Przemys=C5=82owy Instytut Automatyki i Pomiar=C3=B3w PIAP Al. Jerozolimskie 202, 02-486 Warszawa 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 94946C87FCF for ; Wed, 13 Aug 2025 05:12:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=16TG/L3oeMkKztoMYL2vCDqWYdWB3qO0y2p9UTAgyks=; b=PCQAJhBWCRpI/r pzat938DKT0ExABw5wS7V/z7E4yh9dt/1OZveMXrACINAcx4QaGTnU5ADG1X2MqInkJ0dVivZ89R1 mS9+y4/3elZidy9zq7l5iveRsVUw2w4QZ1jRUy7qV31zu3aA+WIfzJcMJobhDteFNIVLrVo7nNz+U kTPpRCnDNun2gezoi4dLVsZjWjhdxWSDgZJB61ws8euSrSsoI2E9gOUiL+oxXtvcZ88gn7G4GjZ08 iAuR7TCnRnNfZjp3tHUzUK/Gvm90UIap1SWo5X/sOEy7PuVme/f8sHITPaehLbhrVMLLRr5S5ZlpE 6vKOjO3Lt1atlf1cA+sQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1um3n6-0000000Cdjg-0arO; Wed, 13 Aug 2025 05:12:48 +0000 Received: from ni.piap.pl ([195.187.100.5]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1um3kX-0000000Cdcx-0ykL; Wed, 13 Aug 2025 05:10:11 +0000 Received: from t19.piap.pl (OSB1819.piap.pl [10.0.9.19]) by ni.piap.pl (Postfix) with ESMTPS id C17F5C3E4DE7; Wed, 13 Aug 2025 07:10:03 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 ni.piap.pl C17F5C3E4DE7 From: =?utf-8?Q?Krzysztof_Ha=C5=82asa?= To: Adam Ford Cc: Stefan Klug , Laurent Pinchart , Dafna Hirschfeld , Heiko Stuebner , Paul Elder , Jacopo Mondi , Ondrej Jirman , linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: FYI: i.MX8MP ISP (RKISP1) MI registers corruption: resolved In-Reply-To: (Adam Ford's message of "Tue, 12 Aug 2025 13:22:54 -0500") References: <175308758352.3134829.9472501038683860006@localhost> <175326599663.2811177.16620980968274114885@localhost> <175344176070.2811177.10693943493658922992@localhost> <20250812103243.GK30054@pendragon.ideasonboard.com> <175501095338.74722.11604545949710100799@localhost> Date: Wed, 13 Aug 2025 07:10:03 +0200 Message-ID: MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250812_221009_437430_E317181C X-CRM114-Status: GOOD ( 11.47 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org SGkgQWRhbSwKCkFkYW0gRm9yZCA8YWZvcmQxNzNAZ21haWwuY29tPiB3cml0ZXM6Cgo+IEkgd2Fz IHJlYWRpbmcgdGhyb3VnaCB0aGUgZGF0YSBzaGVldCAobm90IHRoZSByZWZlcmVuY2UgbWFudWFs KSwgYW5kCj4gaXQgbGlzdHMgYSBmZXcgbGltaXRhdGlvbnMgZm9yIHRoZSBjbG9ja3M6Cj4KPiBG b3Igc2luZ2xlIENhbWVyYSwgTUlQSSBDU0kgMSBjYW4gc3VwcG9ydCB1cCB0byA0MDAvNTAwIE1I eiBwaXhlbAo+IGNsb2NrIGluIHRoZSBOb21pbmFsL092ZXJkcml2ZSBtb2RlLgo+IEZvciBzaW5n bGUgQ2FtZXJhLCBNSVBJIENTSSAyIGNhbiBzdXBwb3J0IHVwIHRvIDI3NyBNSHogcGl4ZWwgY2xv Y2suCj4gRm9yIGR1YWwgQ2FtZXJhLCBib3RoIE1JUEkgQ1NJIGNhbiBzdXBwb3J0IHVwIHRvIDI2 NiBNSHogcGl4ZWwgY2xvY2suCj4KPiBJZiB5b3UncmUgcnVubmluZyBkdWFsIGNhbWVyYXMsIGl0 IHNvdW5kcyBsaWtlIHlvdSdyZSBjYXBwZWQgYXQgMjY2Cj4gTUh6IHJlZ2FyZGxlc3Mgb2Ygd2hl dGhlciBvciBub3QgeW91J3JlIGluIG92ZXJkcml2ZSBvciBub21pbmFsLgoKUmlnaHQuIEJ1dCB0 aGVzZSBhcmUgcGl4ZWwgY2xvY2tzLCBub3QgSVNQIGNsb2NrLiBBdCBsZWFzdCB0aGVvcmV0aWNh bGx5CmVudGlyZWx5IGRpZmZlcmVudCBzdHVmZi4KRm9yIGV4YW1wbGUsIEknbSB1c2luZyAoYXQg bW9zdCkgdHdvIElNWDI5MC80NjIgMTkyMHgxMDgwcDYwIHNlbnNvcnMgaW4KMTItYml0IChCYXll cikgbW9kZSwgcmVzdWx0aW5nIGluOgotIDE0OC41IE1IeiBwaXhlbCBjbG9ja3MKLSAxNzgyIE1i L3MgdG90YWwgYmFuZHdpZHRoIChmb3IgZWFjaCBzZW5zb3IgLSAxMiBiaXRzKQotIDQ0NS41IE1i L3MgcGVyIGxhbmUgKGVhY2ggc2Vuc29yIHVzZXMgNCBNSVBJIGxhbmVzKQotIDIyMi43NSBNSHog TUlQSSBjbG9ja3MgKE1JUEkgdXNlcyBERFIsIHRoZSBkb3VibGUgZGF0YSByYXRlKQoKU28gSSdt IHdlbGwgd2l0aGluIHNwZWNzLiBUaG91Z2ggSSBkb24ndCBrbm93IGlmIHRoZSBwZW9wbGUgd3Jp dGluZyB0aGUKZGF0YXNoZWV0cyBkaWQgcmVhbGx5IG1lYW4gd2hhdCB0aGV5IHdyb3RlLCBvciBt YXliZSB0aGV5IG1lYW50IElTUCBjb3JlCmNsb2NrcyBhcyB3ZWxsLiBXaHk/IEJlY2F1c2UgSVNQ IGJsb2NrcyAoYW5kIGFsbCBzdWNoIGxvZ2ljIGJsb2NrcykKcmVxdWlyZSBhIGNlcnRhaW4gbnVt YmVyIG9mIHRoZWlyIGNvcmUgY2xvY2tzIGZvciBhIHNpZ25hbCByYXRlIChpbiB0aGlzCmNhc2Us IGEgcGl4ZWwgcmF0ZSkuIEFuZCBtYXliZSB3aGF0IHRoZXkgc3BlY2lmaWVkIGluIHRoZSBkYXRh c2hlZXRzIHdhcwphIGNhbGN1bGF0aW9uIG9mIHRoaXMgKHVucHVibGlzaGVkPykgcmVxdWlyZWQg cGl4ZWwvSVNQIGNsb2NrIHJhdGlvIGFuZAp0aGUgbWF4IElTUCBjbG9jayByYXRlIChzcGVjaWZp ZWQgaW5jb3JyZWN0bHkgYXQgNDAwLzUwMCBNSHopLgpBbnl3YXksIHRoaXMgaXMgb25seSBhIHBv c3NpYmlsaXR5LiBBbm90aGVyIG9uZSBpcyB0aGF0IElTUDIgc2ltcGx5CmNhbm5vdCByZWxpYWJs eSBydW4gYXQgNTAwIE1IeiwgZm9yIHNvbWUgImFuYWxvZyIgcmVhc29uLCBkZXNpZ24gYnVnCmV0 Yy4KCj4gTXkgdW5kZXJzdGFuZGluZyBpcyB0aGF0IHRoZSBpbXg4bXAuZHRzaSBpcyBwcmUtY29u ZmlndXJlZCBmb3IKPiBvdmVyZHJpdmUgbW9kZSwgc28gaWYgeW91IG5lZWQgdG8gcnVuIHRoZSBJ U1AgaW4gbm9taW5hbCwgdGhlIGNsb2NrCj4gdXBkYXRlcyBzaG91bGQgZ28gaW50byBpbXg4bXAt bm9taW5pYWwuZHRzaS4KCkl0IHNlZW1zIG15IG1vZHVsZXMgYXJlIHNldHVwIGZvciBvdmVyZHJp dmUgbW9kZS4KLS0gCktyenlzenRvZiAiQ2hyaXMiIEhhxYJhc2EKClNpZcSHIEJhZGF3Y3phIMWB dWthc2lld2ljegpQcnplbXlzxYJvd3kgSW5zdHl0dXQgQXV0b21hdHlraSBpIFBvbWlhcsOzdyBQ SUFQCkFsLiBKZXJvem9saW1za2llIDIwMiwgMDItNDg2IFdhcnN6YXdhCgpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpMaW51eC1yb2NrY2hpcCBtYWlsaW5n IGxpc3QKTGludXgtcm9ja2NoaXBAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5m cmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LXJvY2tjaGlwCg==