From patchwork Sun Dec 22 17:04:15 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dario Binacchi X-Patchwork-Id: 13918169 Received: from mail-ej1-f52.google.com (mail-ej1-f52.google.com [209.85.218.52]) (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 C526039FD9 for ; Sun, 22 Dec 2024 17:05:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734887141; cv=none; b=UUIdZy7r8by0gyEqC26fTZVF1L0A7A1mhM41wdOGqC2VuC/Bs78+ynnxdzDJXrJKNIBHBxzxF5RmKVcYpXQ6Ng1HEDPxr4JKU0/tQJKxXK7fDR3l4gpQDwWbGV1p8DQLbjkoJ8BivhFZv+qBX6BwNd3JLrIjp/mT1Brm38hNryc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734887141; c=relaxed/simple; bh=FduH27wYIXMaSCrVN8+PHj+Jt+MI3H3TFAHd9zqdL70=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=q22o+kHK3wLGT3bzWdosphaONc2MenfOSSUcucmK+yPczZNPRoj+q9aNzeY6n34iNNPthNJYyVV7yhVxNh3LR4YGTfdzeDv7OuVLTNW4VvDL+PycvDZWfcCCHxoNl7OfM6WJ9OPDnn8gtWMDO3uXp3RbVsRP7vJlYWi2H76F93Y= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=amarulasolutions.com; spf=pass smtp.mailfrom=amarulasolutions.com; dkim=pass (1024-bit key) header.d=amarulasolutions.com header.i=@amarulasolutions.com header.b=TlEhV6r4; arc=none smtp.client-ip=209.85.218.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=amarulasolutions.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=amarulasolutions.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amarulasolutions.com header.i=@amarulasolutions.com header.b="TlEhV6r4" Received: by mail-ej1-f52.google.com with SMTP id a640c23a62f3a-aa6c0d1833eso727133166b.1 for ; Sun, 22 Dec 2024 09:05:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amarulasolutions.com; s=google; t=1734887138; x=1735491938; darn=lists.linux.dev; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=RhtmpZ+ZLYSsrqpu6pU/Lyj2nnsgcpK7OCWx/PXTdug=; b=TlEhV6r4w3ZRpv4a48cIgw/Hiwn4kud6DhJjq5pzOwB++FdXfS06tlUyu75RZrueLu bwBk6P7NitYVrcIlM/Vl7KLdzaKyU537XPzp5vhsGsXD+Gex72b7dnVYMGPm5Yke2ZXl A4ZyUXlAFY3zWvYkHsybwwaMhBxwSHNdeicTk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1734887138; x=1735491938; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=RhtmpZ+ZLYSsrqpu6pU/Lyj2nnsgcpK7OCWx/PXTdug=; b=gCI/KONB8oLeOIvQJMq8dnkXm/mbsg2oCiTbSIok6JXY1O3i8OdXfFs/rgAoo0mp2h en1NZ+L68N0EbVqUEmedvPZ44+YOkedcp/szMbjn5tZpGpOVxzyl8TAj+L/DXHYrES9I md4B50lozxhNrNFJ8NtXZIciYXHJuCxMMcscqjE83pTa9xYRr6GBS2kmsH74/2fIxpTI oAmNtSaNtLAo6lCL9OPhcikrixR+LR++hWwGGxTfq5teGJb9NNleMkAvAg5dI4VIwLNH OIW6FZG+giXgR4vMpYfE3JANCXpw86b+2+8sGwm1YrVwHkncu7fiYtYKcRSjel6NcD2P iiug== X-Forwarded-Encrypted: i=1; AJvYcCXMZL9EZZgjarNhqLrNxN6UUutM+Xto9s+sNUFx+7ox0+hJsvTim6jp/7Z/de9WrRsfGTc=@lists.linux.dev X-Gm-Message-State: AOJu0YwBxD9Hj46dNd3TpdfygOr+e4lw1E7DS4FsRSoxB4C8v4OMKL8N WcMAGM8n7iJ9vVPcaPI92xFlBz0AjP+HjkjbquB+AI3MkmTzMuhpSRIqrqtqmuk= X-Gm-Gg: ASbGnctnVpnvJePzzUhPhi24ti+CrKv/IrvyAbAMpMaArQwZ3Fdxzm7rcWeCxAQGYhp a9Kp91HP+UEI1jWF9BDvJxPC3A72z4UzzjmmHNbFtrqPoC5lXV0judhBVdGWbAsM214HG+yTm7N 3a8JEF8fUARaoZODk+N08w1T9hjg2Zi8IDPFzSWDHP+pMP/6xxZwO887QOpUz1K30u/NFZYFiJu ACf3jbhWFe9kkO79GON7addfhnXHasTWf8XMPw9e/+OWx1+G/0UggbH+ewMm0vS+18wydcQbTcR KR8YFxP32mRjykFG2yuoI72Ir0iKKyurKG3riMuP6buOGw== X-Google-Smtp-Source: AGHT+IHvooGlO7jo5Rwpwn79hajX/ICZcskpjcUPejd9ekku1o6JfbpIJup3jevoKr3K6wJwFlFcgg== X-Received: by 2002:a17:907:1819:b0:aae:8308:532c with SMTP id a640c23a62f3a-aae830854dcmr621726066b.59.1734887137954; Sun, 22 Dec 2024 09:05:37 -0800 (PST) Received: from dario-ThinkPad-T14s-Gen-2i.amarulasolutions.com ([2.196.41.87]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-aac0efe48d6sm414056566b.127.2024.12.22.09.05.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 22 Dec 2024 09:05:37 -0800 (PST) From: Dario Binacchi To: linux-kernel@vger.kernel.org Cc: linux-amarula@amarulasolutions.com, Dario Binacchi , Abel Vesa , Conor Dooley , Fabio Estevam , Krzysztof Kozlowski , Michael Turquette , Peng Fan , Pengutronix Kernel Team , Rob Herring , Sascha Hauer , Shawn Guo , Stephen Boyd , devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org Subject: [PATCH v6 00/18] Support spread spectrum clocking for i.MX8N PLLs Date: Sun, 22 Dec 2024 18:04:15 +0100 Message-ID: <20241222170534.3621453-1-dario.binacchi@amarulasolutions.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 The series adds support for spread spectrum clocking for i.MX8MN PLLs (audio, video and DRAM). It has been tested for the video PLL on a board using i.MX8MN. The patches added in version 4, such as the dt-bindings and the driver for anatop, were inspired by the extensive email exchange from version 3: https://lore.kernel.org/imx/20241106090549.3684963-1-dario.binacchi@amarulasolutions.com/ The series added spectrum spread support for the imx8mn platform only, but in case it was merged, confirming that the directives and suggestions made by the maintainers were correctly understood and implemented, I will extend this support to the imx8mm and imx8mp platforms as well. Changes in v6: - Merge patches: 10/20 dt-bindings: clock: imx8mm: add binding definitions for anatop 11/20 dt-bindings: clock: imx8mn: add binding definitions for anatop 12/20 dt-bindings: clock: imx8mp: add binding definitions for anatop to 05/20 dt-bindings: clock: imx8m-anatop: define clocks/clock-names now renamed 05/18 dt-bindings: clock: imx8m-anatop: add oscillators and PLLs - Split the patch 15/20 dt-bindings-clock-imx8m-clock-support-spread-spectru.patch into 12/18 dt-bindings: clock: imx8m-clock: add PLLs 16/18 dt-bindings: clock: imx8m-clock: support spread spectrum clocking Changes in v5: - Fix compilation errors. - Separate driver code from dt-bindings Changes in v4: - Add dt-bindings for anatop - Add anatop driver - Drop fsl,ssc-clocks from spread spectrum dt-bindings Changes in v3: - Patches 1/8 has been added in version 3. The dt-bindings have been moved from fsl,imx8m-anatop.yaml to imx8m-clock.yaml. The anatop device (fsl,imx8m-anatop.yaml) is indeed more or less a syscon, so it represents a memory area accessible by ccm (imx8m-clock.yaml) to setup the PLLs. - Patches {3,5}/8 have been added in version 3. - Patches {4,6,8}/8 use ccm device node instead of the anatop one. Changes in v2: - Add "allOf:" and place it after "required:" block, like in the example schema. - Move the properties definition to the top-level. - Drop unit types as requested by the "make dt_binding_check" command. Dario Binacchi (18): dt-bindings: clock: imx8mm: add VIDEO_PLL clocks clk: imx8mm: rename video_pll1 to video_pll dt-bindings: clock: imx8mp: add VIDEO_PLL clocks clk: imx8mp: rename video_pll1 to video_pll dt-bindings: clock: imx8m-anatop: add oscillators and PLLs arm64: dts: imx8mm: add anatop clocks arm64: dts: imx8mn: add anatop clocks arm64: dts: imx8mp: add anatop clocks arm64: dts: imx8mq: add anatop clocks clk: imx: add hw API imx8m_anatop_get_clk_hw clk: imx: add support for i.MX8MN anatop clock driver dt-bindings: clock: imx8m-clock: add PLLs arm64: dts: imx8mm: add PLLs to clock controller module (ccm) arm64: dts: imx8mn: add PLLs to clock controller module (ccm) arm64: dts: imx8mp: add PLLs to clock controller module (ccm) dt-bindings: clock: imx8m-clock: support spread spectrum clocking clk: imx: pll14xx: support spread spectrum clock generation clk: imx8mn: support spread spectrum clock generation .../bindings/clock/fsl,imx8m-anatop.yaml | 53 +++- .../bindings/clock/imx8m-clock.yaml | 65 +++- arch/arm64/boot/dts/freescale/imx8mm.dtsi | 11 +- arch/arm64/boot/dts/freescale/imx8mn.dtsi | 11 +- arch/arm64/boot/dts/freescale/imx8mp.dtsi | 11 +- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 + drivers/clk/imx/Makefile | 2 +- drivers/clk/imx/clk-imx8mm.c | 102 +++---- drivers/clk/imx/clk-imx8mn-anatop.c | 283 ++++++++++++++++++ drivers/clk/imx/clk-imx8mn.c | 188 ++++++------ drivers/clk/imx/clk-imx8mp.c | 118 ++++---- drivers/clk/imx/clk-pll14xx.c | 134 +++++++++ drivers/clk/imx/clk.c | 28 ++ drivers/clk/imx/clk.h | 23 ++ include/dt-bindings/clock/imx8mm-clock.h | 76 ++++- include/dt-bindings/clock/imx8mn-clock.h | 64 ++++ include/dt-bindings/clock/imx8mp-clock.h | 80 ++++- 17 files changed, 1017 insertions(+), 234 deletions(-) create mode 100644 drivers/clk/imx/clk-imx8mn-anatop.c