From nobody Wed Nov 27 11:40:10 2024 Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6D5E318BB8E; Thu, 10 Oct 2024 06:16:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.196 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540987; cv=none; b=XN8oOGRhWjCjK7VYaq8jumLnmxawqF/BtQ0VGcFltbLQ4qYzmrMe7ugwiQx7aZ6QHoykHlQ/OcFz7BVhgxe8yQp7IL+QyrtEHASSQON/GOqwR4isBo8VPLgVfJggyN8FFSnbdYCgaaRFITALQKw11Ozh2Eoedd3QQvhG+ObUbXg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540987; c=relaxed/simple; bh=09NsXZJb4Q2DRy8Vnj85lLb/Wq+JhM0CD6UP61GzZEU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=mPTckQVoa1bszl0mtzszjfSfdbfd2HkmFeqTdVZpTQ5Hhk+OW011Ebuzg4PYPyWIv9CVXkPHnFw3joh3Qr6BI4LA3FwaFSzoNFbMxNM2was5uMInHYwKmIrPcdIcpGh9Zxx18p6B7OY/kwJSChw47YsHVQTGUP9sf+QEx44F6YM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=GtfzXQ8u; arc=none smtp.client-ip=217.70.183.196 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="GtfzXQ8u" Received: by mail.gandi.net (Postfix) with ESMTPA id DD5D9E0002; Thu, 10 Oct 2024 06:16:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1728540982; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=VE8PV1Xaq1ZMgO/XF6cBMa+Y9p/NzyI5lCHztbLRL3g=; b=GtfzXQ8ulL1PF3nyMfB7233snzv/8mFHXfARUgftugPEEJWeo6LZIy2SLU14J53SsRMC4m OM3AjpI+5u7VH9qskrC8gLNz3DEtL7blpTH82ccK+8PQcv656iiuqXSoYX9DJW+quo/oSH 3XxAGAe9knmS+Kx4i45j06Qcl1f9AFzZ4Fu8/3VfkDIWMp5gA9DVjUCiEK3Qj5FRf+5zSR LTkvPiZGaYHOENWWxXEwHX7mwGrcnrA1aWBIunLWl0tk/Sx6A+VYcN2wyU3cGsOoHunjkQ CxgYQV4/8NoI0WrvaC0lhCI2yl8duGXh5+rpqFpAIdEOscTV3eIXtkMaBESPZg== From: Herve Codina To: Geert Uytterhoeven , Andy Shevchenko , Simon Horman , Lee Jones , Arnd Bergmann , Derek Kiernan , Dragan Cvetic , Greg Kroah-Hartman , Herve Codina , Bjorn Helgaas , Philipp Zabel , Lars Povlsen , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Horatiu Vultur , Andrew Lunn , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Allan Nielsen , Steen Hegelund , Luca Ceresoli , Thomas Petazzoni Subject: [PATCH v8 1/6] misc: Add support for LAN966x PCI device Date: Thu, 10 Oct 2024 08:16:09 +0200 Message-ID: <20241010061615.787073-2-herve.codina@bootlin.com> X-Mailer: git-send-email 2.46.2 In-Reply-To: <20241010061615.787073-1-herve.codina@bootlin.com> References: <20241010061615.787073-1-herve.codina@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-GND-Sasl: herve.codina@bootlin.com Add a PCI driver that handles the LAN966x PCI device using a device-tree overlay. This overlay is applied to the PCI device DT node and allows to describe components that are present in the device. The memory from the device-tree is remapped to the BAR memory thanks to "ranges" properties computed at runtime by the PCI core during the PCI enumeration. The PCI device itself acts as an interrupt controller and is used as the parent of the internal LAN966x interrupt controller to route the interrupts to the assigned PCI INTx interrupt. Signed-off-by: Herve Codina Acked-by: Greg Kroah-Hartman --- drivers/misc/Kconfig | 24 ++++ drivers/misc/Makefile | 3 + drivers/misc/lan966x_pci.c | 215 ++++++++++++++++++++++++++++++++++ drivers/misc/lan966x_pci.dtso | 167 ++++++++++++++++++++++++++ drivers/pci/quirks.c | 1 + 5 files changed, 410 insertions(+) create mode 100644 drivers/misc/lan966x_pci.c create mode 100644 drivers/misc/lan966x_pci.dtso diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig index 3fe7e2a9bd29..8e5b06ac9b6f 100644 --- a/drivers/misc/Kconfig +++ b/drivers/misc/Kconfig @@ -610,6 +610,30 @@ config MARVELL_CN10K_DPI To compile this driver as a module, choose M here: the module will be called mrvl_cn10k_dpi. =20 +config MCHP_LAN966X_PCI + tristate "Microchip LAN966x PCIe Support" + depends on PCI + select OF + select OF_OVERLAY + select IRQ_DOMAIN + help + This enables the support for the LAN966x PCIe device. + This is used to drive the LAN966x PCIe device from the host system + to which it is connected. + + This driver uses an overlay to load other drivers to support for + LAN966x internal components. + Even if this driver does not depend on these other drivers, in order + to have a fully functional board, the following drivers are needed: + - fixed-clock (COMMON_CLK) + - lan966x-oic (LAN966X_OIC) + - lan966x-cpu-syscon (MFD_SYSCON) + - lan966x-switch-reset (RESET_MCHP_SPARX5) + - lan966x-pinctrl (PINCTRL_OCELOT) + - lan966x-serdes (PHY_LAN966X_SERDES) + - lan966x-miim (MDIO_MSCC_MIIM) + - lan966x-switch (LAN966X_SWITCH) + source "drivers/misc/c2port/Kconfig" source "drivers/misc/eeprom/Kconfig" source "drivers/misc/cb710/Kconfig" diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile index a9f94525e181..885b22989580 100644 --- a/drivers/misc/Makefile +++ b/drivers/misc/Makefile @@ -71,4 +71,7 @@ obj-$(CONFIG_TPS6594_ESM) +=3D tps6594-esm.o obj-$(CONFIG_TPS6594_PFSM) +=3D tps6594-pfsm.o obj-$(CONFIG_NSM) +=3D nsm.o obj-$(CONFIG_MARVELL_CN10K_DPI) +=3D mrvl_cn10k_dpi.o +lan966x-pci-objs :=3D lan966x_pci.o +lan966x-pci-objs +=3D lan966x_pci.dtbo.o +obj-$(CONFIG_MCHP_LAN966X_PCI) +=3D lan966x-pci.o obj-y +=3D keba/ diff --git a/drivers/misc/lan966x_pci.c b/drivers/misc/lan966x_pci.c new file mode 100644 index 000000000000..9c79b58137e5 --- /dev/null +++ b/drivers/misc/lan966x_pci.c @@ -0,0 +1,215 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Microchip LAN966x PCI driver + * + * Copyright (c) 2024 Microchip Technology Inc. and its subsidiaries. + * + * Authors: + * Cl=C3=A9ment L=C3=A9ger + * Herv=C3=A9 Codina + */ + +#include +#include +#include +#include +#include +#include +#include +#include + +/* Embedded dtbo symbols created by cmd_wrap_S_dtb in scripts/Makefile.lib= */ +extern char __dtbo_lan966x_pci_begin[]; +extern char __dtbo_lan966x_pci_end[]; + +struct pci_dev_intr_ctrl { + struct pci_dev *pci_dev; + struct irq_domain *irq_domain; + int irq; +}; + +static int pci_dev_irq_domain_map(struct irq_domain *d, unsigned int virq,= irq_hw_number_t hw) +{ + irq_set_chip_and_handler(virq, &dummy_irq_chip, handle_simple_irq); + return 0; +} + +static const struct irq_domain_ops pci_dev_irq_domain_ops =3D { + .map =3D pci_dev_irq_domain_map, + .xlate =3D irq_domain_xlate_onecell, +}; + +static irqreturn_t pci_dev_irq_handler(int irq, void *data) +{ + struct pci_dev_intr_ctrl *intr_ctrl =3D data; + int ret; + + ret =3D generic_handle_domain_irq(intr_ctrl->irq_domain, 0); + return ret ? IRQ_NONE : IRQ_HANDLED; +} + +static struct pci_dev_intr_ctrl *pci_dev_create_intr_ctrl(struct pci_dev *= pdev) +{ + struct pci_dev_intr_ctrl *intr_ctrl __free(kfree) =3D NULL; + struct fwnode_handle *fwnode; + int ret; + + fwnode =3D dev_fwnode(&pdev->dev); + if (!fwnode) + return ERR_PTR(-ENODEV); + + intr_ctrl =3D kmalloc(sizeof(*intr_ctrl), GFP_KERNEL); + if (!intr_ctrl) + return ERR_PTR(-ENOMEM); + + intr_ctrl->pci_dev =3D pdev; + + intr_ctrl->irq_domain =3D irq_domain_create_linear(fwnode, 1, &pci_dev_ir= q_domain_ops, + intr_ctrl); + if (!intr_ctrl->irq_domain) { + pci_err(pdev, "Failed to create irqdomain\n"); + return ERR_PTR(-ENOMEM); + } + + ret =3D pci_alloc_irq_vectors(pdev, 1, 1, PCI_IRQ_INTX); + if (ret < 0) { + pci_err(pdev, "Unable alloc irq vector (%d)\n", ret); + goto err_remove_domain; + } + intr_ctrl->irq =3D pci_irq_vector(pdev, 0); + ret =3D request_irq(intr_ctrl->irq, pci_dev_irq_handler, IRQF_SHARED, + pci_name(pdev), intr_ctrl); + if (ret) { + pci_err(pdev, "Unable to request irq %d (%d)\n", intr_ctrl->irq, ret); + goto err_free_irq_vector; + } + + return_ptr(intr_ctrl); + +err_free_irq_vector: + pci_free_irq_vectors(pdev); +err_remove_domain: + irq_domain_remove(intr_ctrl->irq_domain); + return ERR_PTR(ret); +} + +static void pci_dev_remove_intr_ctrl(struct pci_dev_intr_ctrl *intr_ctrl) +{ + free_irq(intr_ctrl->irq, intr_ctrl); + pci_free_irq_vectors(intr_ctrl->pci_dev); + irq_dispose_mapping(irq_find_mapping(intr_ctrl->irq_domain, 0)); + irq_domain_remove(intr_ctrl->irq_domain); + kfree(intr_ctrl); +} + +static void devm_pci_dev_remove_intr_ctrl(void *intr_ctrl) +{ + pci_dev_remove_intr_ctrl(intr_ctrl); +} + +static int devm_pci_dev_create_intr_ctrl(struct pci_dev *pdev) +{ + struct pci_dev_intr_ctrl *intr_ctrl; + + intr_ctrl =3D pci_dev_create_intr_ctrl(pdev); + if (IS_ERR(intr_ctrl)) + return PTR_ERR(intr_ctrl); + + return devm_add_action_or_reset(&pdev->dev, devm_pci_dev_remove_intr_ctrl= , intr_ctrl); +} + +struct lan966x_pci { + struct device *dev; + int ovcs_id; +}; + +static int lan966x_pci_load_overlay(struct lan966x_pci *data) +{ + u32 dtbo_size =3D __dtbo_lan966x_pci_end - __dtbo_lan966x_pci_begin; + void *dtbo_start =3D __dtbo_lan966x_pci_begin; + + return of_overlay_fdt_apply(dtbo_start, dtbo_size, &data->ovcs_id, dev_of= _node(data->dev)); +} + +static void lan966x_pci_unload_overlay(struct lan966x_pci *data) +{ + of_overlay_remove(&data->ovcs_id); +} + +static int lan966x_pci_probe(struct pci_dev *pdev, const struct pci_device= _id *id) +{ + struct device *dev =3D &pdev->dev; + struct lan966x_pci *data; + int ret; + + /* + * On ACPI system, fwnode can point to the ACPI node. + * This driver needs an of_node to be used as the device-tree overlay + * target. This of_node should be set by the PCI core if it succeeds in + * creating it (CONFIG_PCI_DYNAMIC_OF_NODES feature). + * Check here for the validity of this of_node. + */ + if (!dev_of_node(dev)) + return dev_err_probe(dev, -EINVAL, "Missing of_node for device\n"); + + /* Need to be done before devm_pci_dev_create_intr_ctrl. + * It allocates an IRQ and so pdev->irq is updated. + */ + ret =3D pcim_enable_device(pdev); + if (ret) + return ret; + + ret =3D devm_pci_dev_create_intr_ctrl(pdev); + if (ret) + return ret; + + data =3D devm_kzalloc(dev, sizeof(*data), GFP_KERNEL); + if (!data) + return -ENOMEM; + + pci_set_drvdata(pdev, data); + data->dev =3D dev; + + ret =3D lan966x_pci_load_overlay(data); + if (ret) + return ret; + + pci_set_master(pdev); + + ret =3D of_platform_default_populate(dev_of_node(dev), NULL, dev); + if (ret) + goto err_unload_overlay; + + return 0; + +err_unload_overlay: + lan966x_pci_unload_overlay(data); + return ret; +} + +static void lan966x_pci_remove(struct pci_dev *pdev) +{ + struct lan966x_pci *data =3D pci_get_drvdata(pdev); + + of_platform_depopulate(data->dev); + + lan966x_pci_unload_overlay(data); +} + +static struct pci_device_id lan966x_pci_ids[] =3D { + { PCI_DEVICE(PCI_VENDOR_ID_EFAR, 0x9660) }, + { } +}; +MODULE_DEVICE_TABLE(pci, lan966x_pci_ids); + +static struct pci_driver lan966x_pci_driver =3D { + .name =3D "mchp_lan966x_pci", + .id_table =3D lan966x_pci_ids, + .probe =3D lan966x_pci_probe, + .remove =3D lan966x_pci_remove, +}; +module_pci_driver(lan966x_pci_driver); + +MODULE_AUTHOR("Herve Codina "); +MODULE_DESCRIPTION("Microchip LAN966x PCI driver"); +MODULE_LICENSE("GPL"); diff --git a/drivers/misc/lan966x_pci.dtso b/drivers/misc/lan966x_pci.dtso new file mode 100644 index 000000000000..94c271645ad5 --- /dev/null +++ b/drivers/misc/lan966x_pci.dtso @@ -0,0 +1,167 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2022 Microchip UNG + */ + +#include +#include +#include +#include +#include + +/dts-v1/; +/plugin/; + +/ { + fragment@0 { + target-path=3D""; + __overlay__ { + #address-cells =3D <3>; + #size-cells =3D <2>; + + pci-ep-bus@0 { + compatible =3D "simple-bus"; + #address-cells =3D <1>; + #size-cells =3D <1>; + + /* + * map @0xe2000000 (32MB) to BAR0 (CPU) + * map @0xe0000000 (16MB) to BAR1 (AMBA) + */ + ranges =3D <0xe2000000 0x00 0x00 0x00 0x2000000 + 0xe0000000 0x01 0x00 0x00 0x1000000>; + + oic: oic@e00c0120 { + compatible =3D "microchip,lan966x-oic"; + #interrupt-cells =3D <2>; + interrupt-controller; + interrupts =3D <0>; /* PCI INTx assigned interrupt */ + reg =3D <0xe00c0120 0x190>; + }; + + cpu_clk: cpu_clk { + compatible =3D "fixed-clock"; + #clock-cells =3D <0>; + clock-frequency =3D <600000000>; // CPU clock =3D 600MHz + }; + + ddr_clk: ddr_clk { + compatible =3D "fixed-clock"; + #clock-cells =3D <0>; + clock-frequency =3D <30000000>; // Fabric clock =3D 30MHz + }; + + sys_clk: sys_clk { + compatible =3D "fixed-clock"; + #clock-cells =3D <0>; + clock-frequency =3D <15625000>; // System clock =3D 15.625MHz + }; + + cpu_ctrl: syscon@e00c0000 { + compatible =3D "microchip,lan966x-cpu-syscon", "syscon"; + reg =3D <0xe00c0000 0xa8>; + }; + + reset: reset@e200400c { + compatible =3D "microchip,lan966x-switch-reset"; + reg =3D <0xe200400c 0x4>, <0xe00c0000 0xa8>; + reg-names =3D "gcb","cpu"; + #reset-cells =3D <1>; + cpu-syscon =3D <&cpu_ctrl>; + }; + + gpio: pinctrl@e2004064 { + compatible =3D "microchip,lan966x-pinctrl"; + reg =3D <0xe2004064 0xb4>, + <0xe2010024 0x138>; + resets =3D <&reset 0>; + reset-names =3D "switch"; + gpio-controller; + #gpio-cells =3D <2>; + gpio-ranges =3D <&gpio 0 0 78>; + interrupt-parent =3D <&oic>; + interrupt-controller; + interrupts =3D <17 IRQ_TYPE_LEVEL_HIGH>; + #interrupt-cells =3D <2>; + + tod_pins: tod_pins { + pins =3D "GPIO_36"; + function =3D "ptpsync_1"; + }; + + fc0_a_pins: fcb4-i2c-pins { + /* RXD, TXD */ + pins =3D "GPIO_9", "GPIO_10"; + function =3D "fc0_a"; + }; + + }; + + serdes: serdes@e202c000 { + compatible =3D "microchip,lan966x-serdes"; + reg =3D <0xe202c000 0x9c>, + <0xe2004010 0x4>; + #phy-cells =3D <2>; + }; + + mdio1: mdio@e200413c { + #address-cells =3D <1>; + #size-cells =3D <0>; + compatible =3D "microchip,lan966x-miim"; + reg =3D <0xe200413c 0x24>, + <0xe2010020 0x4>; + + resets =3D <&reset 0>; + reset-names =3D "switch"; + + lan966x_phy0: ethernet-lan966x_phy@1 { + reg =3D <1>; + }; + + lan966x_phy1: ethernet-lan966x_phy@2 { + reg =3D <2>; + }; + }; + + switch: switch@e0000000 { + compatible =3D "microchip,lan966x-switch"; + reg =3D <0xe0000000 0x0100000>, + <0xe2000000 0x0800000>; + reg-names =3D "cpu", "gcb"; + + interrupt-parent =3D <&oic>; + interrupts =3D <12 IRQ_TYPE_LEVEL_HIGH>, + <9 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names =3D "xtr", "ana"; + + resets =3D <&reset 0>; + reset-names =3D "switch"; + + pinctrl-names =3D "default"; + pinctrl-0 =3D <&tod_pins>; + + ethernet-ports { + #address-cells =3D <1>; + #size-cells =3D <0>; + + port0: port@0 { + phy-handle =3D <&lan966x_phy0>; + + reg =3D <0>; + phy-mode =3D "gmii"; + phys =3D <&serdes 0 CU(0)>; + }; + + port1: port@1 { + phy-handle =3D <&lan966x_phy1>; + + reg =3D <1>; + phy-mode =3D "gmii"; + phys =3D <&serdes 1 CU(1)>; + }; + }; + }; + }; + }; + }; +}; diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index dccb60c1d9cc..41dec625ed7b 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -6266,6 +6266,7 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0xa76e,= dpc_log_size); DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_XILINX, 0x5020, of_pci_make_dev_node= ); DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_XILINX, 0x5021, of_pci_make_dev_node= ); DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_REDHAT, 0x0005, of_pci_make_dev_node= ); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_EFAR, 0x9660, of_pci_make_dev_node); =20 /* * Devices known to require a longer delay before first config space access --=20 2.46.2 From nobody Wed Nov 27 11:40:10 2024 Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 13E2D16A395; Thu, 10 Oct 2024 06:16:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.196 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540988; cv=none; b=hJj/yUeEF/rNZ4uszRyGzVlHQ2RN5LAHDBmA7GqFsOMBgFp7NC20U217UskMjUVIC6iuf63+adUz/3czQrKpsICkw3J4zYyx6kh/63ZUEL2Cv9el91/LGjms5Q9qOK5rlEUY0oIoIeyw3jmFSUSHzV3oMWAFpTpGRMzPtDhfoAY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540988; c=relaxed/simple; bh=4kghCVbBraLlQpqx8DAgenOlDS6CE+jDcPdxfrAEDgE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=f38oX32QJWO75UQ8pyJMyf5bkUxGEAKhDSRoG5y/IMTRyLfDBdH6wSRSp8PBwnaVZa28xdUFdG0oriw8fk6mS2DV+iVDgPXoUvN77HetCoGGk1CbprrzKGBOp3p2O1tumFcMCty+WJn1RhN+S9taPE7HuJpA2errWzqyeqHmdnk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=GeTLEo8N; arc=none smtp.client-ip=217.70.183.196 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="GeTLEo8N" Received: by mail.gandi.net (Postfix) with ESMTPA id 91DAEE0005; Thu, 10 Oct 2024 06:16:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1728540984; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=h0Ips3fu7+7BgqtN1VK4fBVu80FSths6Nq7bdNsRq4A=; b=GeTLEo8NYsu9RldZlVsgic6qli6TRymxhx2V3GufCuKD6jIq46takb4a7zsV22RAFsMWqi ExIoT/q0tRVJ2eaRg725HHgPVCrb1ODN0vXoEGcQVaVdNIV1M7VBLYlPw3P01dRhSGAhS/ T96G6ITbiOeMdfZwU630f/xntGIUJ92WxdyQ2nwVH7xAudmOyWrHLL2fcvfEqoLbSDRjr7 d+drSCCc8cbwkQz1T6d9R74ERwKG7qoSXmEtZBEiua1pS5jvu8omTfHpdimehqB23itOA9 di8JxAVqL4SfPj3QueARmaq316rWm3raRx/ghbWy4BNfnwfE6J+haArmX1sjtw== From: Herve Codina To: Geert Uytterhoeven , Andy Shevchenko , Simon Horman , Lee Jones , Arnd Bergmann , Derek Kiernan , Dragan Cvetic , Greg Kroah-Hartman , Herve Codina , Bjorn Helgaas , Philipp Zabel , Lars Povlsen , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Horatiu Vultur , Andrew Lunn , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Allan Nielsen , Steen Hegelund , Luca Ceresoli , Thomas Petazzoni Subject: [PATCH v8 2/6] MAINTAINERS: Add the Microchip LAN966x PCI driver entry Date: Thu, 10 Oct 2024 08:16:10 +0200 Message-ID: <20241010061615.787073-3-herve.codina@bootlin.com> X-Mailer: git-send-email 2.46.2 In-Reply-To: <20241010061615.787073-1-herve.codina@bootlin.com> References: <20241010061615.787073-1-herve.codina@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: herve.codina@bootlin.com Content-Type: text/plain; charset="utf-8" After contributing the driver, add myself as the maintainer for the Microchip LAN966x PCI driver. Signed-off-by: Herve Codina --- MAINTAINERS | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index c27f3190737f..79125b6f7814 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -15184,6 +15184,12 @@ S: Maintained F: Documentation/devicetree/bindings/interrupt-controller/microchip,lan966= x-oic.yaml F: drivers/irqchip/irq-lan966x-oic.c =20 +MICROCHIP LAN966X PCI DRIVER +M: Herve Codina +S: Maintained +F: drivers/misc/lan966x_pci.c +F: drivers/misc/lan966x_pci.dtso + MICROCHIP LCDFB DRIVER M: Nicolas Ferre L: linux-fbdev@vger.kernel.org --=20 2.46.2 From nobody Wed Nov 27 11:40:10 2024 Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C01A11A08D7; Thu, 10 Oct 2024 06:16:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.196 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540990; cv=none; b=eVGNl6hs35203ilC7TXnSASMHIUeLI6nMZ6RQ7w1HHD+/a3IhgZlWyq+YxXUpQdOzbh/XYI/L8h7ez9QtAAhJBL/0ItBsoeXQ7LntQgKASzwA/79R1NJ/pp+Fb1egFOMj58TJ5lGhLWJP6fHvw/C8TlJJSHBSl1gYhrbGq/fLGo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540990; c=relaxed/simple; bh=4jRMbsTtya8sck9o3TXTSwqD4/vs49q7NhKxiAlTRvk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=BXXcWE/pCCoEfXFV0kuUvPO38N5p5ajrGprGAFcjQkIfX8lOllRBE7vNZt4ttXb52G0SmDEd7H0iyZrOpEIDnqDIDtrHTg7tjwQ4jQwiUPYxKMtNSzlh8EmF1K7u7EVn/t6WR18mQOwElX2ajYvASp/pgfIPNGfZz+wwcycx/ic= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=Fk9w5ESl; arc=none smtp.client-ip=217.70.183.196 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="Fk9w5ESl" Received: by mail.gandi.net (Postfix) with ESMTPA id 54A77E0009; Thu, 10 Oct 2024 06:16:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1728540986; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=0CfXTIB149DBJ5rPusvsT7b3a7k7OVBjpLaLXBVFRjQ=; b=Fk9w5ESlv8XUeM0tN84YgA0TJj4Pgzw/ZpprLMS1CafNBAV5qzD1GNSf88qRF0pXbLFa1o Fgsmgk7c0qx8gbaVq7rKlGMyKqeVQNp8U61gbULq6V9XU9ick8r6eZ/3rE9NAGcb+cqSE0 cnszPLaLafo6TeNsAQEgbPFz2pJxhVLeX5Fqu6BqX2v23LkP/JoMbr7DUIXYW4q2MVH0Dq 7dD2dryysA715SBxTYr0Kczl7woBn1WYU7My0wmlXgV6SYLsEv67Xj/9zMQKgioryHMV5V q+hBFHXot9oKRq9+csWgnTw0nsaSrPb0xHYF9bS26G/i7Lh11qI7d7SsVn6vrQ== From: Herve Codina To: Geert Uytterhoeven , Andy Shevchenko , Simon Horman , Lee Jones , Arnd Bergmann , Derek Kiernan , Dragan Cvetic , Greg Kroah-Hartman , Herve Codina , Bjorn Helgaas , Philipp Zabel , Lars Povlsen , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Horatiu Vultur , Andrew Lunn , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Allan Nielsen , Steen Hegelund , Luca Ceresoli , Thomas Petazzoni Subject: [PATCH v8 3/6] reset: mchp: sparx5: Map cpu-syscon locally in case of LAN966x Date: Thu, 10 Oct 2024 08:16:11 +0200 Message-ID: <20241010061615.787073-4-herve.codina@bootlin.com> X-Mailer: git-send-email 2.46.2 In-Reply-To: <20241010061615.787073-1-herve.codina@bootlin.com> References: <20241010061615.787073-1-herve.codina@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: herve.codina@bootlin.com Content-Type: text/plain; charset="utf-8" In the LAN966x PCI device use case, the syscon API cannot be used as it does not support device removal [1]. A syscon device is a core "system" device and not a device available in some addon boards and so, it is not supposed to be removed. The syscon API follows this assumption but this assumption is no longer valid in the LAN966x use case. In order to avoid the use of the syscon API and so, support for removal, use a local mapping of the syscon device. Link: https://lore.kernel.org/all/20240923100741.11277439@bootlin.com/ [1] Signed-off-by: Herve Codina --- drivers/reset/reset-microchip-sparx5.c | 35 +++++++++++++++++++++++++- 1 file changed, 34 insertions(+), 1 deletion(-) diff --git a/drivers/reset/reset-microchip-sparx5.c b/drivers/reset/reset-m= icrochip-sparx5.c index 636e85c388b0..48a62d5da78d 100644 --- a/drivers/reset/reset-microchip-sparx5.c +++ b/drivers/reset/reset-microchip-sparx5.c @@ -62,6 +62,28 @@ static const struct reset_control_ops sparx5_reset_ops = =3D { .reset =3D sparx5_reset_noop, }; =20 +static const struct regmap_config mchp_lan966x_syscon_regmap_config =3D { + .reg_bits =3D 32, + .val_bits =3D 32, + .reg_stride =3D 4, +}; + +static struct regmap *mchp_lan966x_syscon_to_regmap(struct device *dev, + struct device_node *syscon_np) +{ + struct regmap_config regmap_config =3D mchp_lan966x_syscon_regmap_config; + resource_size_t size; + void __iomem *base; + + base =3D devm_of_iomap(dev, syscon_np, 0, &size); + if (IS_ERR(base)) + return ERR_CAST(base); + + regmap_config.max_register =3D size - 4; + + return devm_regmap_init_mmio(dev, base, ®map_config); +} + static int mchp_sparx5_map_syscon(struct platform_device *pdev, char *name, struct regmap **target) { @@ -72,7 +94,18 @@ static int mchp_sparx5_map_syscon(struct platform_device= *pdev, char *name, syscon_np =3D of_parse_phandle(pdev->dev.of_node, name, 0); if (!syscon_np) return -ENODEV; - regmap =3D syscon_node_to_regmap(syscon_np); + + /* + * The syscon API doesn't support syscon device removal. + * When used in LAN966x PCI device, the cpu-syscon device needs to be + * removed when the PCI device is removed. + * In case of LAN966x, map the syscon device locally to support the + * device removal. + */ + if (of_device_is_compatible(pdev->dev.of_node, "microchip,lan966x-switch-= reset")) + regmap =3D mchp_lan966x_syscon_to_regmap(&pdev->dev, syscon_np); + else + regmap =3D syscon_node_to_regmap(syscon_np); of_node_put(syscon_np); if (IS_ERR(regmap)) { err =3D PTR_ERR(regmap); --=20 2.46.2 From nobody Wed Nov 27 11:40:10 2024 Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5ED491B5337; Thu, 10 Oct 2024 06:16:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.196 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540992; cv=none; b=i60DwGdPmmwW9nTUQgohsQYEOIKpi3f4dnxa9hgwb1WbwMCxnoqx7uvhNyp2lg6bU3SLGK4B689Wql3Pw34ACE1U4jMG9c+8FsK2iaMBtI1a9vQi5PUUXyynvlcDYoA+z2OvQhZKvSjj84/au/euGIqGvBoP2x15hzwRIxeAdNM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540992; c=relaxed/simple; bh=WSxkcQynmxjawv4H3aRQeiP3sf72gh/S+dU84juIKj8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=kxTb+S1MkViWhRYSh6aio4kTXL6HYLwd+JbjE9E/BKjB5J69CHEM6/oGRVQZ8CiOd4tp//cP+HObCnQI7TAHMJocxMsnSI/AlIecmT+8CNRvXBGgpxqBEPgo3gVxMQmvXIi+wqO7F2UgDMZXQLARtk6gDBjhOyWpUFmVCRPTXlc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=TI2Hag4v; arc=none smtp.client-ip=217.70.183.196 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="TI2Hag4v" Received: by mail.gandi.net (Postfix) with ESMTPA id 1CD6BE0007; Thu, 10 Oct 2024 06:16:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1728540987; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=t9Lhp8n+UpjM11+QDMchiC8zSHm9YjBJzA7hB9o+0Tg=; b=TI2Hag4vMbcDd3lVhqX7r3D+k3q3K6bJCG6gXFFxNp/8ip62gkLrHLTBzNR9DlVHFyOZ+k I1G+qnGT8E+vgY6PrJLxa4PWoj5IQUb+T1CrKm0IlbN+NmIZGd8XE+BPPTAjCVjUKgyqR2 GMwgrfN/LhLnba/nrc/KG4ZY0hs6dYROvL3kCH9cp15fgU3bRANobZ8ftw2lIr9c9L4HvU VcLAptPblFdVJj9IeOkmsdRa+1g/fHUlyZultyv2cM6luRlohTUvFPTlMbfuGr6u6kxRgP iECc6qFr3fZpV5oWCSbW7p5AEZspy89Ohi2tgSB9dOYkHRXhC5DRELpR2hEdtg== From: Herve Codina To: Geert Uytterhoeven , Andy Shevchenko , Simon Horman , Lee Jones , Arnd Bergmann , Derek Kiernan , Dragan Cvetic , Greg Kroah-Hartman , Herve Codina , Bjorn Helgaas , Philipp Zabel , Lars Povlsen , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Horatiu Vultur , Andrew Lunn , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Allan Nielsen , Steen Hegelund , Luca Ceresoli , Thomas Petazzoni Subject: [PATCH v8 4/6] reset: mchp: sparx5: Add MCHP_LAN966X_PCI dependency Date: Thu, 10 Oct 2024 08:16:12 +0200 Message-ID: <20241010061615.787073-5-herve.codina@bootlin.com> X-Mailer: git-send-email 2.46.2 In-Reply-To: <20241010061615.787073-1-herve.codina@bootlin.com> References: <20241010061615.787073-1-herve.codina@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: herve.codina@bootlin.com Content-Type: text/plain; charset="utf-8" The sparx5 reset controller depends on the SPARX5 architecture or the LAN966x SoC. This reset controller can be used by the LAN966x PCI device and so it needs to be available when the LAN966x PCI device is enabled. Signed-off-by: Herve Codina Reviewed-by: Steen Hegelund --- drivers/reset/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig index 5484a65f66b9..86a5504950cb 100644 --- a/drivers/reset/Kconfig +++ b/drivers/reset/Kconfig @@ -147,7 +147,7 @@ config RESET_LPC18XX =20 config RESET_MCHP_SPARX5 bool "Microchip Sparx5 reset driver" - depends on ARCH_SPARX5 || SOC_LAN966 || COMPILE_TEST + depends on ARCH_SPARX5 || SOC_LAN966 || MCHP_LAN966X_PCI || COMPILE_TEST default y if SPARX5_SWITCH select MFD_SYSCON help --=20 2.46.2 From nobody Wed Nov 27 11:40:10 2024 Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EFE4F1BBBC0; Thu, 10 Oct 2024 06:16:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.196 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540993; cv=none; b=IdOkcRAtWvQvB0kfyg5rhH5mKGswOhU/GvDUlsXidhrBFMajqBGAg6uOCMJ+hj3hDyxvxr7+/CDM9tcp2QiIlo2WYfv3M6Y/mvYtwYbB+Q1f1VNNVrZufacAJlgufnkp3JumMtR8La1kKnk043S7fMM8y6m9FBpZ0/oTQM3Ev0k= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540993; c=relaxed/simple; bh=zOwm1cxscEGCjNLHPiQEn5PI7KQsbTev9ncSYRPwirA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=AI8vIhVz9a6pD/V3C9Z1me5JYcW3bC5Vq79qULBA9/LmM4oK0WOqpmZ6muudF18i+fghcxH9k8Q6acd8Bb2FoC1287RZh2lQU2tZpuoRD3lA1Ko51Yzz2ezv0RRHRrcAiA+0QkiGUe7ggVT2tiUeMVyfV0SXu6Ag6pr/Mb36uuA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=Jp1bCegg; arc=none smtp.client-ip=217.70.183.196 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="Jp1bCegg" Received: by mail.gandi.net (Postfix) with ESMTPA id D0B1FE000E; Thu, 10 Oct 2024 06:16:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1728540989; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=EOckyBBlTDDnkyo3cyvGWjnrUpORqQ7Zmtbq/7BIU3A=; b=Jp1bCegg8QC2uRHpYTP57pzql+JaomweDkcTCHe5AHiZTYpwEm+JM1FgDbtZeQV+swTmRw sqCxHBudDN/jA+XA/GXzMGankPusf7XJfjLogUo7/oIPmrLYOAIz8IakniCh5bgCY4Uthq RP+zLKlOPglrpp1ODcUm5kKnZuqTJk+nQS48AXoke+o9gBS4pFwr12AiuoI8U3d3mlYwM6 ssL7pNBfE5c5X4QL4Wbzq0D8ElmAg/PBvb93dxEg5cg+8Ovz/EnpsQdQXlAAMJ9vSX70f7 f2ZnzGH3xl4W5LrT7jyC2utVhKxDOYVlXGVzIQEFcysHFeZUZDK9zmMnR4XbRw== From: Herve Codina To: Geert Uytterhoeven , Andy Shevchenko , Simon Horman , Lee Jones , Arnd Bergmann , Derek Kiernan , Dragan Cvetic , Greg Kroah-Hartman , Herve Codina , Bjorn Helgaas , Philipp Zabel , Lars Povlsen , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Horatiu Vultur , Andrew Lunn , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Allan Nielsen , Steen Hegelund , Luca Ceresoli , Thomas Petazzoni , =?UTF-8?q?Cl=C3=A9ment=20L=C3=A9ger?= Subject: [PATCH v8 5/6] reset: mchp: sparx5: Allow building as a module Date: Thu, 10 Oct 2024 08:16:13 +0200 Message-ID: <20241010061615.787073-6-herve.codina@bootlin.com> X-Mailer: git-send-email 2.46.2 In-Reply-To: <20241010061615.787073-1-herve.codina@bootlin.com> References: <20241010061615.787073-1-herve.codina@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-GND-Sasl: herve.codina@bootlin.com From: Cl=C3=A9ment L=C3=A9ger This reset controller can be used by the LAN966x PCI device. The LAN966x PCI device driver can be built as a module and this reset controller driver has no reason to be a builtin driver in that case. Signed-off-by: Cl=C3=A9ment L=C3=A9ger Signed-off-by: Herve Codina Reviewed-by: Steen Hegelund --- drivers/reset/Kconfig | 2 +- drivers/reset/reset-microchip-sparx5.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig index 86a5504950cb..93cddbe8609b 100644 --- a/drivers/reset/Kconfig +++ b/drivers/reset/Kconfig @@ -146,7 +146,7 @@ config RESET_LPC18XX This enables the reset controller driver for NXP LPC18xx/43xx SoCs. =20 config RESET_MCHP_SPARX5 - bool "Microchip Sparx5 reset driver" + tristate "Microchip Sparx5 reset driver" depends on ARCH_SPARX5 || SOC_LAN966 || MCHP_LAN966X_PCI || COMPILE_TEST default y if SPARX5_SWITCH select MFD_SYSCON diff --git a/drivers/reset/reset-microchip-sparx5.c b/drivers/reset/reset-m= icrochip-sparx5.c index 48a62d5da78d..c4cc0edbb250 100644 --- a/drivers/reset/reset-microchip-sparx5.c +++ b/drivers/reset/reset-microchip-sparx5.c @@ -191,6 +191,7 @@ static const struct of_device_id mchp_sparx5_reset_of_m= atch[] =3D { }, { } }; +MODULE_DEVICE_TABLE(of, mchp_sparx5_reset_of_match); =20 static struct platform_driver mchp_sparx5_reset_driver =3D { .probe =3D mchp_sparx5_reset_probe, @@ -213,3 +214,4 @@ postcore_initcall(mchp_sparx5_reset_init); =20 MODULE_DESCRIPTION("Microchip Sparx5 switch reset driver"); MODULE_AUTHOR("Steen Hegelund "); +MODULE_LICENSE("GPL"); --=20 2.46.2 From nobody Wed Nov 27 11:40:10 2024 Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E6F0E1BBBF7; Thu, 10 Oct 2024 06:16:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.196 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540996; cv=none; b=qu9FRYWZqzNPKLMiI//sltUxyD9fmtx5P5sUcP8vKbE7Qpdx8U+SvC33oCSc93KNVlvyOtSs/XbPsbLiMeUi1nZvd8CXUcIZtlR4D2nK5swLAEhxtbiKpKSoSDdld30w4Tb0+YoC4EqIrM19THrZIp8ikYFeotDyxI0RirT6X3c= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728540996; c=relaxed/simple; bh=pay+04IBkb6N6YqImnzm66VLMa5BmJvHR/MOacq+wLM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=mTnjBGXsN5fmicNEr1vp21SII/Lzev+dBrV09TCPi1tO0mBYsnIQMGfmmAoivjU2Fc/DZj/BI89+0rkCYjh5O4CYKxx/IciLAZlJdnOw6l7CtYajIpijQUudUdnXmfEHveNA/Vtp++KIAo+qTahlvTUeV2JyA760qFEHzV/W0Cc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=VDyxzHMW; arc=none smtp.client-ip=217.70.183.196 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="VDyxzHMW" Received: by mail.gandi.net (Postfix) with ESMTPA id AE7FDE000C; Thu, 10 Oct 2024 06:16:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1728540991; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=edIBHVUlBen2+dM0A9C3oLe4ngVIrfCfYAeOnVLi3Ts=; b=VDyxzHMWMrW/+0OrpAje2GziXWPYSI0ePRTQtXwG/wusoylMqbHJEjL/MQtl/YfmOVW4bs PAhNWZZ8cgaikRk0g5P+o3nv07fxP6xFuX0xGznoRYb7K8Ip0lA3mA3vtS3G1puY7i4Duy ihUpppqet9O5IzRzUEoP5SSx9svA30KMuRpXMi8XZOZdUKZ2SJWDvGSU+9ETRgkIcoqyu6 7vsYCtMFAxm9rkg15yPrhDQrVnaTTxPErMjSl0xkUuBmr/cjq4+9nVj8XjZ9vEeOQqi5TX 1/MSXmR4toVAZiGyfRyzQqBxvObHBeNPoQXtbjq86HyyV2UtXOjXSZH5ClbzLg== From: Herve Codina To: Geert Uytterhoeven , Andy Shevchenko , Simon Horman , Lee Jones , Arnd Bergmann , Derek Kiernan , Dragan Cvetic , Greg Kroah-Hartman , Herve Codina , Bjorn Helgaas , Philipp Zabel , Lars Povlsen , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Horatiu Vultur , Andrew Lunn , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Allan Nielsen , Steen Hegelund , Luca Ceresoli , Thomas Petazzoni , =?UTF-8?q?Cl=C3=A9ment=20L=C3=A9ger?= Subject: [PATCH v8 6/6] reset: mchp: sparx5: set the dev member of the reset controller Date: Thu, 10 Oct 2024 08:16:14 +0200 Message-ID: <20241010061615.787073-7-herve.codina@bootlin.com> X-Mailer: git-send-email 2.46.2 In-Reply-To: <20241010061615.787073-1-herve.codina@bootlin.com> References: <20241010061615.787073-1-herve.codina@bootlin.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-GND-Sasl: herve.codina@bootlin.com From: Cl=C3=A9ment L=C3=A9ger In order to guarantee the device will not be deleted by the reset controller consumer, set the dev member of the reset controller. Signed-off-by: Cl=C3=A9ment L=C3=A9ger Signed-off-by: Herve Codina Reviewed-by: Steen Hegelund --- drivers/reset/reset-microchip-sparx5.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/reset/reset-microchip-sparx5.c b/drivers/reset/reset-m= icrochip-sparx5.c index c4cc0edbb250..aa5464be7053 100644 --- a/drivers/reset/reset-microchip-sparx5.c +++ b/drivers/reset/reset-microchip-sparx5.c @@ -154,6 +154,7 @@ static int mchp_sparx5_reset_probe(struct platform_devi= ce *pdev) return err; =20 ctx->rcdev.owner =3D THIS_MODULE; + ctx->rcdev.dev =3D &pdev->dev; ctx->rcdev.nr_resets =3D 1; ctx->rcdev.ops =3D &sparx5_reset_ops; ctx->rcdev.of_node =3D dn; --=20 2.46.2