From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f41.google.com (mail-ed1-f41.google.com [209.85.208.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E78F32116E7 for ; Tue, 27 May 2025 21:22:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748380971; cv=none; b=emCPuk/QtLR6+fcvCnS4+RHx+Jaaz4KkYkfD+S5MEXqJB4h78S5zvvuCh7F2a/xYETQaM5wtBvmDWcslr7VQMNe+p6kCLMbqQkJ7EUyxDmZy7eCRf9DQSkt0E8BbsbEicA9Tp7zCbw8FXX8Oy0MprB9qjeFlJ/6meDXa8OP80HA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748380971; c=relaxed/simple; bh=CmXNjfU2YBa1jt2Z+F/46nl2cI2JRC7z389h5i3eWiA=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=nShUVUwKLqgNe0PN8bxs7tC+bsxoR1auoWN/sWpX4V2FeCKI5XSqhcfQJF/G3oNqQy7/W+P4v7rP0mdBvB4hB2k2ceAGv+SYGEuxR+zSLqmp2CnE6Vx+yoat+vQ9h9+Xl6iD+aH0BVrql6N3140e1gZcQxRrDc6h8iT/ux3cLCI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=H69psyHK; arc=none smtp.client-ip=209.85.208.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="H69psyHK" Received: by mail-ed1-f41.google.com with SMTP id 4fb4d7f45d1cf-60477f1a044so4308064a12.0 for ; Tue, 27 May 2025 14:22:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1748380967; x=1748985767; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:date:from:from:to :cc:subject:date:message-id:reply-to; bh=rf03yAgQQGBhErkrSSaEFDElj+fjBRlARFLOXQZblAk=; b=H69psyHKbQd+b1T7e2t1AoZ7tFyB8lK5HSQmjuD4D/7k4SnwZHcfu1urhxpPqWYzwc qw3Da9W98rPjPC8ZszP9Mcd5P2DBqssZl30GNyhWM66Yb/IdMXGAQXObQO1lNdzTw7zu yLWwhTGy04iCD04EUbey1mHl7vQs8uiYqk+b+VEatSNwgPTfVN+XHTWFz5wHHdQf6t78 PfPNlOGGSo3fYrlwSkDTATULV6VvjZ7Vjd6ezZOhT3c3ch9kFOIPjD4Zyk5hAsXvZEjY FU2Gda7Cf1knFTS16k5nMmS7BMg/rG2zpr5I7z8D3yy+na7KjrTQRVXitX4ZsZLNHJO0 BFDA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1748380967; x=1748985767; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:date:from :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=rf03yAgQQGBhErkrSSaEFDElj+fjBRlARFLOXQZblAk=; b=g/a8JCM07KaYG/D1qdCY1en+6O6MJWxG7jv/2WX+EDvURaDeHKeJoujDljeCmJKz8H S9YjMPVSIBSJNGs+k9iSP3yEvd4SClbxdn8/BIOmlnHS/b/wqrR8YqHPVMj4zQ7GyjRo tNDNjx+vOoaVH8tELmkDBnx4ssvvm4jD/to/G78k3b9U6bgdfOxpyyyTHdX/VyOoZLbY JVBlfvJ3er1JBqbbn1TALyzS6Sn9GgTwaRx931vcWbkN63KhDzLVle/k5bL4AHh8aI+E jYnX0AjUKJosqmj32QlfqBb9css4RJamjqaI0mN8k2uDXO9xFQfKm7yIkjvDildecNcZ K3Ow== X-Forwarded-Encrypted: i=1; AJvYcCWFqQV5RWXirihPX/a5DH4ZBLoEIPG3ZyrwYu+MzbVISljk4dyaVHy7yxKd4mnvYtcwvfccWax110U=@vger.kernel.org X-Gm-Message-State: AOJu0Yz34MXVVCmnH1bPMRKloMa2ao+T0y6a/UmHc1wQSSLm+VKor152 ffHDrpwb00DrVVOLzfUuSsOIOjUYWuaqQ0stXmlPIzk5f+7PfxThiKTIqgm984QjPdI= X-Gm-Gg: ASbGncvsqOc1OtQPORjLetq2zru3v+NHrspVnhQiC6zEjjxGTRUxzMbhRcPG+Zfx8xD V70/L+THk4T76YcmiUaX1p/S8jbmpyIq7F/ANrvX5LIzJLu7EnL8BGcsTzb9nzLYAQapc3IOTjs r42f6+A82IHo01RKNDHBymfL0xspbu0HEKBS6VRHD5hV9sHJKYjjCw0WckrkgeBbxmd6643PVSL cgvPMnE9dSBGogJEHPNCevpEhoVAxCsovY250FxHyofPv0uQCeM4ifVh4K7mjAU3k8P9ou2yrk5 AvKpwKHVGL4QoLXZOVSfutQ+Wwann9td3yNW6ZtO028dNQQH+aS117P69v1NspkylXJoL7N0FJJ V/4kYDxgpKg7o3/8YkqZjgBbWIo8hROMg X-Google-Smtp-Source: AGHT+IFERg8h0M0q68ymhrrsY3ViILpvzLyzrWuNOWeP4SVGe1Wg+ZNpOzN/Rnmv7ShgttQHfLkmmw== X-Received: by 2002:a05:6402:1d4b:b0:601:f185:fc2 with SMTP id 4fb4d7f45d1cf-602d8e5e352mr13569888a12.3.1748380967163; Tue, 27 May 2025 14:22:47 -0700 (PDT) Received: from localhost (host-87-21-228-106.retail.telecomitalia.it. [87.21.228.106]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-60517a7f920sm128296a12.53.2025.05.27.14.22.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 May 2025 14:22:46 -0700 (PDT) From: Andrea della Porta X-Google-Original-From: Andrea della Porta Date: Tue, 27 May 2025 23:24:20 +0200 To: Florian Fainelli Cc: Matthias Brugger , Andrea della Porta , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Broadcom internal kernel review list , Lorenzo Pieralisi , Krzysztof Wilczynski , Manivannan Sadhasivam , Bjorn Helgaas , Linus Walleij , Catalin Marinas , Will Deacon , Bartosz Golaszewski , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , Saravana Kannan , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-gpio@vger.kernel.org, Masahiro Yamada , Stefan Wahren , Herve Codina , Luca Ceresoli , Thomas Petazzoni , Andrew Lunn , Phil Elwell , Dave Stevenson , kernel-list@raspberrypi.com Subject: Re: [PATCH v9 -next 08/12] arm64: dts: bcm2712: Add external clock for RP1 chipset on Rpi5 Message-ID: References: <38514415df9c174be49e72b88410d56c8de586c5.1745347417.git.andrea.porta@suse.com> <96272e42-855c-4acc-ac18-1ae9c5d4617f@broadcom.com> <48AFA657-5683-42A4-888E-3E98A515F3B1@broadcom.com> <3899c82c-d6a7-4daf-889b-b4d7f3185909@suse.com> <6953caf0-0fef-4bd6-898c-4f86ee738f30@broadcom.com> Precedence: bulk X-Mailing-List: linux-clk@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <6953caf0-0fef-4bd6-898c-4f86ee738f30@broadcom.com> Hi Florian, On 09:18 Tue 27 May , Florian Fainelli wrote: > On 5/26/25 07:06, Matthias Brugger wrote: > > > > > > On 12/05/2025 18:42, Andrea della Porta wrote: > > > Hi Florian, > > > > > > On 15:02 Mon 12 May     , Florian Fainelli wrote: > > > > On May 7, 2025 5:01:05 PM GMT+02:00, Andrea della Porta > > > > wrote: > > > > > Hi Florian, to accept the patches, what would work best for you? > > > > > > > > > > 1) Send only the relevant updated patches (maybe as an entirely new > > > > > patchset with > > > > >    only those specific patches) > > > > > > > > Only the updated patches work for me. I don't think there is > > > > that much coupling between the DT changes and the non-DT changes > > > > (other than without DT entries nothing is activated) > > > > > > It's a little bit more involved than that: > > > > > > - Patch 7 (misc driver) depends on 6 (RP1 common dts) which in turn > > >    depends on 1 (clock binding header). Should be taken by Greg. > > > > Greg gave an Acked-by so I think Florian is good to take that patch. > > Which leaves us to the clock patches (driver + dt-bindings). > > > > > > > > - Patch 9 and 10 (board dts) depends on 6 (RP1 common dts) which again > > >    depends on 1 (clock binding header). Should be taken by Florian. > > > > > > - Patch 4 (clock driver) depends on 1 (clock binding header) and > > >    should be taken by Stephen. > > > > > > > Steven reviewed the patches (driver + dt-binding) so he is waiting for a > > new version which addresses the review. He offered to either take them > > and provide a branch that Florian can merge into his branch or provide a > > Acked-by tag. > > > > @Florian what would you prefer? > > I am fine either way, it's definitively simpler if I can take all of the > patches in the respective Broadcom ARM SoC branches, but pulling a branch > from another maintainer's tree works just as well. > > Andrea, sorry to ask you this, can you post a v10 and we aim to get that > version applied? No problem, just to avoid any confusion I'll summarize what-goes-where with respect to branches in your repo broadcom/stblinux, so I can adapt each patch to the relevant branch: - dt-binding/DTS (patch 1,2,3,6,8,9,10) -> devicetree/next - defconfig (patch 11,12) -> defconfig/next - drivers (patch 4,5,7) -> drivers/next or soc/next? Also, should I split any patches that have MAINTAINERS changes so you can apply them to your maintainers/next branch? Those are patches 4,5,6,7. Many thanks, Andrea > Thanks! > -- > Florian