lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240820-baritone-delegate-5711f7a0bc76@spud>
Date: Tue, 20 Aug 2024 17:19:02 +0100
From: Conor Dooley <conor@...nel.org>
To: Andrea della Porta <andrea.porta@...e.com>
Cc: Michael Turquette <mturquette@...libre.com>,
	Stephen Boyd <sboyd@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Florian Fainelli <florian.fainelli@...adcom.com>,
	Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>,
	Linus Walleij <linus.walleij@...aro.org>,
	Catalin Marinas <catalin.marinas@....com>,
	Will Deacon <will@...nel.org>,
	Derek Kiernan <derek.kiernan@....com>,
	Dragan Cvetic <dragan.cvetic@....com>,
	Arnd Bergmann <arnd@...db.de>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Nicolas Ferre <nicolas.ferre@...rochip.com>,
	Claudiu Beznea <claudiu.beznea@...on.dev>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Saravana Kannan <saravanak@...gle.com>,
	Bjorn Helgaas <bhelgaas@...gle.com>, linux-clk@...r.kernel.org,
	devicetree@...r.kernel.org, linux-rpi-kernel@...ts.infradead.org,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	linux-gpio@...r.kernel.org, netdev@...r.kernel.org,
	linux-pci@...r.kernel.org, linux-arch@...r.kernel.org,
	Lee Jones <lee@...nel.org>, Andrew Lunn <andrew@...n.ch>,
	Stefan Wahren <wahrenst@....net>
Subject: Re: [PATCH 01/11] dt-bindings: clock: Add RaspberryPi RP1 clock
 bindings

