From: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
To: linux-aspeed@lists.ozlabs.org
Subject: [PATCH v3 3/4] reset: aspeed: register AST2700 reset auxiliary bus device
Date: Mon, 16 Sep 2024 20:41:01 +0200 [thread overview]
Message-ID: <8412982c-17c0-4596-a9eb-cd28ad747048@wanadoo.fr> (raw)
In-Reply-To: <20240916091039.3584505-4-ryan_chen@aspeedtech.com>
Le 16/09/2024 ? 11:10, Ryan Chen a ?crit?:
> The AST2700 reset driver is registered as an auxiliary device
> due to reset and clock controller share the same register region.
>
> Signed-off-by: Ryan Chen <ryan_chen@aspeedtech.com>
Hi,
...
> +static const struct ast2700_reset_signal ast2700_reset0_signals[] = {
> + [SCU0_RESET_SDRAM] = { true, SCU0_RESET_CTRL1, BIT(0) },
> + [SCU0_RESET_DDRPHY] = { true, SCU0_RESET_CTRL1, BIT(1) },
> + [SCU0_RESET_RSA] = { true, SCU0_RESET_CTRL1, BIT(2) },
> + [SCU0_RESET_SHA3] = { true, SCU0_RESET_CTRL1, BIT(3) },
> + [SCU0_RESET_HACE] = { true, SCU0_RESET_CTRL1, BIT(4) },
> + [SCU0_RESET_SOC] = { true, SCU0_RESET_CTRL1, BIT(5) },
> + [SCU0_RESET_VIDEO] = { true, SCU0_RESET_CTRL1, BIT(6) },
> + [SCU0_RESET_2D] = { true, SCU0_RESET_CTRL1, BIT(7) },
> + [SCU0_RESET_PCIS] = { true, SCU0_RESET_CTRL1, BIT(8) },
> + [SCU0_RESET_RVAS0] = { true, SCU0_RESET_CTRL1, BIT(9) },
> + [SCU0_RESET_RVAS1] = { true, SCU0_RESET_CTRL1, BIT(10) },
> + [SCU0_RESET_SM3] = { true, SCU0_RESET_CTRL1, BIT(11) },
> + [SCU0_RESET_SM4] = { true, SCU0_RESET_CTRL1, BIT(12) },
> + [SCU0_RESET_CRT0] = { true, SCU0_RESET_CTRL1, BIT(13) },
> + [SCU0_RESET_ECC] = { true, SCU0_RESET_CTRL1, BIT(14) },
> + [SCU0_RESET_DP_PCI] = { true, SCU0_RESET_CTRL1, BIT(15) },
> + [SCU0_RESET_UFS] = { true, SCU0_RESET_CTRL1, BIT(16) },
> + [SCU0_RESET_EMMC] = { true, SCU0_RESET_CTRL1, BIT(17) },
> + [SCU0_RESET_PCIE1RST] = { true, SCU0_RESET_CTRL1, BIT(18) },
> + [SCU0_RESET_PCIE1RSTOE] = { true, SCU0_RESET_CTRL1, BIT(19) },
> + [SCU0_RESET_PCIE0RST] = { true, SCU0_RESET_CTRL1, BIT(20) },
> + [SCU0_RESET_PCIE0RSTOE] = { true, SCU0_RESET_CTRL1, BIT(21) },
> + [SCU0_RESET_JTAG] = { true, SCU0_RESET_CTRL1, BIT(22) },
> + [SCU0_RESET_MCTP0] = { true, SCU0_RESET_CTRL1, BIT(23) },
> + [SCU0_RESET_MCTP1] = { true, SCU0_RESET_CTRL1, BIT(24) },
> + [SCU0_RESET_XDMA0] = { true, SCU0_RESET_CTRL1, BIT(25) },
> + [SCU0_RESET_XDMA1] = { true, SCU0_RESET_CTRL1, BIT(26) },
> + [SCU0_RESET_H2X1] = { true, SCU0_RESET_CTRL1, BIT(27) },
> + [SCU0_RESET_DP] = { true, SCU0_RESET_CTRL1, BIT(28) },
> + [SCU0_RESET_DP_MCU] = { true, SCU0_RESET_CTRL1, BIT(29) },
> + [SCU0_RESET_SSP] = { true, SCU0_RESET_CTRL1, BIT(30) },
> + [SCU0_RESET_H2X0] = { true, SCU0_RESET_CTRL1, BIT(31) },
> + [SCU0_RESET_PORTA_VHUB] = { true, SCU0_RESET_CTRL2, BIT(0) },
> + [SCU0_RESET_PORTA_PHY3] = { true, SCU0_RESET_CTRL2, BIT(1) },
> + [SCU0_RESET_PORTA_XHCI] = { true, SCU0_RESET_CTRL2, BIT(2) },
> + [SCU0_RESET_PORTB_VHUB] = { true, SCU0_RESET_CTRL2, BIT(3) },
> + [SCU0_RESET_PORTB_PHY3] = { true, SCU0_RESET_CTRL2, BIT(4) },
> + [SCU0_RESET_PORTB_XHCI] = { true, SCU0_RESET_CTRL2, BIT(5) },
> + [SCU0_RESET_PORTA_VHUB_EHCI] = { true, SCU0_RESET_CTRL2, BIT(6) },
> + [SCU0_RESET_PORTB_VHUB_EHCI] = { true, SCU0_RESET_CTRL2, BIT(7) },
> + [SCU0_RESET_UHCI] = { true, SCU0_RESET_CTRL2, BIT(8) },
> + [SCU0_RESET_TSP] = { true, SCU0_RESET_CTRL2, BIT(9) },
> + [SCU0_RESET_E2M0] = { true, SCU0_RESET_CTRL2, BIT(10) },
> + [SCU0_RESET_E2M1] = { true, SCU0_RESET_CTRL2, BIT(11) },
> + [SCU0_RESET_VLINK] = { true, SCU0_RESET_CTRL2, BIT(12) },
> +};
The sapces and tabs in both tables look not consistent.
> +
> +static const struct ast2700_reset_signal ast2700_reset1_signals[] = {
> + [SCU1_RESET_LPC0] = { true, SCU1_RESET_CTRL1, BIT(0) },
> + [SCU1_RESET_LPC1] = { true, SCU1_RESET_CTRL1, BIT(1) },
> + [SCU1_RESET_MII] = { true, SCU1_RESET_CTRL1, BIT(2) },
> + [SCU1_RESET_PECI] = { true, SCU1_RESET_CTRL1, BIT(3) },
> + [SCU1_RESET_PWM] = { true, SCU1_RESET_CTRL1, BIT(4) },
> + [SCU1_RESET_MAC0] = { true, SCU1_RESET_CTRL1, BIT(5) },
> + [SCU1_RESET_MAC1] = { true, SCU1_RESET_CTRL1, BIT(6) },
> + [SCU1_RESET_MAC2] = { true, SCU1_RESET_CTRL1, BIT(7) },
> + [SCU1_RESET_ADC] = { true, SCU1_RESET_CTRL1, BIT(8) },
> + [SCU1_RESET_SD] = { true, SCU1_RESET_CTRL1, BIT(9) },
> + [SCU1_RESET_ESPI0] = { true, SCU1_RESET_CTRL1, BIT(10) },
> + [SCU1_RESET_ESPI1] = { true, SCU1_RESET_CTRL1, BIT(11) },
> + [SCU1_RESET_JTAG1] = { true, SCU1_RESET_CTRL1, BIT(12) },
> + [SCU1_RESET_SPI0] = { true, SCU1_RESET_CTRL1, BIT(13) },
> + [SCU1_RESET_SPI1] = { true, SCU1_RESET_CTRL1, BIT(14) },
> + [SCU1_RESET_SPI2] = { true, SCU1_RESET_CTRL1, BIT(15) },
> + [SCU1_RESET_I3C0] = { true, SCU1_RESET_CTRL1, BIT(16) },
> + [SCU1_RESET_I3C1] = { true, SCU1_RESET_CTRL1, BIT(17) },
> + [SCU1_RESET_I3C2] = { true, SCU1_RESET_CTRL1, BIT(18) },
> + [SCU1_RESET_I3C3] = { true, SCU1_RESET_CTRL1, BIT(19) },
> + [SCU1_RESET_I3C4] = { true, SCU1_RESET_CTRL1, BIT(20) },
> + [SCU1_RESET_I3C5] = { true, SCU1_RESET_CTRL1, BIT(21) },
> + [SCU1_RESET_I3C6] = { true, SCU1_RESET_CTRL1, BIT(22) },
> + [SCU1_RESET_I3C7] = { true, SCU1_RESET_CTRL1, BIT(23) },
> + [SCU1_RESET_I3C8] = { true, SCU1_RESET_CTRL1, BIT(24) },
> + [SCU1_RESET_I3C9] = { true, SCU1_RESET_CTRL1, BIT(25) },
> + [SCU1_RESET_I3C10] = { true, SCU1_RESET_CTRL1, BIT(26) },
> + [SCU1_RESET_I3C11] = { true, SCU1_RESET_CTRL1, BIT(27) },
> + [SCU1_RESET_I3C12] = { true, SCU1_RESET_CTRL1, BIT(28) },
> + [SCU1_RESET_I3C13] = { true, SCU1_RESET_CTRL1, BIT(29) },
> + [SCU1_RESET_I3C14] = { true, SCU1_RESET_CTRL1, BIT(30) },
> + [SCU1_RESET_I3C15] = { true, SCU1_RESET_CTRL1, BIT(31) },
> + [SCU1_RESET_MCU0] = { true, SCU1_RESET_CTRL2, BIT(0) },
> + [SCU1_RESET_MCU1] = { true, SCU1_RESET_CTRL2, BIT(1) },
> + [SCU1_RESET_H2A_SPI1] = { true, SCU1_RESET_CTRL2, BIT(2) },
> + [SCU1_RESET_H2A_SPI2] = { true, SCU1_RESET_CTRL2, BIT(3) },
> + [SCU1_RESET_UART0] = { true, SCU1_RESET_CTRL2, BIT(4) },
> + [SCU1_RESET_UART1] = { true, SCU1_RESET_CTRL2, BIT(5) },
> + [SCU1_RESET_UART2] = { true, SCU1_RESET_CTRL2, BIT(6) },
> + [SCU1_RESET_UART3] = { true, SCU1_RESET_CTRL2, BIT(7) },
> + [SCU1_RESET_I2C_FILTER] = { true, SCU1_RESET_CTRL2, BIT(8) },
> + [SCU1_RESET_CALIPTRA] = { true, SCU1_RESET_CTRL2, BIT(9) },
> + [SCU1_RESET_XDMA] = { true, SCU1_RESET_CTRL2, BIT(10) },
> + [SCU1_RESET_FSI] = { true, SCU1_RESET_CTRL2, BIT(12) },
> + [SCU1_RESET_CAN] = { true, SCU1_RESET_CTRL2, BIT(13) },
> + [SCU1_RESET_MCTP] = { true, SCU1_RESET_CTRL2, BIT(14) },
> + [SCU1_RESET_I2C] = { true, SCU1_RESET_CTRL2, BIT(15) },
> + [SCU1_RESET_UART6] = { true, SCU1_RESET_CTRL2, BIT(16) },
> + [SCU1_RESET_UART7] = { true, SCU1_RESET_CTRL2, BIT(17) },
> + [SCU1_RESET_UART8] = { true, SCU1_RESET_CTRL2, BIT(18) },
> + [SCU1_RESET_UART9] = { true, SCU1_RESET_CTRL2, BIT(19) },
> + [SCU1_RESET_LTPI0] = { true, SCU1_RESET_CTRL2, BIT(20) },
> + [SCU1_RESET_VGAL] = { true, SCU1_RESET_CTRL2, BIT(21) },
> + [SCU1_RESET_LTPI1] = { true, SCU1_RESET_CTRL2, BIT(22) },
> + [SCU1_RESET_ACE] = { true, SCU1_RESET_CTRL2, BIT(23) },
> + [SCU1_RESET_E2M] = { true, SCU1_RESET_CTRL2, BIT(24) },
> + [SCU1_RESET_UHCI] = { true, SCU1_RESET_CTRL2, BIT(25) },
> + [SCU1_RESET_PORTC_USB2UART] = { true, SCU1_RESET_CTRL2, BIT(26) },
> + [SCU1_RESET_PORTC_VHUB_EHCI] = { true, SCU1_RESET_CTRL2, BIT(27) },
> + [SCU1_RESET_PORTD_USB2UART] = { true, SCU1_RESET_CTRL2, BIT(28) },
> + [SCU1_RESET_PORTD_VHUB_EHCI] = { true, SCU1_RESET_CTRL2, BIT(29) },
> + [SCU1_RESET_H2X] = { true, SCU1_RESET_CTRL2, BIT(30) },
> + [SCU1_RESET_I3CDMA] = { true, SCU1_RESET_CTRL2, BIT(31) },
> + [SCU1_RESET_PCIE2RST] = { false, SCU1_PCIE3_CTRL, BIT(0) },
> +};
...
> +static int aspeed_reset_probe(struct auxiliary_device *adev,
> + const struct auxiliary_device_id *id)
> +{
> + struct aspeed_reset *reset;
> + struct device *dev = &adev->dev;
> +
> + reset = devm_kzalloc(dev, sizeof(*reset), GFP_KERNEL);
> + if (!reset)
> + return -ENOMEM;
> +
> + spin_lock_init(&reset->lock);
> +
> + reset->info = (struct aspeed_reset_info *)(id->driver_data);
> + reset->rcdev.owner = THIS_MODULE;
> + reset->rcdev.nr_resets = reset->info->nr_resets;
> + reset->rcdev.ops = &aspeed_reset_ops;
> + reset->rcdev.of_node = dev->parent->of_node;
> + reset->rcdev.dev = dev;
> + reset->rcdev.of_reset_n_cells = 1;
> + reset->base = (void __iomem *)adev->dev.platform_data;
The spaces and tabs look broken for 'info' and 'rcdev.dev'.
> +
> + if (!reset->base)
> + return -ENOMEM;
> +
> + dev_set_drvdata(dev, reset);
Is it needed?
(there is no dev_get_drvdata())
> +
> + return devm_reset_controller_register(dev, &reset->rcdev);
> +}
...
CJ
next prev parent reply other threads:[~2024-09-16 18:41 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-16 9:10 [PATCH v3 0/4] Add support for AST2700 clk driver Ryan Chen
2024-09-16 9:10 ` [PATCH v3 1/4] dt-bindings: mfd: aspeed: support for AST2700 Ryan Chen
2024-09-16 10:08 ` Krzysztof Kozlowski
2024-09-19 6:05 ` Ryan Chen
2024-09-19 6:57 ` Krzysztof Kozlowski
2024-09-19 7:13 ` Ryan Chen
2024-09-19 10:32 ` Krzysztof Kozlowski
2024-09-20 7:33 ` Ryan Chen
2024-09-16 9:10 ` [PATCH v3 2/4] dt-bindings: Add AST2700 bindings Ryan Chen
2024-09-16 10:06 ` Krzysztof Kozlowski
2024-09-19 6:01 ` Ryan Chen
2024-09-19 6:58 ` Krzysztof Kozlowski
2024-09-19 7:15 ` Ryan Chen
2024-09-19 10:30 ` Krzysztof Kozlowski
2024-09-20 7:31 ` Ryan Chen
2024-09-16 9:10 ` [PATCH v3 3/4] reset: aspeed: register AST2700 reset auxiliary bus device Ryan Chen
2024-09-16 18:41 ` Christophe JAILLET [this message]
2024-09-23 2:50 ` Ryan Chen
2024-09-16 9:10 ` [PATCH v3 4/4] clk: aspeed: add AST2700 clock driver Ryan Chen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8412982c-17c0-4596-a9eb-cd28ad747048@wanadoo.fr \
--to=christophe.jaillet@wanadoo.fr \
--cc=linux-aspeed@lists.ozlabs.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox