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 B6DB3C982C1 for ; Thu, 17 Sep 2026 00:08:37 +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:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: References:In-Reply-To:Cc:To:Subject:From:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=l03uO7Uk8o6j0ab+w/47rT1+6f9b5RkpZ9DYh3lRnZY=; b=k4/dgZ2DUnntWS ZqQ7ZortXFBkAf/JcTF3LgrKEMNA8TMEo9fvPMmInLDsy+jxsX9wqqotIib/Wj3VvbMbvXXzX/N+/ LAY4XlPkiXvbJPxn6d/E30vz0NSzr8OuOaqWm3FukTHl7mYuoIFfp+8ZVRT47gnWd+laJAhbJnqxW nN1+nVBwf6w9bCLTqHTUKK7Vwv9/XNbUGq4SH1Dh0at8JZmGUxQFfmUcTMlRNLWjxeL/Erqm8lpFV B+rVV7E5XoB8clUUwa/Exg/3ems2CmzohOu/LYUqq2OJjDIUVLdZoHzMhVQgFyc4Ryhw2p7tuA5c3 lrYHDrga3ToH7fh+TwWg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x6zg5-0000000AJM9-1oT4; Thu, 17 Sep 2026 00:08:37 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x6zg4-0000000AJLz-0kOB for linux-phy@lists.infradead.org; Thu, 17 Sep 2026 00:08:36 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id 262BE601FD; Thu, 17 Sep 2026 00:08:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 613251F000FF; Thu, 17 Sep 2026 00:08:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1789603714; bh=MMxh5Y2XGIBli5WDqvkQ78g//L7qYT0cVlHqUzTO6h8=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=RQoKq/YMv2cOCHvZyScKI2OESrMTyhzR6fnoT4JwMBRC53LrbPBss/pOyTjFH+dPe Sn4WwSnyypS2gK2KsYDqeQFJnp8Lpa5mtnxP7BuB8hmdl/nZ+GLnV2Ke+mmqCUDBs7 vlnoR+qazYcXZyn4iuam1Mua74C4NG1JNTlP1RFoZ0b4XqPl1dn8t6sqLGKOjcm/oj vMJuT1QrTcWesM202nk+p5sUdz4bCRd+f+Q12WFw4dUNPwo0UpDiL9zb/nDN9ARhM5 ZKBONLQ3cY02Uj1Dnvtq676QKUcblF46aQM5owoNpPoYiPixXlRmVXhCGmJe6LKtJU 1GO2/k4WO1Nbg== From: sashiko-bot@kernel.org Subject: Re: [PATCH v18 2/2] phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver To: =?utf-8?b?QnJ5YW4gTydEb25vZ2h1ZQ==?= Cc: vkoul@kernel.org, olteanv@gmail.com, conor+dt@kernel.org, robh@kernel.org, linux-phy@lists.infradead.org, media-ci@linuxtv.org, devicetree@vger.kernel.org, neil.armstrong@linaro.org In-Reply-To: <20260917-x1e-csi2-phy-v18-2-6515b5255fa9@linaro.org> References: <20260917-x1e-csi2-phy-v18-0-6515b5255fa9@linaro.org> <20260917-x1e-csi2-phy-v18-2-6515b5255fa9@linaro.org> Date: Thu, 17 Sep 2026 00:08:33 +0000 Message-Id: <20260917000834.613251F000FF@smtp.kernel.org> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg MyBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6Ci0gW01lZGl1bV0gcGh5OiBtaXBpLWNz aTI6IHZhbGlkIHNldHRsZV9jbnQgb2YgMCBpcyBpbmNvcnJlY3RseSB0cmVhdGVkIGFzIGFuIGVy cm9yCi0gW0xvd10gcGh5OiBtaXBpLWNzaTI6IG1pc3NpbmcgYml0bWFzayBvbiBoYXJkd2FyZSB2 ZXJzaW9uIHJlYWQKLSBbTG93XSBwaHk6IG1pcGktY3NpMjogdW51c2VkIGNsa19uYW1lcyBhcnJh eSBhbmQgaW5jb21wbGV0ZSBjbG9jayB2YWxpZGF0aW9uCi0tCgpjb21taXQgOTZmOTg5Y2Q3Mjkz MTc2MDUxZDU2NWM2N2E1NzVhYmMwZTNmMThlMQpBdXRob3I6IEJyeWFuIE8nRG9ub2dodWUgPGJy eWFuLm9kb25vZ2h1ZUBsaW5hcm8ub3JnPgoKcGh5OiBxY29tLW1pcGktY3NpMjogQWRkIGEgQ1NJ MiBNSVBJIERQSFkgZHJpdmVyCgpUaGlzIGNvbW1pdCBhZGRzIGEgbmV3IE1JUEkgQ1NJMiBEUEhZ IGRyaXZlciBmb3IgUXVhbGNvbW0gcGxhdGZvcm1zLApzdXBwb3J0aW5nIHRoZSAicWNvbSx4MWU4 MDEwMC1jc2kyLXBoeSIgZGV2aWNlLiBJdCBpbXBvcnRzIGV4aXN0aW5nCkNBTVNTIENTSSBQSFkg aW5pdGlhbGl6YXRpb24gc2VxdWVuY2VzIHRvIGNvbmZpZ3VyZSBkYXRhIGFuZCBjbG9jayBsYW5l cy4KCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvcGh5L3F1YWxjb21tL3BoeS1xY29tLW1pcGktY3Np Mi0zcGgtZHBoeS5jIGIvZHJpdmVycy9waHkvcXVhbGNvbW0vcGh5LXFjb20tbWlwaS1jc2kyLTNw aC1kcGh5LmMKPiBpbmRleCAwMDAwMDAwMDAwMDAwLi45NjZkNzljOThmOWIyIDEwMDY0NAo+IC0t LSAvZGV2L251bGwKPiArKysgYi9kcml2ZXJzL3BoeS9xdWFsY29tbS9waHktcWNvbS1taXBpLWNz aTItM3BoLWRwaHkuYwo+IEBAIC0wLDAgKzEsMzg1IEBAClsgLi4uIF0KPiArc3RhdGljIHZvaWQg cGh5X3Fjb21fbWlwaV9jc2kyX2h3X3ZlcnNpb25fcmVhZChzdHJ1Y3QgbWlwaV9jc2kycGh5X2Rl dmljZSAqY3NpMnBoeSkKPiArewo+ICsJY29uc3Qgc3RydWN0IG1pcGlfY3NpMnBoeV9kZXZpY2Vf cmVncyAqcmVncyA9IGNzaTJwaHlfZGV2X3RvX3JlZ3MoY3NpMnBoeSk7Cj4gKwl1MzIgdG1wOwo+ ICsKPiArCXdyaXRlbChDU0lQSFlfM1BIX0NNTl9DU0lfQ09NTU9OX0NUUkw2X1NIT1dfUkVWX0lE LCBjc2kycGh5LT5iYXNlICsKPiArCSAgICAgICBDU0lQSFlfM1BIX0NNTl9DU0lfQ09NTU9OX0NU UkxuKHJlZ3MtPmNvbW1vbl9yZWdzX29mZnNldCwgNikpOwo+ICsKPiArCXRtcCA9IHJlYWRsX3Jl bGF4ZWQoY3NpMnBoeS0+YmFzZSArCj4gKwkJCSAgICBDU0lQSFlfM1BIX0NNTl9DU0lfQ09NTU9O X1NUQVRVU24ocmVncy0+Y29tbW9uX3JlZ3Nfb2Zmc2V0LCAxMikpOwo+ICsJY3NpMnBoeS0+aHdf dmVyc2lvbiA9IHRtcDsKCltTZXZlcml0eTogTG93XQpEb2VzIHRoaXMgcmVhZCBhbGxvdyBub24t dmVyc2lvbiBiaXRzIHRvIGNvcnJ1cHQgdGhlIGZpbmFsIGhhcmR3YXJlIHZlcnNpb24KdmFsdWU/ CgpUaGUgc3Vic2VxdWVudCByZWFkcyBmb3IgYnl0ZXMgMS0zIGFwcGx5IGEgYml0bWFzayAoZS5n LiwgJiAweEZGMDApIHRvCmRpc2NhcmQgZ2FyYmFnZSBpbiB0aGUgdXBwZXIgYml0cywgYnV0IHRo aXMgZmlyc3QgcmVhZCBhc3NpZ25zIHRoZSB2YWx1ZQpkaXJlY3RseS4gQ291bGQgc3RyYXkgYml0 cyBpbiB0aGUgaGlnaGVyIHJlZ2lzdGVycyBmcm9tIG9mZnNldCAxMiBjb3JydXB0CnRoZSBhc3Nl bWJsZWQgdmVyc2lvbiBzdHJpbmc/CgpbIC4uLiBdCj4gK3N0YXRpYyB1OCBwaHlfcWNvbV9taXBp X2NzaTJfc2V0dGxlX2NudF9jYWxjKHM2NCBsaW5rX2ZyZXEsIHUzMiB0aW1lcl9jbGtfcmF0ZSkK PiArewo+ICsJdTMyIHRfaHNfcHJlcGFyZV9tYXhfcHM7Cj4gKwl1MzIgdGltZXJfcGVyaW9kX3Bz Owo+ICsJdTMyIHRfaHNfc2V0dGxlX3BzOwo+ICsJdTggc2V0dGxlX2NudDsKPiArCXUzMiB1aV9w czsKPiArCj4gKwl1aV9wcyA9IGRpdjY0X3U2NChQU0VDX1BFUl9TRUMsIGxpbmtfZnJlcSk7Cj4g Kwl1aV9wcyAvPSAyOwo+ICsJdF9oc19wcmVwYXJlX21heF9wcyA9IDg1MDAwICsgNiAqIHVpX3Bz Owo+ICsJdF9oc19zZXR0bGVfcHMgPSB0X2hzX3ByZXBhcmVfbWF4X3BzOwo+ICsKPiArCXRpbWVy X3BlcmlvZF9wcyA9IGRpdl91NjQoUFNFQ19QRVJfU0VDLCB0aW1lcl9jbGtfcmF0ZSk7Cj4gKwo+ ICsJaWYgKCh0X2hzX3NldHRsZV9wcyAvIHRpbWVyX3BlcmlvZF9wcykgPCA2KQo+ICsJCXJldHVy biAwOwo+ICsKPiArCXNldHRsZV9jbnQgPSB0X2hzX3NldHRsZV9wcyAvIHRpbWVyX3BlcmlvZF9w cyAtIDY7Cj4gKwo+ICsJcmV0dXJuIHNldHRsZV9jbnQ7Cj4gK30KWyAuLi4gXQo+ICtzdGF0aWMg aW50IHBoeV9xY29tX21pcGlfY3NpMl9sYW5lc19lbmFibGUoc3RydWN0IG1pcGlfY3NpMnBoeV9k ZXZpY2UgKmNzaTJwaHksCj4gKwkJCQkJICAgc3RydWN0IG1pcGlfY3NpMnBoeV9zdHJlYW1fY2Zn ICpjZmcpCj4gK3sKPiArCWNvbnN0IHN0cnVjdCBtaXBpX2NzaTJwaHlfZGV2aWNlX3JlZ3MgKnJl Z3MgPSBjc2kycGh5X2Rldl90b19yZWdzKGNzaTJwaHkpOwo+ICsJc3RydWN0IG1pcGlfY3NpMnBo eV9sYW5lc19jZmcgKmxhbmVfY2ZnID0gJmNmZy0+bGFuZV9jZmc7Cj4gKwl1OCBzZXR0bGVfY250 Owo+ICsJdTggdmFsOwo+ICsJaW50IGk7Cj4gKwo+ICsJaWYgKGNmZy0+bGlua19mcmVxIDw9IDAp Cj4gKwkJcmV0dXJuIC1FSU5WQUw7Cj4gKwo+ICsJc2V0dGxlX2NudCA9IHBoeV9xY29tX21pcGlf Y3NpMl9zZXR0bGVfY250X2NhbGMoY2ZnLT5saW5rX2ZyZXEsIGNzaTJwaHktPnRpbWVyX2Nsa19y YXRlKTsKPiArCWlmICghc2V0dGxlX2NudCkKPiArCQlyZXR1cm4gLUVOT0RFVjsKCltTZXZlcml0 eTogTWVkaXVtXQpXaWxsIHRoaXMgZXJyb25lb3VzbHkgcmVqZWN0IHZhbGlkIGhhcmR3YXJlIGNv bmZpZ3VyYXRpb25zPwoKVGhlIGNhbGN1bGF0aW9uIGluIHBoeV9xY29tX21pcGlfY3NpMl9zZXR0 bGVfY250X2NhbGMoKSBldmFsdWF0ZXMgdG8gMCB3aGVuCnRoZSByYXRpbyBpcyBleGFjdGx5IDYs IGJ1dCBpdCBhbHNvIHJldHVybnMgMCB3aGVuIHRoZSByYXRpbyBpcyBsZXNzIHRoYW4gNgp0byBp bmRpY2F0ZSBhbiBlcnJvci4KClNpbmNlIHBoeV9xY29tX21pcGlfY3NpMl9sYW5lc19lbmFibGUo KSBjaGVja3MgaWYgKCFzZXR0bGVfY250KSwgZG9lcyB0aGlzCm92ZXJsYXAgY2F1c2UgYSB2YWxp ZCBjb25maWd1cmF0aW9uIHRvIGJlIHNwdXJpb3VzbHkgcmVqZWN0ZWQgYXMgYW4gZXJyb3I/Cgo+ IGRpZmYgLS1naXQgYS9kcml2ZXJzL3BoeS9xdWFsY29tbS9waHktcWNvbS1taXBpLWNzaTItY29y ZS5jIGIvZHJpdmVycy9waHkvcXVhbGNvbW0vcGh5LXFjb20tbWlwaS1jc2kyLWNvcmUuYwo+IGlu ZGV4IDAwMDAwMDAwMDAwMDAuLmMwYTRmYTIwZWFmMGMgMTAwNjQ0Cj4gLS0tIC9kZXYvbnVsbAo+ ICsrKyBiL2RyaXZlcnMvcGh5L3F1YWxjb21tL3BoeS1xY29tLW1pcGktY3NpMi1jb3JlLmMKPiBA QCAtMCwwICsxLDQ2MyBAQApbIC4uLiBdCj4gK3N0YXRpYyBpbnQgcGh5X3Fjb21fbWlwaV9jc2ky X3Byb2JlKHN0cnVjdCBwbGF0Zm9ybV9kZXZpY2UgKnBkZXYpCj4gK3sKWyAuLi4gXQo+ICsJZm9y IChpID0gMDsgaSA8IG51bV9jbGs7IGkrKykgewo+ICsJCWlmICghY3NpMnBoeS0+Y2xrc1tpXS5p ZCkKPiArCQkJcmV0dXJuIGRldl9lcnJfcHJvYmUoZGV2LCAtRUlOVkFMLCAiTWlzc2luZyBjbG9j ay1uYW1lc1xuIik7Cj4gKwo+ICsJCWlmICghc3RyY21wKGNzaTJwaHktPmNsa3NbaV0uaWQsICJ0 aW1lciIpKSB7Cj4gKwkJCWNzaTJwaHktPnRpbWVyX2NsayA9IGNzaTJwaHktPmNsa3NbaV0uY2xr Owo+ICsJCQlicmVhazsKPiArCQl9Cj4gKwl9CgpbU2V2ZXJpdHk6IExvd10KRG9lcyB0aGlzIGxv b3AgZXhpdCBlYXJseSB1cG9uIGZpbmRpbmcgdGhlIHRpbWVyIGNsb2NrLCBmYWlsaW5nIHRvIHZh bGlkYXRlCnRoZSByZW1haW5pbmcgZHluYW1pY2FsbHkgbG9hZGVkIGNsb2Nrcz8KCkJlY2F1c2Ug ZGV2bV9jbGtfYnVsa19nZXRfYWxsKCkgZG9lcyBub3QgdXNlIHRoZSBjYXJlZnVsbHkgZGVmaW5l ZApzb2NfY2ZnLT5jbGtfbmFtZXMgYXJyYXksIGJyZWFraW5nIGVhcmx5IGhlcmUgYXBwZWFycyB0 byBieXBhc3MgZXhwZWN0ZWQKY2xvY2sgdmFsaWRhdGlvbi4gV291bGQgdGhlIGRyaXZlciBibGlu ZGx5IGFjY2VwdCBtaXNuYW1lZCBjbG9ja3MgaW5zdGVhZCBvZgplbmZvcmNpbmcgdGhlIGV4cGVj dGVkIHBsYXRmb3JtIGNsb2NrIGxpc3Q/CgotLSAKU2FzaGlrbyBBSSByZXZpZXcgwrcgaHR0cHM6 Ly9zYXNoaWtvLmRldi8jL3BhdGNoc2V0LzIwMjYwOTE3LXgxZS1jc2kyLXBoeS12MTgtMC02NTE1 YjUyNTVmYTlAbGluYXJvLm9yZz9wYXJ0PTIKCi0tIApsaW51eC1waHkgbWFpbGluZyBsaXN0Cmxp bnV4LXBoeUBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHBzOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2xpbnV4LXBoeQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 E005B225403 for ; Thu, 17 Sep 2026 00:08:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789603718; cv=none; b=U8+l3KX3FcNAgMdN8JAaDTRROqzh1sX/yq5EvFJDQlaYJ8Jgn/vADeD0GNua2TK56Qnkpt7p1a8maL6KRcM5eXwfyk06y9PmQ+uqqeP9F+N4wMiDZZMqNhZLaFa9uZ4jZ1K7fTNzfkSOLTwegDdxPNNYkZDGstXWleTthjKW5ss= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789603718; c=relaxed/simple; bh=pRAk7foY/DUGjEbnQZoFLzbD9g4nCnqGCV5/GSBSOfM=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=l42mV7EqEyrwosSCa3IS2d58+LRUR7hcXeKCcZw8WBWVp/d0p/cOX3SU+3ckCxy43IxO5A3Vi5B31Np3gD/hXRYrPZnllgOwNeLDDx4gQVTXhc/A7sqpuja75D/ntJdp3KSY00XbvxtF2m4/UD31Dm+bxTgk/LuTXS515th+m5U= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=RQoKq/YM; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="RQoKq/YM" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 613251F000FF; Thu, 17 Sep 2026 00:08:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1789603714; bh=MMxh5Y2XGIBli5WDqvkQ78g//L7qYT0cVlHqUzTO6h8=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=RQoKq/YMv2cOCHvZyScKI2OESrMTyhzR6fnoT4JwMBRC53LrbPBss/pOyTjFH+dPe Sn4WwSnyypS2gK2KsYDqeQFJnp8Lpa5mtnxP7BuB8hmdl/nZ+GLnV2Ke+mmqCUDBs7 vlnoR+qazYcXZyn4iuam1Mua74C4NG1JNTlP1RFoZ0b4XqPl1dn8t6sqLGKOjcm/oj vMJuT1QrTcWesM202nk+p5sUdz4bCRd+f+Q12WFw4dUNPwo0UpDiL9zb/nDN9ARhM5 ZKBONLQ3cY02Uj1Dnvtq676QKUcblF46aQM5owoNpPoYiPixXlRmVXhCGmJe6LKtJU 1GO2/k4WO1Nbg== From: sashiko-bot@kernel.org Subject: Re: [PATCH v18 2/2] phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver Reply-To: sashiko-reviews@lists.linux.dev To: =?utf-8?b?QnJ5YW4gTydEb25vZ2h1ZQ==?= Cc: vkoul@kernel.org, olteanv@gmail.com, conor+dt@kernel.org, robh@kernel.org, linux-phy@lists.infradead.org, media-ci@linuxtv.org, devicetree@vger.kernel.org, neil.armstrong@linaro.org In-Reply-To: <20260917-x1e-csi2-phy-v18-2-6515b5255fa9@linaro.org> References: <20260917-x1e-csi2-phy-v18-0-6515b5255fa9@linaro.org> <20260917-x1e-csi2-phy-v18-2-6515b5255fa9@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Thu, 17 Sep 2026 00:08:33 +0000 Message-Id: <20260917000834.613251F000FF@smtp.kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 3 potential issue(= s) to consider: - [Medium] phy: mipi-csi2: valid settle_cnt of 0 is incorrectly treated as = an error - [Low] phy: mipi-csi2: missing bitmask on hardware version read - [Low] phy: mipi-csi2: unused clk_names array and incomplete clock validat= ion -- commit 96f989cd7293176051d565c67a575abc0e3f18e1 Author: Bryan O'Donoghue phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver This commit adds a new MIPI CSI2 DPHY driver for Qualcomm platforms, supporting the "qcom,x1e80100-csi2-phy" device. It imports existing CAMSS CSI PHY initialization sequences to configure data and clock lanes. > diff --git a/drivers/phy/qualcomm/phy-qcom-mipi-csi2-3ph-dphy.c b/drivers= /phy/qualcomm/phy-qcom-mipi-csi2-3ph-dphy.c > index 0000000000000..966d79c98f9b2 100644 > --- /dev/null > +++ b/drivers/phy/qualcomm/phy-qcom-mipi-csi2-3ph-dphy.c > @@ -0,0 +1,385 @@ [ ... ] > +static void phy_qcom_mipi_csi2_hw_version_read(struct mipi_csi2phy_devic= e *csi2phy) > +{ > + const struct mipi_csi2phy_device_regs *regs =3D csi2phy_dev_to_regs(csi= 2phy); > + u32 tmp; > + > + writel(CSIPHY_3PH_CMN_CSI_COMMON_CTRL6_SHOW_REV_ID, csi2phy->base + > + CSIPHY_3PH_CMN_CSI_COMMON_CTRLn(regs->common_regs_offset, 6)); > + > + tmp =3D readl_relaxed(csi2phy->base + > + CSIPHY_3PH_CMN_CSI_COMMON_STATUSn(regs->common_regs_offset, 12)); > + csi2phy->hw_version =3D tmp; [Severity: Low] Does this read allow non-version bits to corrupt the final hardware version value? The subsequent reads for bytes 1-3 apply a bitmask (e.g., & 0xFF00) to discard garbage in the upper bits, but this first read assigns the value directly. Could stray bits in the higher registers from offset 12 corrupt the assembled version string? [ ... ] > +static u8 phy_qcom_mipi_csi2_settle_cnt_calc(s64 link_freq, u32 timer_cl= k_rate) > +{ > + u32 t_hs_prepare_max_ps; > + u32 timer_period_ps; > + u32 t_hs_settle_ps; > + u8 settle_cnt; > + u32 ui_ps; > + > + ui_ps =3D div64_u64(PSEC_PER_SEC, link_freq); > + ui_ps /=3D 2; > + t_hs_prepare_max_ps =3D 85000 + 6 * ui_ps; > + t_hs_settle_ps =3D t_hs_prepare_max_ps; > + > + timer_period_ps =3D div_u64(PSEC_PER_SEC, timer_clk_rate); > + > + if ((t_hs_settle_ps / timer_period_ps) < 6) > + return 0; > + > + settle_cnt =3D t_hs_settle_ps / timer_period_ps - 6; > + > + return settle_cnt; > +} [ ... ] > +static int phy_qcom_mipi_csi2_lanes_enable(struct mipi_csi2phy_device *c= si2phy, > + struct mipi_csi2phy_stream_cfg *cfg) > +{ > + const struct mipi_csi2phy_device_regs *regs =3D csi2phy_dev_to_regs(csi= 2phy); > + struct mipi_csi2phy_lanes_cfg *lane_cfg =3D &cfg->lane_cfg; > + u8 settle_cnt; > + u8 val; > + int i; > + > + if (cfg->link_freq <=3D 0) > + return -EINVAL; > + > + settle_cnt =3D phy_qcom_mipi_csi2_settle_cnt_calc(cfg->link_freq, csi2p= hy->timer_clk_rate); > + if (!settle_cnt) > + return -ENODEV; [Severity: Medium] Will this erroneously reject valid hardware configurations? The calculation in phy_qcom_mipi_csi2_settle_cnt_calc() evaluates to 0 when the ratio is exactly 6, but it also returns 0 when the ratio is less than 6 to indicate an error. Since phy_qcom_mipi_csi2_lanes_enable() checks if (!settle_cnt), does this overlap cause a valid configuration to be spuriously rejected as an error? > diff --git a/drivers/phy/qualcomm/phy-qcom-mipi-csi2-core.c b/drivers/phy= /qualcomm/phy-qcom-mipi-csi2-core.c > index 0000000000000..c0a4fa20eaf0c 100644 > --- /dev/null > +++ b/drivers/phy/qualcomm/phy-qcom-mipi-csi2-core.c > @@ -0,0 +1,463 @@ [ ... ] > +static int phy_qcom_mipi_csi2_probe(struct platform_device *pdev) > +{ [ ... ] > + for (i =3D 0; i < num_clk; i++) { > + if (!csi2phy->clks[i].id) > + return dev_err_probe(dev, -EINVAL, "Missing clock-names\n"); > + > + if (!strcmp(csi2phy->clks[i].id, "timer")) { > + csi2phy->timer_clk =3D csi2phy->clks[i].clk; > + break; > + } > + } [Severity: Low] Does this loop exit early upon finding the timer clock, failing to validate the remaining dynamically loaded clocks? Because devm_clk_bulk_get_all() does not use the carefully defined soc_cfg->clk_names array, breaking early here appears to bypass expected clock validation. Would the driver blindly accept misnamed clocks instead of enforcing the expected platform clock list? --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260917-x1e-csi2-p= hy-v18-0-6515b5255fa9@linaro.org?part=3D2