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 A1530C4167B for ; Thu, 7 Dec 2023 17:45:18 +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-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=wXwi8bwAJE5UTgz1makbqP90twgCBjoLVI3EYDs9Vz0=; b=lNQOnZUAfeQzOr7ZvW8X8YfdEi 9NrX6I6/GSifGSA/RAXnWALuCpac0Jt+9teVrFYSvq4btAPRGpd2pXDYBEtx7bZ54fZ/pMJq8nvsB JJIxLqXZQr6TIEf1Rwx+CGBxs6jYXnx5x9YCWLVdwQd6fiFoLOAAnK8nRRgmeytod4gZTcnhfNcim OW3y1M6c22+6MFyLP7/KA6DqPsr9is7bFSXi41fQ3P4/0Wng9BVIgScT6K/IckROHKCJN681dIOlW npmOrvkF4z7o3EJCFASWzP0M3Q+BU224uLsmcjz9mRG/9Md+JyseE7X9xDZeGV2tX7T2byK8te2/0 sGrQVI0g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rBIQa-00DZ1t-16; Thu, 07 Dec 2023 17:44:48 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rBIQW-00DYzd-2W; Thu, 07 Dec 2023 17:44:46 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id B8BB7B82986; Thu, 7 Dec 2023 17:44:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 33A48C433C7; Thu, 7 Dec 2023 17:44:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1701971081; bh=kE0n2gr2CaIRus73ES8kcVH1cf0yVteSBzOYtQB63UE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Wx2uHuTI6SpzzO+4J24LEQXHwI8kMMjn/qOqux/fXdUtdAWG2QCL8sILWDpJv+ZRF KHg3m32XXp9M0JHczC1pJ5mbyuf2v+IBiI4TxCk3wJuJ8gF3M9cRUR/+1F7JeGjaIn sGFKQ4y5dzNIUi712KgDXAfT8xTWq/lrdNamvAiVIjOxeBxsCzjK3vwAts6s3KZll+ lAHTujKHv4mzVF64GL9BrxMY0ixS8kqGzoqDwQphiKSb7NTU8hSLtNWWvAHjjsYKSn eEpZKEROovsfYnyVFHO8rc0FM5HySlfHMJ+nD6KqHaBHXSEB2auLMHiVxI+C6WhlWq rLdKHrenbru5g== Date: Thu, 7 Dec 2023 17:44:32 +0000 From: Conor Dooley To: Krzysztof Kozlowski Cc: Sakari Ailus , Zhi Mao , mchehab@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, shengnan.wang@mediatek.com, yaya.chang@mediatek.com, 10572168@qq.com, Project_Global_Chrome_Upstream_Group@mediatek.com, yunkec@chromium.org, conor+dt@kernel.org, matthias.bgg@gmail.com, angelogioacchino.delregno@collabora.com, jacopo.mondi@ideasonboard.com, hverkuil-cisco@xs4all.nl, heiko@sntech.de, jernej.skrabec@gmail.com, macromorgan@hotmail.com, linus.walleij@linaro.org, laurent.pinchart@ideasonboard.com, hdegoede@redhat.com, tomi.valkeinen@ideasonboard.com, gerald.loacker@wolfvision.net, andy.shevchenko@gmail.com, bingbu.cao@intel.com, dan.scally@ideasonboard.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH 1/2] media: i2c: Add GC08A3 image sensor driver Message-ID: <20231207-outcome-acclaim-d179c8c07fff@spud> References: <20231207052016.25954-1-zhi.mao@mediatek.com> <20231207052016.25954-2-zhi.mao@mediatek.com> <129e3a8b-5e91-424a-8ff8-b015d5175f1a@linaro.org> MIME-Version: 1.0 In-Reply-To: <129e3a8b-5e91-424a-8ff8-b015d5175f1a@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231207_094444_947018_B750FCF4 X-CRM114-Status: GOOD ( 13.41 ) 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: , Content-Type: multipart/mixed; boundary="===============1853539801962531108==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============1853539801962531108== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="N40z4CafL7X1rYMV" Content-Disposition: inline --N40z4CafL7X1rYMV Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Dec 07, 2023 at 01:30:35PM +0100, Krzysztof Kozlowski wrote: > On 07/12/2023 12:34, Sakari Ailus wrote: > >> + ret =3D gc08a3_parse_fwnode(dev); > >> + if (ret) > >> + return ret; > >> + > >> + gc08a3 =3D devm_kzalloc(dev, sizeof(*gc08a3), GFP_KERNEL); > >> + if (!gc08a3) > >> + return -ENOMEM; > >> + > >> + gc08a3->dev =3D dev; > >> + > >> + gc08a3->xclk =3D devm_clk_get(dev, NULL); > >> + if (IS_ERR(gc08a3->xclk)) > >> + return dev_err_probe(dev, PTR_ERR(gc08a3->xclk), > >> + "failed to get xclk\n"); > >> + > >> + ret =3D clk_set_rate(gc08a3->xclk, GC08A3_DEFAULT_CLK_FREQ); > >=20 > > Please see: > > . >=20 >=20 > Oh, that's cool it was documented! >=20 > The canonical link would be: > https://www.kernel.org/doc/html/latest/driver-api/media/camera-sensor.htm= l#devicetree I believe this is that answer to the "why are these needed" that I asked on the previous version and never got a response to. Instead, they were just removed from the binding etc. --N40z4CafL7X1rYMV Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZXIEgAAKCRB4tDGHoIJi 0ouqAQDUTa6OU1g0zJUvJGN1UaS5mtsII2EiSZDKd4calhNwgAD8D7luYf3WpePJ KUxA5v4yxt6pUlaZ/sKzTzU/x1c2/QQ= =YQab -----END PGP SIGNATURE----- --N40z4CafL7X1rYMV-- --===============1853539801962531108== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============1853539801962531108==--