From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f54.google.com (mail-wr1-f54.google.com [209.85.221.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 57676143C45 for ; Sun, 27 Oct 2024 11:15:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730027710; cv=none; b=JVNFMdpE93zAETuLBjm6s3eX9FzWJdDRoreveX+d3Fr/cXpcJ8R+Hz1IocMLAywakIA95K4kAdKT7FqyFDf3UAdEtp4X11z+BR9EVEM3I+tBe9w8KVVz41AoQUVf46wq4tb3D+FjJWm+fnY6E/ryE0tCFWOE/QX8MNyj3j94AY0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730027710; c=relaxed/simple; bh=xDDeU6p25FtrTyo3f2uKG58qzH1nBHIOOMUYrGXVBAs=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=dIrNTRgu21W8E77dYrIeV2TNBCnCmJYc9s5COV3+5nBXM0UO+jFUfHWs/3BYy01O70B0Cmcd9rh3p1DCwoWZB3ZQH3BtpTxATWWMLm8MlhGj2/r1c7HO3kx4UuZrLIteqP7Np0x+bcrwJgzfAV+4jRi8uEtW2J1kR2d7AJcpxBs= 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.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="dAwRgyZa" Received: by mail-wr1-f54.google.com with SMTP id ffacd0b85a97d-37d495d217bso3182540f8f.0 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=vDjhNrfNrWGQQO7GbZTusgveIvZD/MrindrB6AFqXgP0uImNFGSVbnOm181pCHEJgG rFzNhyJsnoJcvzEhXd6MCo2s6QDMhDgGetQ/aCYKNGFwGwO0vORwB2E9E+XGnQppBl51 yNgZWQwVU+6vSdzm+h8doCPKqtvILxp1rMPeZ/C2g8FMTsEig1z/w1phrT74cPm9/AIn MegaZa0p9vedsZiirp3vD8sPLEX4BhiwVmcrUZKz3Z4CNS3/nhHiHiZzYkWu6NDvNj16 rhhCbFcinGJ0EmvSt3IXDE+IWSLeP6EZ53w7kP4mM2oXG/0bmV7t17ak9A05WI8f2Ff9 SsFQ== X-Forwarded-Encrypted: i=1; AJvYcCUDFcI6/v50YBm2/S0X0q05oUrQD9nqa21amzVzP51/Yio6XHgJHYEq9umoHRVwUgWM3JNclYSPazzN@vger.kernel.org X-Gm-Message-State: AOJu0YxGxpp6IOnPO5zBD/tidYB9eXeaHJuTJt9IOYKt/5rc5uC8zgQL 9g2vjGvGyQbziDalz3iD5W6lSTCXsX8Y1roz+OFWs/LH8l53ug/usTdYtHD60Yk= 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-gpio@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é