diff mbox series

[1/2] mmc: sdhci_am654: Fix TI SoC dependencies

Message ID 20231220135950.433588-1-pbrobinson@gmail.com (mailing list archive)
State New, archived
Headers show
Series [1/2] mmc: sdhci_am654: Fix TI SoC dependencies | expand

Commit Message

Peter Robinson Dec. 20, 2023, 1:59 p.m. UTC
The sdhci_am654 is specific to recent TI SoCs, update the
dependencies for those SoCs and compile testing. While we're
at it update the text to reflect the wider range of
supported TI SoCS the driver now supports.

Fixes: 41fd4caeb00b ("mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver")
Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
---
 drivers/mmc/host/Kconfig | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Adrian Hunter Jan. 2, 2024, 10:06 a.m. UTC | #1
On 20/12/23 15:59, Peter Robinson wrote:
> The sdhci_am654 is specific to recent TI SoCs, update the
> dependencies for those SoCs and compile testing. While we're
> at it update the text to reflect the wider range of
> supported TI SoCS the driver now supports.
> 
> Fixes: 41fd4caeb00b ("mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver")

Is this really a fix?  Seems like a minor improvement.
Same question for patch 2/2.

> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>

It would be good to get an Ack from a TI person.
Same for patch 2/2.

> ---
>  drivers/mmc/host/Kconfig | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> index 58bd5fe4cd25..24ce5576b61a 100644
> --- a/drivers/mmc/host/Kconfig
> +++ b/drivers/mmc/host/Kconfig
> @@ -1041,14 +1041,15 @@ config MMC_SDHCI_OMAP
>  
>  config MMC_SDHCI_AM654
>  	tristate "Support for the SDHCI Controller in TI's AM654 SOCs"
> +	depends on ARCH_K3 || COMPILE_TEST
>  	depends on MMC_SDHCI_PLTFM && OF
>  	select MMC_SDHCI_IO_ACCESSORS
>  	select MMC_CQHCI
>  	select REGMAP_MMIO
>  	help
>  	  This selects the Secure Digital Host Controller Interface (SDHCI)
> -	  support present in TI's AM654 SOCs. The controller supports
> -	  SD/MMC/SDIO devices.
> +	  support present in TI's AM65x/AM64x/AM62x/J721E SOCs. The controller
> +	  supports SD/MMC/SDIO devices.
>  
>  	  If you have a controller with this interface, say Y or M here.
>
Peter Robinson Jan. 2, 2024, 10:10 a.m. UTC | #2
On Tue, Jan 2, 2024 at 10:06 AM Adrian Hunter <adrian.hunter@intel.com> wrote:
>
> On 20/12/23 15:59, Peter Robinson wrote:
> > The sdhci_am654 is specific to recent TI SoCs, update the
> > dependencies for those SoCs and compile testing. While we're
> > at it update the text to reflect the wider range of
> > supported TI SoCS the driver now supports.
> >
> > Fixes: 41fd4caeb00b ("mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver")
>
> Is this really a fix?  Seems like a minor improvement.
> Same question for patch 2/2.

It is for distro kernels at least :)

> > Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
>
> It would be good to get an Ack from a TI person.
> Same for patch 2/2.
>
> > ---
> >  drivers/mmc/host/Kconfig | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> > index 58bd5fe4cd25..24ce5576b61a 100644
> > --- a/drivers/mmc/host/Kconfig
> > +++ b/drivers/mmc/host/Kconfig
> > @@ -1041,14 +1041,15 @@ config MMC_SDHCI_OMAP
> >
> >  config MMC_SDHCI_AM654
> >       tristate "Support for the SDHCI Controller in TI's AM654 SOCs"
> > +     depends on ARCH_K3 || COMPILE_TEST
> >       depends on MMC_SDHCI_PLTFM && OF
> >       select MMC_SDHCI_IO_ACCESSORS
> >       select MMC_CQHCI
> >       select REGMAP_MMIO
> >       help
> >         This selects the Secure Digital Host Controller Interface (SDHCI)
> > -       support present in TI's AM654 SOCs. The controller supports
> > -       SD/MMC/SDIO devices.
> > +       support present in TI's AM65x/AM64x/AM62x/J721E SOCs. The controller
> > +       supports SD/MMC/SDIO devices.
> >
> >         If you have a controller with this interface, say Y or M here.
> >
>
Adrian Hunter Jan. 2, 2024, 10:22 a.m. UTC | #3
On 2/01/24 12:10, Peter Robinson wrote:
> On Tue, Jan 2, 2024 at 10:06 AM Adrian Hunter <adrian.hunter@intel.com> wrote:
>>
>> On 20/12/23 15:59, Peter Robinson wrote:
>>> The sdhci_am654 is specific to recent TI SoCs, update the
>>> dependencies for those SoCs and compile testing. While we're
>>> at it update the text to reflect the wider range of
>>> supported TI SoCS the driver now supports.
>>>
>>> Fixes: 41fd4caeb00b ("mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver")
>>
>> Is this really a fix?  Seems like a minor improvement.
>> Same question for patch 2/2.
> 
> It is for distro kernels at least :)

Distros ship thousands of modules.  I am not sure one more or
less makes any difference?

> 
>>> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
>>
>> It would be good to get an Ack from a TI person.
>> Same for patch 2/2.
>>
>>> ---
>>>  drivers/mmc/host/Kconfig | 5 +++--
>>>  1 file changed, 3 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
>>> index 58bd5fe4cd25..24ce5576b61a 100644
>>> --- a/drivers/mmc/host/Kconfig
>>> +++ b/drivers/mmc/host/Kconfig
>>> @@ -1041,14 +1041,15 @@ config MMC_SDHCI_OMAP
>>>
>>>  config MMC_SDHCI_AM654
>>>       tristate "Support for the SDHCI Controller in TI's AM654 SOCs"
>>> +     depends on ARCH_K3 || COMPILE_TEST
>>>       depends on MMC_SDHCI_PLTFM && OF
>>>       select MMC_SDHCI_IO_ACCESSORS
>>>       select MMC_CQHCI
>>>       select REGMAP_MMIO
>>>       help
>>>         This selects the Secure Digital Host Controller Interface (SDHCI)
>>> -       support present in TI's AM654 SOCs. The controller supports
>>> -       SD/MMC/SDIO devices.
>>> +       support present in TI's AM65x/AM64x/AM62x/J721E SOCs. The controller
>>> +       supports SD/MMC/SDIO devices.
>>>
>>>         If you have a controller with this interface, say Y or M here.
>>>
>>
Peter Robinson Jan. 2, 2024, 10:29 a.m. UTC | #4
On Tue, Jan 2, 2024 at 10:22 AM Adrian Hunter <adrian.hunter@intel.com> wrote:
>
> On 2/01/24 12:10, Peter Robinson wrote:
> > On Tue, Jan 2, 2024 at 10:06 AM Adrian Hunter <adrian.hunter@intel.com> wrote:
> >>
> >> On 20/12/23 15:59, Peter Robinson wrote:
> >>> The sdhci_am654 is specific to recent TI SoCs, update the
> >>> dependencies for those SoCs and compile testing. While we're
> >>> at it update the text to reflect the wider range of
> >>> supported TI SoCS the driver now supports.
> >>>
> >>> Fixes: 41fd4caeb00b ("mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver")
> >>
> >> Is this really a fix?  Seems like a minor improvement.
> >> Same question for patch 2/2.
> >
> > It is for distro kernels at least :)
>
> Distros ship thousands of modules.  I am not sure one more or
> less makes any difference?

I added the fixes because that's what a lot of other similar patches
do, see 4920bded3e and 1f223208e in CAN drivers I was looking at
recently, if you want to drop the fixes tag I don't mind but I was
mostly just following what numerous other similar patches do :)

> >
> >>> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
> >>
> >> It would be good to get an Ack from a TI person.
> >> Same for patch 2/2.
> >>
> >>> ---
> >>>  drivers/mmc/host/Kconfig | 5 +++--
> >>>  1 file changed, 3 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> >>> index 58bd5fe4cd25..24ce5576b61a 100644
> >>> --- a/drivers/mmc/host/Kconfig
> >>> +++ b/drivers/mmc/host/Kconfig
> >>> @@ -1041,14 +1041,15 @@ config MMC_SDHCI_OMAP
> >>>
> >>>  config MMC_SDHCI_AM654
> >>>       tristate "Support for the SDHCI Controller in TI's AM654 SOCs"
> >>> +     depends on ARCH_K3 || COMPILE_TEST
> >>>       depends on MMC_SDHCI_PLTFM && OF
> >>>       select MMC_SDHCI_IO_ACCESSORS
> >>>       select MMC_CQHCI
> >>>       select REGMAP_MMIO
> >>>       help
> >>>         This selects the Secure Digital Host Controller Interface (SDHCI)
> >>> -       support present in TI's AM654 SOCs. The controller supports
> >>> -       SD/MMC/SDIO devices.
> >>> +       support present in TI's AM65x/AM64x/AM62x/J721E SOCs. The controller
> >>> +       supports SD/MMC/SDIO devices.
> >>>
> >>>         If you have a controller with this interface, say Y or M here.
> >>>
> >>
>
Ulf Hansson Jan. 2, 2024, 4:57 p.m. UTC | #5
On Wed, 20 Dec 2023 at 14:59, Peter Robinson <pbrobinson@gmail.com> wrote:
>
> The sdhci_am654 is specific to recent TI SoCs, update the
> dependencies for those SoCs and compile testing. While we're
> at it update the text to reflect the wider range of
> supported TI SoCS the driver now supports.
>
> Fixes: 41fd4caeb00b ("mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver")
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>

Applied for next (and patch2/2), thanks!

Kind regards
Uffe


> ---
>  drivers/mmc/host/Kconfig | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> index 58bd5fe4cd25..24ce5576b61a 100644
> --- a/drivers/mmc/host/Kconfig
> +++ b/drivers/mmc/host/Kconfig
> @@ -1041,14 +1041,15 @@ config MMC_SDHCI_OMAP
>
>  config MMC_SDHCI_AM654
>         tristate "Support for the SDHCI Controller in TI's AM654 SOCs"
> +       depends on ARCH_K3 || COMPILE_TEST
>         depends on MMC_SDHCI_PLTFM && OF
>         select MMC_SDHCI_IO_ACCESSORS
>         select MMC_CQHCI
>         select REGMAP_MMIO
>         help
>           This selects the Secure Digital Host Controller Interface (SDHCI)
> -         support present in TI's AM654 SOCs. The controller supports
> -         SD/MMC/SDIO devices.
> +         support present in TI's AM65x/AM64x/AM62x/J721E SOCs. The controller
> +         supports SD/MMC/SDIO devices.
>
>           If you have a controller with this interface, say Y or M here.
>
> --
> 2.43.0
>
diff mbox series

Patch

diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
index 58bd5fe4cd25..24ce5576b61a 100644
--- a/drivers/mmc/host/Kconfig
+++ b/drivers/mmc/host/Kconfig
@@ -1041,14 +1041,15 @@  config MMC_SDHCI_OMAP
 
 config MMC_SDHCI_AM654
 	tristate "Support for the SDHCI Controller in TI's AM654 SOCs"
+	depends on ARCH_K3 || COMPILE_TEST
 	depends on MMC_SDHCI_PLTFM && OF
 	select MMC_SDHCI_IO_ACCESSORS
 	select MMC_CQHCI
 	select REGMAP_MMIO
 	help
 	  This selects the Secure Digital Host Controller Interface (SDHCI)
-	  support present in TI's AM654 SOCs. The controller supports
-	  SD/MMC/SDIO devices.
+	  support present in TI's AM65x/AM64x/AM62x/J721E SOCs. The controller
+	  supports SD/MMC/SDIO devices.
 
 	  If you have a controller with this interface, say Y or M here.