2020-06-19 13:05:04

by Faiz Abbas

[permalink] [raw]
Subject: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

Enable CONFIG_SDHCI_AM654 to Support AM65x sdhci controller.

Signed-off-by: Faiz Abbas <[email protected]>
---
arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 883e8bace3ed..40dd13e0adc5 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -731,6 +731,7 @@ CONFIG_MMC_DW_ROCKCHIP=y
CONFIG_MMC_SUNXI=y
CONFIG_MMC_BCM2835=y
CONFIG_MMC_SDHCI_XENON=y
+CONFIG_MMC_SDHCI_AM654=y
CONFIG_MMC_OWL=y
CONFIG_NEW_LEDS=y
CONFIG_LEDS_CLASS=y
--
2.17.1


2020-07-16 12:24:06

by Faiz Abbas

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

Hi,

On 19/06/20 6:28 pm, Faiz Abbas wrote:
> Enable CONFIG_SDHCI_AM654 to Support AM65x sdhci controller.
>
> Signed-off-by: Faiz Abbas <[email protected]>
> ---
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 883e8bace3ed..40dd13e0adc5 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -731,6 +731,7 @@ CONFIG_MMC_DW_ROCKCHIP=y
> CONFIG_MMC_SUNXI=y
> CONFIG_MMC_BCM2835=y
> CONFIG_MMC_SDHCI_XENON=y
> +CONFIG_MMC_SDHCI_AM654=y
> CONFIG_MMC_OWL=y
> CONFIG_NEW_LEDS=y
> CONFIG_LEDS_CLASS=y
>

Gentle ping. Will, Catalin, can this patch be picked up?

Thanks,
Faiz

2020-07-16 13:29:04

by Sekhar Nori

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

On 7/16/20 5:49 PM, Faiz Abbas wrote:
> Hi,
>
> On 19/06/20 6:28 pm, Faiz Abbas wrote:
>> Enable CONFIG_SDHCI_AM654 to Support AM65x sdhci controller.
>>
>> Signed-off-by: Faiz Abbas <[email protected]>
>> ---
>> arch/arm64/configs/defconfig | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>> index 883e8bace3ed..40dd13e0adc5 100644
>> --- a/arch/arm64/configs/defconfig
>> +++ b/arch/arm64/configs/defconfig
>> @@ -731,6 +731,7 @@ CONFIG_MMC_DW_ROCKCHIP=y
>> CONFIG_MMC_SUNXI=y
>> CONFIG_MMC_BCM2835=y
>> CONFIG_MMC_SDHCI_XENON=y
>> +CONFIG_MMC_SDHCI_AM654=y
>> CONFIG_MMC_OWL=y
>> CONFIG_NEW_LEDS=y
>> CONFIG_LEDS_CLASS=y
>>
>
> Gentle ping. Will, Catalin, can this patch be picked up?

From logs, Arnd has been picking up patches for this file. Looping in
Arnd and ARM-SoC team.

Thanks,
Sekhar

2020-07-16 18:30:05

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

On Thu, Jul 16, 2020 at 3:25 PM Sekhar Nori <[email protected]> wrote:
>
> On 7/16/20 5:49 PM, Faiz Abbas wrote:
> > Hi,
> >
> > On 19/06/20 6:28 pm, Faiz Abbas wrote:
> >> Enable CONFIG_SDHCI_AM654 to Support AM65x sdhci controller.
> >>
> >> Signed-off-by: Faiz Abbas <[email protected]>
> >> ---
> >> arch/arm64/configs/defconfig | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> >> index 883e8bace3ed..40dd13e0adc5 100644
> >> --- a/arch/arm64/configs/defconfig
> >> +++ b/arch/arm64/configs/defconfig
> >> @@ -731,6 +731,7 @@ CONFIG_MMC_DW_ROCKCHIP=y
> >> CONFIG_MMC_SUNXI=y
> >> CONFIG_MMC_BCM2835=y
> >> CONFIG_MMC_SDHCI_XENON=y
> >> +CONFIG_MMC_SDHCI_AM654=y
> >> CONFIG_MMC_OWL=y
> >> CONFIG_NEW_LEDS=y
> >> CONFIG_LEDS_CLASS=y
> >>
> >
> > Gentle ping. Will, Catalin, can this patch be picked up?
>
> From logs, Arnd has been picking up patches for this file. Looping in
> Arnd and ARM-SoC team.

I tend to ignore individual patches to the defconfig file unless
they are sent to:[email protected]. The best way to get them
included is to have the platform maintainers pick up the
changes and send them that way as a separate pull request
at the same time as sending any DT updates.

The MAINTAINERS file lists Tero and Nishanth as maintainers
for the platform. If they want, I can apply this one directly, but in
the future, send it to them.

Arnd

2020-07-17 08:40:54

