From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f54.google.com (mail-ed1-f54.google.com [209.85.208.54]) (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 E5B64211484 for ; Tue, 27 May 2025 21:22:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748380971; cv=none; b=skxt3YyILhHljrS2T1cAEi1OvqgZ0J1wdkgrM7+znE8hOGrTKbCoh+RFDDt+Xe/b5ndZAyGS7pS1C6D3McI1fr+CyQVJ6e4tM27gJvGGMiA8VMB418ANL/dlkzbvEDGgNLUKT5JRxpjNB2uqnBuM/QrTAzZhuQF3Ijja3racaS8= 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.54 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-f54.google.com with SMTP id 4fb4d7f45d1cf-60477f1a044so4308066a12.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=BhDZE3KsT3x5X8yGP19SR+fYz2i3e3KCa72uX4kWx/0un/TT5eRPPmMNn9L6T8wTuM DsU7fK6gYtzooQ22Ve98L14eYAHKD++MoZDDhrwDZGF+aOUB465sWK+KhYH4EJB4Nw52 4aR3xUtbmT53F/VdFd/ZKH+XAldvIi5PmsknGxzkY1az7IvCW6SMYsjk7K6SeznrPMhU B/za5qRUQhMMbNdsw/72A4Q0uLRlzavyUWJj6iImKV6FmQ4BX/9qs+nPTidrVzFyJEsW asnV7RNP3n04h4pvpV8r7hQCLv8GvyOPFAtHuz1IZSgMEoE9ffUSUew5XYxivuiK9qWa aZ7A== X-Forwarded-Encrypted: i=1; AJvYcCXCguoy2UHfK5k1y4Tak0hbprxx9u2GxzTd3UJcTsDPomcDvhNneEVFvWSoGGFXbeZVvOLtIvilmbmY@vger.kernel.org X-Gm-Message-State: AOJu0Yy+0yAtMxkhnNUdb3UqCK8Ay3CO/PyxpDfynbz3jhbIocL7jgR+ G+WBtUT+iEPysLDz1wPpGgDGXNDn1T8GR2v99YBQh+Cy7B+Ra5LmvnE5j1lNFGtXi4g= X-Gm-Gg: ASbGnctAcFOkndxC3VgNAOrkKJPSR07NGaNx7OcUDqzIIhzx5rK/2rYRqe6nJYiJESa r24/z3ZTVrw3Ocy8Mc6ZwjxNSIjk7vVAiPfeKmvheTeQiH37N/5SKuy+qs07UaP+12VSQdZDsIV N5LZHIhwciDHsDyTtvTl3YIE2XEwQrKqaHlDyWXt09Olw5eVtz3dIFBZkDExVqjS2w+ORt8Bpti PR3Os1f55er7/qDxIxwoZO2QrHeiFvdPXrHthJ3aGrzhAVwgTw2JWTV0tqiNTZVhzibjHWrfWQS gFTkGWsbAZDDafwQ44/qI5jI9LaQUSbjUUkPvG9M55q169iWA6M1NIvgDAoV+z75lFj3G9KwSEB b0ylAZD3oHdPOoMZh7y+8+gfQjN5ky1b8 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: devicetree@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