From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f51.google.com (mail-wr1-f51.google.com [209.85.221.51]) (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 5EDFD14A088 for ; Sun, 27 Oct 2024 11:15:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730027709; cv=none; b=BwF1qAtTA4C0VYGRSntS0sF5yDIR7LksUv8NOI1P6QRh+5p+s1XaPBaUPRwya30uLdA3Cn7M1K5HF8v2kuzQ7FGzk0bwYU++UAssbUo+dEKzh0hwcUbOmfPDAoe0OO4sbaictP5qYJgSfvDAEfE/EivfWinIQsL6xRmNgqG/kFI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730027709; c=relaxed/simple; bh=xDDeU6p25FtrTyo3f2uKG58qzH1nBHIOOMUYrGXVBAs=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=OD4GHaRl6gGXE/VH2+bKYxCo5nocgtJZ4m3lN8LPDxXfW7dBuoxohC7lLGBO2EUWqtZucYCvqrXJYKhEcBmDdR/qZossvKOMJFKGfpZ/mwawY1Hlnme8kVoiEZrpLFqwv9L0hiNvL3tuqIxRVsIbjEUTWxof6ArNHtTClHghYEw= 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=dAwRgyZa; arc=none smtp.client-ip=209.85.221.51 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="dAwRgyZa" Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-37d473c4bb6so3115853f8f.3 for ; Sun, 27 Oct 2024 04:15:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1730027706; x=1730632506; 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=3yR+elAtNrZb0dYqaY4oOlXhVHYbUSfEfohSukSkC5o=; b=dAwRgyZaIAOhWfLAdYHlMNRIi+agvqIW7TC9InsP8G6P9TAamwmaBckNxt9e5fTcF7 F5JhPmu3IvVVPJCyF1oT3m6vfz3GPXHvuDvm2dKs9S48V/I71udfUa2xpf4gRu9N9vE5 VH0NoYrcqSM/83tMFB9nkQsXJAidjgs3Wvs7fdXExhzQR57fRx+VoACnr0p+SZFprFk8 wCR59CRXaGqRQNbQT9XQ6WA/d/dN/+Yy0BTV072iPUjvfEUxGCr4h72iwxYvnne4uj8U N4oHpa5KXZcIBniguH6JszII8kYkmkPFPdOZ1wvxgicsdORmOAQpiuOBJy0D7Vs7+u3q eeig== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1730027706; x=1730632506; 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=3yR+elAtNrZb0dYqaY4oOlXhVHYbUSfEfohSukSkC5o=; b=ikM0x4FZLzK552jCHpR+i18CGMyQjX61l78HCuxeGZag5hzIniHzyiAogM1AR8nEVe zCjmY3q3BG5h0CzL/8OmY/Gawg4i/qox6vtEBhnlipGaFWip52fvgC8K4P0vPyjk9Cn3 DhD3UYBhNBv3sF2+3l1HHnMat25ZT07UHJ199jutN7MDP12/UEZA7+iximLtF0fiaQ76 s7bX4QQjzJISMoJIW+IBL28MCCr2rhdvJ5tWfQ5d0tlSAjybPIw/CKHaLnI6LpU7sYXu hHn0AtceHpDWBKxTWQ6KPD8h57c4TDzwxTrnn6YvBT01VPh5gJoqb+vtWIKa304l7Fh9 fTKw== X-Forwarded-Encrypted: i=1; AJvYcCXI1FHr4piSnAcEF6i98mQ1uFQStGr5GK+VyRTA7l7rHiL8Y1TdWoe7JpuVonZN+AOoTl/fhabbXXE=@vger.kernel.org X-Gm-Message-State: AOJu0YwF2OIN0W+wwXKOrY6FwpbMoyHxBUCmLeyzWft48s1H+IzD1Fvm E0l8vQp8vH+5DVfjxurSge3dYIqP6sFjJ6shm+JySl8BI3Hl0CeJ1hPCoUd2SuU= X-Google-Smtp-Source: AGHT+IFOjK3SXwMnrJ05PYWHAYrOeHgTV8mQIzdbz7LLP2yDbdJwRnYmcIkVqvkxVFzW4DkqcW0Zzw== X-Received: by 2002:adf:f884:0:b0:37c:d1ea:f1ce with SMTP id ffacd0b85a97d-380611a46famr4631904f8f.25.1730027705624; Sun, 27 Oct 2024 04:15:05 -0700 (PDT) Received: from localhost (host-79-35-211-193.retail.telecomitalia.it. [79.35.211.193]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a9b1f0697easm270520366b.94.2024.10.27.04.15.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 27 Oct 2024 04:15:05 -0700 (PDT) From: Andrea della Porta X-Google-Original-From: Andrea della Porta Date: Sun, 27 Oct 2024 12:15:28 +0100 To: Herve Codina Cc: Andrea della Porta , Stephen Boyd , Andrew Lunn , Arnd Bergmann , Bartosz Golaszewski , Bjorn Helgaas , Broadcom internal kernel review list , Catalin Marinas , Conor Dooley , Derek Kiernan , Dragan Cvetic , Florian Fainelli , Greg Kroah-Hartman , Krzysztof Kozlowski , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Linus Walleij , Lorenzo Pieralisi , Luca Ceresoli , Manivannan Sadhasivam , Masahiro Yamada , Michael Turquette , Rob Herring , Saravana Kannan , St efan Wahren , Thomas Petazzoni , Will Deacon , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, phil@raspberrypi.com, jonathan@raspberrypi.com Subject: Re: [PATCH v2 08/14] clk: rp1: Add support for clocks provided by RP1 Message-ID: References: <022cf4920f8147cc720eaf02fd52c0fa56f565c5.1728300189.git.andrea.porta@suse.com> <611de50b5f083ea4c260f920ccc0e300.sboyd@kernel.org> <20241023183240.0dffae03@bootlin.com> Precedence: bulk X-Mailing-List: linux-pci@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: <20241023183240.0dffae03@bootlin.com> Hi Herve, On 18:32 Wed 23 Oct , Herve Codina wrote: > Hi Andrea, > > > On Wed, 23 Oct 2024 17:36:06 +0200 > Andrea della Porta wrote: > > > Hi Stephen, > > > > On 15:08 Wed 09 Oct , Stephen Boyd wrote: > > > Quoting Andrea della Porta (2024-10-07 05:39:51) > > > > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > > > > index 299bc678ed1b..537019987f0c 100644 > > > > --- a/drivers/clk/Kconfig > > > > +++ b/drivers/clk/Kconfig > > > > @@ -88,6 +88,15 @@ config COMMON_CLK_RK808 > > > > These multi-function devices have two fixed-rate oscillators, clocked at 32KHz each. > > > > Clkout1 is always on, Clkout2 can off by control register. > > > > > > > > +config COMMON_CLK_RP1 > > > > + tristate "Raspberry Pi RP1-based clock support" > > > > + depends on PCI || COMPILE_TEST > > > > > > A better limit would be some ARCH_* config. > > > > I've avoided ARCH_BCM2835 since the original intention is for this driver > > to work (in the future) also for custom PCI cards with RP1 on-board, and not > > only for Rpi5. > > Maybe depends on CONFIG_MISC_RP1 ? > CONFIG_MISC_RP1 enables the RP1 PCI driver Yep, seems the most logical one, thanks. Regards, Andrea > > Best regards, > Hervé