by Faiz Abbas

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

Hi,

On 16/07/20 11:58 pm, Arnd Bergmann wrote:
> On Thu, Jul 16, 2020 at 3:25 PM Sekhar Nori <[email protected]> wrote:
>>
>> On 7/16/20 5:49 PM, Faiz Abbas wrote:
>>> Hi,
>>>
>>> On 19/06/20 6:28 pm, Faiz Abbas wrote:
>>>> Enable CONFIG_SDHCI_AM654 to Support AM65x sdhci controller.
>>>>
>>>> Signed-off-by: Faiz Abbas <[email protected]>
>>>> ---
>>>> arch/arm64/configs/defconfig | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>>>> index 883e8bace3ed..40dd13e0adc5 100644
>>>> --- a/arch/arm64/configs/defconfig
>>>> +++ b/arch/arm64/configs/defconfig
>>>> @@ -731,6 +731,7 @@ CONFIG_MMC_DW_ROCKCHIP=y
>>>> CONFIG_MMC_SUNXI=y
>>>> CONFIG_MMC_BCM2835=y
>>>> CONFIG_MMC_SDHCI_XENON=y
>>>> +CONFIG_MMC_SDHCI_AM654=y
>>>> CONFIG_MMC_OWL=y
>>>> CONFIG_NEW_LEDS=y
>>>> CONFIG_LEDS_CLASS=y
>>>>
>>>
>>> Gentle ping. Will, Catalin, can this patch be picked up?
>>
>> From logs, Arnd has been picking up patches for this file. Looping in
>> Arnd and ARM-SoC team.
>
> I tend to ignore individual patches to the defconfig file unless
> they are sent to:[email protected]. The best way to get them
> included is to have the platform maintainers pick up the
> changes and send them that way as a separate pull request
> at the same time as sending any DT updates.
>
> The MAINTAINERS file lists Tero and Nishanth as maintainers
> for the platform. If they want, I can apply this one directly, but in
> the future, send it to them.
>

Thanks for clarifying Arnd. Tero, can you pick this up?

Thanks,
Faiz

2020-07-17 11:21:17

by Tero Kristo

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

On 17/07/2020 11:38, Faiz Abbas wrote:
> Hi,
>
> On 16/07/20 11:58 pm, Arnd Bergmann wrote:
>> On Thu, Jul 16, 2020 at 3:25 PM Sekhar Nori <[email protected]> wrote:
>>>
>>> On 7/16/20 5:49 PM, Faiz Abbas wrote:
>>>> Hi,
>>>>
>>>> On 19/06/20 6:28 pm, Faiz Abbas wrote:
>>>>> Enable CONFIG_SDHCI_AM654 to Support AM65x sdhci controller.
>>>>>
>>>>> Signed-off-by: Faiz Abbas <[email protected]>
>>>>> ---
>>>>> arch/arm64/configs/defconfig | 1 +
>>>>> 1 file changed, 1 insertion(+)
>>>>>
>>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>>>>> index 883e8bace3ed..40dd13e0adc5 100644
>>>>> --- a/arch/arm64/configs/defconfig
>>>>> +++ b/arch/arm64/configs/defconfig
>>>>> @@ -731,6 +731,7 @@ CONFIG_MMC_DW_ROCKCHIP=y
>>>>> CONFIG_MMC_SUNXI=y
>>>>> CONFIG_MMC_BCM2835=y
>>>>> CONFIG_MMC_SDHCI_XENON=y
>>>>> +CONFIG_MMC_SDHCI_AM654=y
>>>>> CONFIG_MMC_OWL=y
>>>>> CONFIG_NEW_LEDS=y
>>>>> CONFIG_LEDS_CLASS=y
>>>>>
>>>>
>>>> Gentle ping. Will, Catalin, can this patch be picked up?
>>>
>>> From logs, Arnd has been picking up patches for this file. Looping in
>>> Arnd and ARM-SoC team.
>>
>> I tend to ignore individual patches to the defconfig file unless
>> they are sent to:[email protected]. The best way to get them
>> included is to have the platform maintainers pick up the
>> changes and send them that way as a separate pull request
>> at the same time as sending any DT updates.
>>
>> The MAINTAINERS file lists Tero and Nishanth as maintainers
>> for the platform. If they want, I can apply this one directly, but in
>> the future, send it to them.
>>
>
> Thanks for clarifying Arnd. Tero, can you pick this up?

Ok, this topic has been bit unclear for me also, but if you say I can
pick the patches myself and send a pull request out, I can do that.

-Tero
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

2020-07-17 13:10:20

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