On Tue, Aug 20, 2024 at 04:36:03PM +0200, Andrea della Porta wrote:
> Add device tree bindings for the clock generator found in RP1 multi
> function device, and relative entries in MAINTAINERS file.
> 
> Signed-off-by: Andrea della Porta <andrea.porta@...e.com>
> ---
>  .../clock/raspberrypi,rp1-clocks.yaml         | 87 +++++++++++++++++++
>  MAINTAINERS                                   |  6 ++
>  include/dt-bindings/clock/rp1.h               | 56 ++++++++++++
>  3 files changed, 149 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/clock/raspberrypi,rp1-clocks.yaml
>  create mode 100644 include/dt-bindings/clock/rp1.h
> 
> diff --git a/Documentation/devicetree/bindings/clock/raspberrypi,rp1-clocks.yaml b/Documentation/devicetree/bindings/clock/raspberrypi,rp1-clocks.yaml
> new file mode 100644
> index 000000000000..b27db86d0572
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/raspberrypi,rp1-clocks.yaml
> @@ -0,0 +1,87 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/clock/raspberrypi,rp1-clocks.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: RaspberryPi RP1 clock generator
> +
> +maintainers:
> +  - Andrea della Porta <andrea.porta@...e.com>
> +
> +description: |
> +  The RP1 contains a clock generator designed as three PLLs (CORE, AUDIO,
> +  VIDEO), and each PLL output can be programmed though dividers to generate
> +  the clocks to drive the sub-peripherals embedded inside the chipset.
> +
> +  Link to datasheet:
> +  https://datasheets.raspberrypi.com/rp1/rp1-peripherals.pdf
> +
> +properties:
> +  compatible:
> +    const: raspberrypi,rp1-clocks
> +
> +  reg:
> +    maxItems: 1
> +
> +  '#clock-cells':
> +    description:
> +      The index in the assigned-clocks is mapped to the output clock as per
> +      definitions in dt-bindings/clock/rp1.h.
> +    const: 1
> +
> +  clocks:
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +  - '#clock-cells'
> +  - clocks
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/clock/rp1.h>
> +
> +    rp1 {
> +        #address-cells = <2>;
> +        #size-cells = <2>;
> +
> +        rp1_clocks: clocks@...00 {

The unit address does not match the reg property. I'm surprised that
dtc doesn't complain about that.

> +            compatible = "raspberrypi,rp1-clocks";
> +            reg = <0xc0 0x40018000 0x0 0x10038>;

This is a rather oddly specific size. It leads me to wonder if this
region is inside some sort of syscon area?

> +            #clock-cells = <1>;
> +            clocks = <&clk_xosc>;
> +
> +            assigned-clocks = <&rp1_clocks RP1_PLL_SYS_CORE>,

FWIW, I don't think any of these assigned clocks are helpful for the
example. That said, why do you need to configure all of these assigned
clocks via devicetree when this node is the provider of them?

> +                              <&rp1_clocks RP1_PLL_AUDIO_CORE>,
> +                              /* RP1_PLL_VIDEO_CORE and dividers are now managed by VEC,DPI drivers */

Comments like this also do not seem relevant to the binding.


Cheers,
Conor.


> +                              <&rp1_clocks RP1_PLL_SYS>,
> +                              <&rp1_clocks RP1_PLL_SYS_SEC>,
> +                              <&rp1_clocks RP1_PLL_AUDIO>,
> +                              <&rp1_clocks RP1_PLL_AUDIO_SEC>,
> +                              <&rp1_clocks RP1_CLK_SYS>,
> +                              <&rp1_clocks RP1_PLL_SYS_PRI_PH>,
> +                              /* RP1_CLK_SLOW_SYS is used for the frequency counter (FC0) */
> +                              <&rp1_clocks RP1_CLK_SLOW_SYS>,
> +                              <&rp1_clocks RP1_CLK_SDIO_TIMER>,
> +                              <&rp1_clocks RP1_CLK_SDIO_ALT_SRC>,
> +                              <&rp1_clocks RP1_CLK_ETH_TSU>;
> +
> +            assigned-clock-rates = <1000000000>, // RP1_PLL_SYS_CORE
> +                                   <1536000000>, // RP1_PLL_AUDIO_CORE
> +                                   <200000000>,  // RP1_PLL_SYS
> +                                   <125000000>,  // RP1_PLL_SYS_SEC
> +                                   <61440000>,   // RP1_PLL_AUDIO
> +                                   <192000000>,  // RP1_PLL_AUDIO_SEC
> +                                   <200000000>,  // RP1_CLK_SYS
> +                                   <100000000>,  // RP1_PLL_SYS_PRI_PH
> +                                   /* Must match the XOSC frequency */
> +                                   <50000000>, // RP1_CLK_SLOW_SYS
> +                                   <1000000>, // RP1_CLK_SDIO_TIMER
> +                                   <200000000>, // RP1_CLK_SDIO_ALT_SRC
> +                                   <50000000>; // RP1_CLK_ETH_TSU
> +        };
> +    };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 42decde38320..6e7db9bce278 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19116,6 +19116,12 @@ F:	Documentation/devicetree/bindings/media/raspberrypi,pispbe.yaml
>  F:	drivers/media/platform/raspberrypi/pisp_be/
>  F:	include/uapi/linux/media/raspberrypi/
>  
> +RASPBERRY PI RP1 PCI DRIVER
> +M:	Andrea della Porta <andrea.porta@...e.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/clock/raspberrypi,rp1-clocks.yaml
> +F:	include/dt-bindings/clock/rp1.h
> +
>  RC-CORE / LIRC FRAMEWORK
>  M:	Sean Young <sean@...s.org>
>  L:	linux-media@...r.kernel.org
> diff --git a/include/dt-bindings/clock/rp1.h b/include/dt-bindings/clock/rp1.h
> new file mode 100644
> index 000000000000..1ed67b8a5229
> --- /dev/null
> +++ b/include/dt-bindings/clock/rp1.h
> @@ -0,0 +1,56 @@
> +/* SPDX-License-Identifier: GPL-2.0 OR MIT */
> +/*
> + * Copyright (C) 2021 Raspberry Pi Ltd.
> + */
> +
> +#define RP1_PLL_SYS_CORE		0
> +#define RP1_PLL_AUDIO_CORE		1
> +#define RP1_PLL_VIDEO_CORE		2
> +
> +#define RP1_PLL_SYS			3
> +#define RP1_PLL_AUDIO			4
> +#define RP1_PLL_VIDEO			5
> +
> +#define RP1_PLL_SYS_PRI_PH		6
> +#define RP1_PLL_SYS_SEC_PH		7
> +#define RP1_PLL_AUDIO_PRI_PH		8
> +
> +#define RP1_PLL_SYS_SEC			9
> +#define RP1_PLL_AUDIO_SEC		10
> +#define RP1_PLL_VIDEO_SEC		11
> +
> +#define RP1_CLK_SYS			12
> +#define RP1_CLK_SLOW_SYS		13
> +#define RP1_CLK_DMA			14
> +#define RP1_CLK_UART			15
> +#define RP1_CLK_ETH			16
> +#define RP1_CLK_PWM0			17
> +#define RP1_CLK_PWM1			18
> +#define RP1_CLK_AUDIO_IN		19
> +#define RP1_CLK_AUDIO_OUT		20
> +#define RP1_CLK_I2S			21
> +#define RP1_CLK_MIPI0_CFG		22
> +#define RP1_CLK_MIPI1_CFG		23
> +#define RP1_CLK_PCIE_AUX		24
> +#define RP1_CLK_USBH0_MICROFRAME	25
> +#define RP1_CLK_USBH1_MICROFRAME	26
> +#define RP1_CLK_USBH0_SUSPEND		27
> +#define RP1_CLK_USBH1_SUSPEND		28
> +#define RP1_CLK_ETH_TSU			29
> +#define RP1_CLK_ADC			30
> +#define RP1_CLK_SDIO_TIMER		31
> +#define RP1_CLK_SDIO_ALT_SRC		32
> +#define RP1_CLK_GP0			33
> +#define RP1_CLK_GP1			34
> +#define RP1_CLK_GP2			35
> +#define RP1_CLK_GP3			36
> +#define RP1_CLK_GP4			37
> +#define RP1_CLK_GP5			38
> +#define RP1_CLK_VEC			39
> +#define RP1_CLK_DPI			40
> +#define RP1_CLK_MIPI0_DPI		41
> +#define RP1_CLK_MIPI1_DPI		42
> +
> +/* Extra PLL output channels - RP1B0 only */
> +#define RP1_PLL_VIDEO_PRI_PH		43
> +#define RP1_PLL_AUDIO_TERN		44
> -- 
> 2.35.3
> 

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