Re: [PATCH 3/8] sata: Rearrange Kconfig for SATA

On 1/31/22 15:49, Simon Glass wrote:

Move the SATA options inside an 'if SATA' part, so they are grouped.

Fix the 'Complient' typo while we are here.

Signed-off-by: Simon Glass sjg@chromium.org

Reviewed-by: Stefan Roese sr@denx.de

Thanks, Stefan

---

drivers/ata/Kconfig | 71 +++++++++++++++++++++++---------------------- 1 file changed, 37 insertions(+), 34 deletions(-)

diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig index 1def9305dd5..6e8363c04d9 100644 --- a/drivers/ata/Kconfig +++ b/drivers/ata/Kconfig @@ -48,16 +48,6 @@ config SPL_AHCI_PCI depends on SPL_PCI depends on SPL_SATA_SUPPORT && DM_SCSI

-config SATA_CEVA

• bool "Ceva Sata controller"
• depends on AHCI
• depends on DM_SCSI
• help

•  This option enables Ceva Sata controller hard IP available on Xilinx
  

•  ZynqMP. Support up to 2 external devices. Complient with SATA 3.1 and
  

•  AHCI 1.3 specifications with hot-plug detect feature.
  

• config DWC_AHCI bool "Enable Synopsys DWC AHCI driver support" select SCSI_AHCI

@@ -83,30 +73,6 @@ config DWC_AHSATA_AHCI help Enable this option unless you need your private ahci implementation

-config FSL_SATA

• bool "Enable Freescale SATA controller driver support"
• select AHCI
• select LIBATA
• help

•  Enable this driver to support the SATA controller found in
  

•  some Freescale PowerPC SoCs.
  

-config SATA_MV

• bool "Enable Marvell SATA controller driver support"
• select AHCI
• select LIBATA
• help

•  Enable this driver to support the SATA controller found in
  

•  some Marvell SoCs.
  

-config SATA_SIL

• bool "Enable Silicon Image SIL3131 / SIL3132 / SIL3124 SATA driver support"
• select AHCI
• select LIBATA
• help

•  Enable this driver to support the SIL3131, SIL3132 and SIL3124
  

•  SATA controllers.
  

• config SUNXI_AHCI bool "Enable Allwinner SATA driver support" depends on AHCI

@@ -134,4 +100,41 @@ config MTK_AHCI Enable this driver to support Sata devices through Mediatek AHCI controller (e.g. MT7622).

+if SATA

+config SATA_CEVA

• bool "Ceva Sata controller"
• depends on AHCI
• depends on DM_SCSI
• help

•  This option enables Ceva Sata controller hard IP available on Xilinx
  

•  ZynqMP. Support up to 2 external devices. Compliant with SATA 3.1 and
  

•  AHCI 1.3 specifications with hot-plug detect feature.
  

+config FSL_SATA

• bool "Enable Freescale SATA controller driver support"
• select AHCI
• select LIBATA
• help

•  Enable this driver to support the SATA controller found in
  

•  some Freescale PowerPC SoCs.
  

+config SATA_MV

• bool "Enable Marvell SATA controller driver support"
• select AHCI
• select LIBATA
• help

•  Enable this driver to support the SATA controller found in
  

•  some Marvell SoCs.
  

+config SATA_SIL

• bool "Enable Silicon Image SIL3131 / SIL3132 / SIL3124 SATA driver support"
• select AHCI
• select LIBATA
• help

•  Enable this driver to support the SIL3131, SIL3132 and SIL3124
  

•  SATA controllers.
  

+endif # SATA

• endmenu

Viele Grüße, Stefan Roese