On Fri, Jul 17, 2020 at 1:20 PM Tero Kristo <[email protected]> wrote:
> On 17/07/2020 11:38, Faiz Abbas wrote:
> > On 16/07/20 11:58 pm, Arnd Bergmann wrote:
> >> On Thu, Jul 16, 2020 at 3:25 PM Sekhar Nori <[email protected]> wrote:
> >> I tend to ignore individual patches to the defconfig file unless
> >> they are sent to:[email protected]. The best way to get them
> >> included is to have the platform maintainers pick up the
> >> changes and send them that way as a separate pull request
> >> at the same time as sending any DT updates.
> >>
> >> The MAINTAINERS file lists Tero and Nishanth as maintainers
> >> for the platform. If they want, I can apply this one directly, but in
> >> the future, send it to them.
> >>
> >
> > Thanks for clarifying Arnd. Tero, can you pick this up?
>
> Ok, this topic has been bit unclear for me also, but if you say I can
> pick the patches myself and send a pull request out, I can do that.

Right. To clarify, the soc tree usually has separate branches for dts
files, soc specific drivers, defconfig files and 32-bit platform code.

When you pick up patches into your tree, please put them into
branches that fit into those categories. You can group the patches
into branches with more fine-grained categories if it makes sense
(e.g. adding a particularly large driver, adding a new dts files for a
new soc, or cosmetic cleanups across dts files).

If any of the categories only have a couple of patches in them, you
can decide to forward those as patches to [email protected], but a
pull request is always ok as well, even for a one-line patch.

Arnd

Arnd

2020-07-17 13:37:26

by Faiz Abbas

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

Tero,

On 17/07/20 6:39 pm, Arnd Bergmann wrote:
> On Fri, Jul 17, 2020 at 1:20 PM Tero Kristo <[email protected]> wrote:
>> On 17/07/2020 11:38, Faiz Abbas wrote:
>>> On 16/07/20 11:58 pm, Arnd Bergmann wrote:
>>>> On Thu, Jul 16, 2020 at 3:25 PM Sekhar Nori <[email protected]> wrote:
>>>> I tend to ignore individual patches to the defconfig file unless
>>>> they are sent to:[email protected]. The best way to get them
>>>> included is to have the platform maintainers pick up the
>>>> changes and send them that way as a separate pull request
>>>> at the same time as sending any DT updates.
>>>>
>>>> The MAINTAINERS file lists Tero and Nishanth as maintainers
>>>> for the platform. If they want, I can apply this one directly, but in
>>>> the future, send it to them.
>>>>
>>>
>>> Thanks for clarifying Arnd. Tero, can you pick this up?
>>
>> Ok, this topic has been bit unclear for me also, but if you say I can
>> pick the patches myself and send a pull request out, I can do that.
>

Patches 1-6 are being merged from mmc tree. You only need to pick up this patch.

Thanks,
Faiz

2020-07-17 13:46:33

by Tero Kristo

[permalink] [raw]
Subject: Re: [PATCH 7/7] arm64: defconfig: Enable AM654x SDHCI controller

On 17/07/2020 16:09, Arnd Bergmann wrote:
> On Fri, Jul 17, 2020 at 1:20 PM Tero Kristo <[email protected]> wrote:
>> On 17/07/2020 11:38, Faiz Abbas wrote:
>>> On 16/07/20 11:58 pm, Arnd Bergmann wrote:
>>>> On Thu, Jul 16, 2020 at 3:25 PM Sekhar Nori <[email protected]> wrote:
>>>> I tend to ignore individual patches to the defconfig file unless
>>>> they are sent to:[email protected]. The best way to get them
>>>> included is to have the platform maintainers pick up the
>>>> changes and send them that way as a separate pull request
>>>> at the same time as sending any DT updates.
>>>>
>>>> The MAINTAINERS file lists Tero and Nishanth as maintainers
>>>> for the platform. If they want, I can apply this one directly, but in
>>>> the future, send it to them.
>>>>
>>>
>>> Thanks for clarifying Arnd. Tero, can you pick this up?
>>
>> Ok, this topic has been bit unclear for me also, but if you say I can
>> pick the patches myself and send a pull request out, I can do that.
>
> Right. To clarify, the soc tree usually has separate branches for dts
> files, soc specific drivers, defconfig files and 32-bit platform code.
>
> When you pick up patches into your tree, please put them into
> branches that fit into those categories. You can group the patches
> into branches with more fine-grained categories if it makes sense
> (e.g. adding a particularly large driver, adding a new dts files for a
> new soc, or cosmetic cleanups across dts files).
>
> If any of the categories only have a couple of patches in them, you
> can decide to forward those as patches to [email protected], but a
> pull request is always ok as well, even for a one-line patch.

Ok thanks for clarification, Arnd.

Based on that, queuing this up for 5.9 myself, thanks.

Will post pull-request next week for it, there appears to be another K3
SoC related config change pending which I'll pick up also. Just want to
capture -next results for these to see how well they integrate.

-Tero
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki