* [PATCH] ata: ahci: Cleanup ahci_reset_controller()
@ 2023-06-13 6:34 Damien Le Moal
2023-06-13 9:02 ` Niklas Cassel
0 siblings, 1 reply; 2+ messages in thread
From: Damien Le Moal @ 2023-06-13 6:34 UTC (permalink / raw)
To: linux-ide
Fix multi-line comment style in ahci_reset_controller() and change the
code to return early if ahci_skip_host_reset is true, reducing
indentation by one level for the bulk of the function code.
No functional changes.
Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
---
drivers/ata/libahci.c | 61 +++++++++++++++++++++----------------------
1 file changed, 30 insertions(+), 31 deletions(-)
diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c
index 9c2cb6cbea76..bd38ff72e543 100644
--- a/drivers/ata/libahci.c
+++ b/drivers/ata/libahci.c
@@ -975,44 +975,43 @@ int ahci_reset_controller(struct ata_host *host)
void __iomem *mmio = hpriv->mmio;
u32 tmp;
- /* we must be in AHCI mode, before using anything
- * AHCI-specific, such as HOST_RESET.
+ /*
+ * We must be in AHCI mode, before using anything AHCI-specific, such
+ * as HOST_RESET.
*/
ahci_enable_ahci(mmio);
- /* global controller reset */
- if (!ahci_skip_host_reset) {
- tmp = readl(mmio + HOST_CTL);
- if ((tmp & HOST_RESET) == 0) {
- writel(tmp | HOST_RESET, mmio + HOST_CTL);
- readl(mmio + HOST_CTL); /* flush */
- }
+ /* Global controller reset */
+ if (ahci_skip_host_reset) {
+ dev_info(host->dev, "Skipping global host reset\n");
+ return 0;
+ }
- /*
- * to perform host reset, OS should set HOST_RESET
- * and poll until this bit is read to be "0".
- * reset must complete within 1 second, or
- * the hardware should be considered fried.
- */
- tmp = ata_wait_register(NULL, mmio + HOST_CTL, HOST_RESET,
- HOST_RESET, 10, 1000);
+ tmp = readl(mmio + HOST_CTL);
+ if ((tmp & HOST_RESET) == 0) {
+ writel(tmp | HOST_RESET, mmio + HOST_CTL);
+ readl(mmio + HOST_CTL); /* flush */
+ }
- if (tmp & HOST_RESET) {
- dev_err(host->dev, "controller reset failed (0x%x)\n",
- tmp);
- return -EIO;
- }
+ /*
+ * To perform host reset, OS should set HOST_RESET and poll until this
+ * bit is read to be "0". Reset must complete within 1 second, or the
+ * hardware should be considered fried.
+ */
+ tmp = ata_wait_register(NULL, mmio + HOST_CTL, HOST_RESET,
+ HOST_RESET, 10, 1000);
+ if (tmp & HOST_RESET) {
+ dev_err(host->dev, "Controller reset failed (0x%x)\n",
+ tmp);
+ return -EIO;
+ }
- /* turn on AHCI mode */
- ahci_enable_ahci(mmio);
+ /* Turn on AHCI mode */
+ ahci_enable_ahci(mmio);
- /* Some registers might be cleared on reset. Restore
- * initial values.
- */
- if (!(hpriv->flags & AHCI_HFLAG_NO_WRITE_TO_RO))
- ahci_restore_initial_config(host);
- } else
- dev_info(host->dev, "skipping global host reset\n");
+ /* Some registers might be cleared on reset. Restore initial values. */
+ if (!(hpriv->flags & AHCI_HFLAG_NO_WRITE_TO_RO))
+ ahci_restore_initial_config(host);
return 0;
}
--
2.40.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ata: ahci: Cleanup ahci_reset_controller()
2023-06-13 6:34 [PATCH] ata: ahci: Cleanup ahci_reset_controller() Damien Le Moal
@ 2023-06-13 9:02 ` Niklas Cassel
0 siblings, 0 replies; 2+ messages in thread
From: Niklas Cassel @ 2023-06-13 9:02 UTC (permalink / raw)
To: Damien Le Moal; +Cc: linux-ide@vger.kernel.org
On Tue, Jun 13, 2023 at 03:34:36PM +0900, Damien Le Moal wrote:
> Fix multi-line comment style in ahci_reset_controller() and change the
> code to return early if ahci_skip_host_reset is true, reducing
> indentation by one level for the bulk of the function code.
>
> No functional changes.
>
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
> ---
> drivers/ata/libahci.c | 61 +++++++++++++++++++++----------------------
> 1 file changed, 30 insertions(+), 31 deletions(-)
>
> diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c
> index 9c2cb6cbea76..bd38ff72e543 100644
> --- a/drivers/ata/libahci.c
> +++ b/drivers/ata/libahci.c
> @@ -975,44 +975,43 @@ int ahci_reset_controller(struct ata_host *host)
> void __iomem *mmio = hpriv->mmio;
> u32 tmp;
>
> - /* we must be in AHCI mode, before using anything
> - * AHCI-specific, such as HOST_RESET.
> + /*
> + * We must be in AHCI mode, before using anything AHCI-specific, such
> + * as HOST_RESET.
> */
> ahci_enable_ahci(mmio);
>
> - /* global controller reset */
> - if (!ahci_skip_host_reset) {
> - tmp = readl(mmio + HOST_CTL);
> - if ((tmp & HOST_RESET) == 0) {
> - writel(tmp | HOST_RESET, mmio + HOST_CTL);
> - readl(mmio + HOST_CTL); /* flush */
> - }
> + /* Global controller reset */
> + if (ahci_skip_host_reset) {
> + dev_info(host->dev, "Skipping global host reset\n");
> + return 0;
> + }
>
> - /*
> - * to perform host reset, OS should set HOST_RESET
> - * and poll until this bit is read to be "0".
> - * reset must complete within 1 second, or
> - * the hardware should be considered fried.
> - */
> - tmp = ata_wait_register(NULL, mmio + HOST_CTL, HOST_RESET,
> - HOST_RESET, 10, 1000);
> + tmp = readl(mmio + HOST_CTL);
> + if ((tmp & HOST_RESET) == 0) {
Nit: perhaps use if (!(tmp & HOST_RESET)) instead, such that we use the
same pattern as:
if (!(hpriv->flags & AHCI_HFLAG_NO_WRITE_TO_RO))
below.
> + writel(tmp | HOST_RESET, mmio + HOST_CTL);
> + readl(mmio + HOST_CTL); /* flush */
> + }
>
> - if (tmp & HOST_RESET) {
> - dev_err(host->dev, "controller reset failed (0x%x)\n",
> - tmp);
> - return -EIO;
> - }
> + /*
> + * To perform host reset, OS should set HOST_RESET and poll until this
> + * bit is read to be "0". Reset must complete within 1 second, or the
> + * hardware should be considered fried.
> + */
> + tmp = ata_wait_register(NULL, mmio + HOST_CTL, HOST_RESET,
> + HOST_RESET, 10, 1000);
> + if (tmp & HOST_RESET) {
> + dev_err(host->dev, "Controller reset failed (0x%x)\n",
> + tmp);
> + return -EIO;
> + }
>
> - /* turn on AHCI mode */
> - ahci_enable_ahci(mmio);
> + /* Turn on AHCI mode */
> + ahci_enable_ahci(mmio);
>
> - /* Some registers might be cleared on reset. Restore
> - * initial values.
> - */
> - if (!(hpriv->flags & AHCI_HFLAG_NO_WRITE_TO_RO))
> - ahci_restore_initial_config(host);
> - } else
> - dev_info(host->dev, "skipping global host reset\n");
> + /* Some registers might be cleared on reset. Restore initial values. */
> + if (!(hpriv->flags & AHCI_HFLAG_NO_WRITE_TO_RO))
> + ahci_restore_initial_config(host);
>
> return 0;
> }
> --
> 2.40.1
>
Reviewed-by: Niklas Cassel <niklas.cassel@wdc.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-06-13 9:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-13 6:34 [PATCH] ata: ahci: Cleanup ahci_reset_controller() Damien Le Moal
2023-06-13 9:02 ` Niklas Cassel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox