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 8F112C43458 for ; Tue, 30 Jun 2026 11:28: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:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:References:Content-Type: Content-Transfer-Encoding:MIME-Version:Message-ID:Date:Subject:In-Reply-To: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=WBlom0iDPCVsCjE864JxXWp8X0m3OrVlX25asskwS8g=; b=GabbkjqiLTN8c4krnPD0vq02TC C2DTOXd4ttJVulh/cBgj/uKwHQ+fE0QvYH3esIzYdZ0DADjGhStdGyBmnx4SjtPBA4UpTmNrJNfyn QYgTcP68uYf+yPfX80MlHALret0IvwyCLL6p6vuvcqmdQqIvGcHuTHR5ryonMP1TBTXXn6q+/c7sf ol3e2g/9q4HJW/uvks0Fl/IME6IinHYr4ZgUPmZO9tgqSWz2pRtmtQ5zQf6vsknx0wMpRwXbNMDyD hmytsOD0fV5kJHHMZ00+/vyzeXB2Ux/EUhoAW4P5W6WpWu0D+dcjMXaML60u1X0xYHGqo1HNKfoX5 dXiurJ1Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1weWdz-0000000GqUF-3xfM; Tue, 30 Jun 2026 11:28:47 +0000 Received: from mailout2.samsung.com ([203.254.224.25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1weWdw-0000000GqTB-1PnE for linux-arm-kernel@lists.infradead.org; Tue, 30 Jun 2026 11:28:46 +0000 Received: from epcas5p3.samsung.com (unknown [182.195.41.41]) by mailout2.samsung.com (KnoxPortal) with ESMTP id 20260630112838epoutp02af952518716bd7cf78cc024f70d48a46~92FwzShO61232612326epoutp025 for ; Tue, 30 Jun 2026 11:28:38 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.samsung.com 20260630112838epoutp02af952518716bd7cf78cc024f70d48a46~92FwzShO61232612326epoutp025 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1782818918; bh=WBlom0iDPCVsCjE864JxXWp8X0m3OrVlX25asskwS8g=; h=From:To:Cc:In-Reply-To:Subject:Date:References:From; b=N8CMIt18qyqh4UyXXu6C2pgNITJJ3eLl/7VDzD0CpSERHQiRJMwiN7Jmlssm0yWb6 +Us5ntXEs0j374OaCF5zkXsDgxbhrpnHoOLZaO+faHvRo880HqpzkRw9FRERxHM3EE 6KyEDpAJ6Awe4ZEXTpsv2TxSHatLGCXmvS3UsMgo= Received: from epsnrtp01.localdomain (unknown [182.195.42.153]) by epcas5p4.samsung.com (KnoxPortal) with ESMTPS id 20260630112837epcas5p42699c74e7bdfdc08e7a387ecddd1610f~92FwTOXXd2430624306epcas5p4W; Tue, 30 Jun 2026 11:28:37 +0000 (GMT) Received: from epcas5p3.samsung.com (unknown [182.195.38.87]) by epsnrtp01.localdomain (Postfix) with ESMTP id 4gqLW83pG4z6B9m9; Tue, 30 Jun 2026 11:28:36 +0000 (GMT) Received: from epsmtip1.samsung.com (unknown [182.195.34.30]) by epcas5p4.samsung.com (KnoxPortal) with ESMTPA id 20260630112836epcas5p49964fca2cb45fb07d2bbd7f3a7dfcbf1~92FvDdikf2430624306epcas5p4U; Tue, 30 Jun 2026 11:28:36 +0000 (GMT) Received: from INBRO002756 (unknown [107.122.3.168]) by epsmtip1.samsung.com (KnoxPortal) with ESMTPA id 20260630112833epsmtip1093e0631eef1002d20979d73c9441641~92FsWGGhq1723317233epsmtip1E; Tue, 30 Jun 2026 11:28:33 +0000 (GMT) From: "Alim Akhtar" To: "'Krzysztof Kozlowski'" , "'Peter Griffin'" Cc: "'Denzeel Oliva'" , "'Sylwester Nawrocki'" , "'Chanwoo Choi'" , "'Michael Turquette'" , "'Stephen Boyd'" , "'Brian Masney'" , "'Rob Herring'" , "'Conor Dooley'" , , , , , In-Reply-To: Subject: RE: [PATCH v3 3/3] clk: samsung: exynos990: Fix PERIS gate clock parents Date: Tue, 30 Jun 2026 16:58:31 +0530 Message-ID: <10d301dd0883$970925c0$c51b7140$@samsung.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQEx4sfAWwy4gnk+MuXqAo8FFpNhngGTah19AjUYwH4DLBf1wQK+JwWqAckmQ8cBG1pq6bdIhjIw Content-Language: en-us X-CMS-MailID: 20260630112836epcas5p49964fca2cb45fb07d2bbd7f3a7dfcbf1 X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" CMS-TYPE: 105P cpgsPolicy: CPGSC10-543,Y X-CFilter-Loop: Reflected X-CMS-RootMailID: 20260629123246epcas5p2aab9da9ef417274bb0b70f74c386f7b4 References: <20260613-exynos990-peris-fix-v3-v3-0-2b230db78ae4@gmail.com> <20260613-exynos990-peris-fix-v3-v3-3-2b230db78ae4@gmail.com> <0f1e01dd0844$01190c40$034b24c0$@samsung.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260630_042845_038981_537977B6 X-CRM114-Status: GOOD ( 29.88 ) 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 > -----Original Message----- > From: Krzysztof Kozlowski > Sent: Tuesday, June 30, 2026 4:43 PM > To: Peter Griffin ; Alim Akhtar > > Cc: Denzeel Oliva ; Sylwester Nawrocki > ; Chanwoo Choi ; > Michael Turquette ; Stephen Boyd > ; Brian Masney ; Rob Herring > ; Conor Dooley ; linux-samsung- > soc=40vger.kernel.org; linux-clk=40vger.kernel.org; > devicetree=40vger.kernel.org; linux-arm-kernel=40lists.infradead.org; lin= ux- > kernel=40vger.kernel.org > Subject: Re: =5BPATCH v3 3/3=5D clk: samsung: exynos990: Fix PERIS gate c= lock > parents >=20 > On 30/06/2026 13:02, Peter Griffin wrote: > > Hi Alim, > > > > On Tue, 30 Jun 2026 at 04:53, Alim Akhtar > wrote: > >> > >> > >> > >>> -----Original Message----- > >>> From: Peter Griffin > >>> Sent: Monday, June 29, 2026 6:02 PM > >>> To: Denzeel Oliva > >>> Cc: Krzysztof Kozlowski ; Sylwester Nawrocki > >>> ; Chanwoo Choi > ; Alim > >>> Akhtar ; Michael Turquette > >>> ; Stephen Boyd ; Brian > >>> Masney ; Rob Herring ; > Conor > >>> Dooley ; linux-samsung-soc=40vger.kernel.org; > >>> linux- clk=40vger.kernel.org; devicetree=40vger.kernel.org; linux-arm= - > >>> kernel=40lists.infradead.org; linux-kernel=40vger.kernel.org > >>> Subject: Re: =5BPATCH v3 3/3=5D clk: samsung: exynos990: Fix PERIS ga= te > >>> clock parents > >>> > >>> Hi Krysztof & Denzeel, > >>> > >>> On Sat, 13 Jun 2026 at 13:36, Denzeel Oliva > >>> > >>> wrote: > >>>> > >>>> Correct eight PERIS gate clock parents to match the hardware clock > >>>> tree and reorder the GIC mux parents so mout_peris_bus_user is the > >>>> default source. > >>>> > >>>> Signed-off-by: Denzeel Oliva > >>>> --- > >>> > >>> Reviewed-by: Peter Griffin > >>> > >>> =40Krysztof: I was thinking, maybe we should establish a new rule/bes= t > >>> practice for Samsung clock upstream submissions whereby patch > >>> contributors should link to the downstream cal-if code for the SoC > >>> after the -- > >>> - line. That would make reviewing the patches' correctness a bit > >>> easier, as the downstream cal-if code would be readily available to t= he > reviewer. > >>> > >> We can leave this choice to the reviewer if they want to refer to > downstream cal-if code. > > > > Generally I would like to, but I also don't have time to hunt around > > the internet for a downstream kernel tree. My rationale was that the > > submitter is most likely to know where the downstream code is, and is > > likely using it for the upstream clock implementation. So, linking to > > it as part of the submission should hopefully be fairly easy. > > > > If it is a Samsung SoC for which no public code is available that's > > fine. I didn't intend this to be a hard requirement: =22you can't > > upstream x,y,z unless you link to the cal-if code=22. I meant it more a= s > > =22best practice/guidance=22; if the cal-if code is publicly available, > > linking to it would be a useful reference for reviewers. >=20 > cal-if as vendor tree? Some contributors just base their work on downstre= am > GPL-compliance dumps from opensource.samsung.com, so not sure how > that link would work. >=20 Right, I am not against adding any best practice guideline, but I wonder on= ly few of us will end up reviewing those patches.=20 And for other reviewer, cal-if will add more confusion as it is completely = out of =7Btree / clk subsystem=7D interface.=20 > Best regards, > Krzysztof