diff mbox

[RFC] arm64: defconfig: enable SMMUv3 config

Message ID 1483962646-33767-1-git-send-email-wangzhou1@hisilicon.com (mailing list archive)
State New, archived
Headers show

Commit Message

Zhou Wang Jan. 9, 2017, 11:50 a.m. UTC 
Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Zhou Wang Jan. 12, 2017, 12:36 a.m. UTC | #1 
On 2017/1/9 19:50, Zhou Wang wrote:
> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
> ---
>  arch/arm64/configs/defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 869dded..3520c50 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
>  CONFIG_BCM2835_MBOX=y
>  CONFIG_HI6220_MBOX=y
>  CONFIG_ARM_SMMU=y
> +CONFIG_ARM_SMMU_V3=y
>  CONFIG_RASPBERRYPI_POWER=y
>  CONFIG_QCOM_SMEM=y
>  CONFIG_QCOM_SMD=y
>

Hi Catalin, Will and Arnd,

I just happened to find there is no SMMUv3 config in arm64 defconfig.

Maybe we should add it in defconfig or I miss something.

Thanks,
Zhou

.
Catalin Marinas Jan. 12, 2017, 9:47 a.m. UTC | #2 
On Thu, Jan 12, 2017 at 08:36:12AM +0800, Zhou Wang wrote:
> On 2017/1/9 19:50, Zhou Wang wrote:
> > Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
> > ---
> >  arch/arm64/configs/defconfig | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index 869dded..3520c50 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
> >  CONFIG_BCM2835_MBOX=y
> >  CONFIG_HI6220_MBOX=y
> >  CONFIG_ARM_SMMU=y
> > +CONFIG_ARM_SMMU_V3=y
> >  CONFIG_RASPBERRYPI_POWER=y
> >  CONFIG_QCOM_SMEM=y
> >  CONFIG_QCOM_SMD=y
> 
> I just happened to find there is no SMMUv3 config in arm64 defconfig.
> 
> Maybe we should add it in defconfig or I miss something.

It looks fine to me but it's usually the arm-soc guys picking the
defconfig patches.
Wei Xu Jan. 13, 2017, 3:35 p.m. UTC | #3 
On 2017/1/12 9:47, Catalin Marinas wrote:
> On Thu, Jan 12, 2017 at 08:36:12AM +0800, Zhou Wang wrote:
>> On 2017/1/9 19:50, Zhou Wang wrote:
>>> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
>>> ---
>>>  arch/arm64/configs/defconfig | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>>> index 869dded..3520c50 100644
>>> --- a/arch/arm64/configs/defconfig
>>> +++ b/arch/arm64/configs/defconfig
>>> @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
>>>  CONFIG_BCM2835_MBOX=y
>>>  CONFIG_HI6220_MBOX=y
>>>  CONFIG_ARM_SMMU=y
>>> +CONFIG_ARM_SMMU_V3=y
>>>  CONFIG_RASPBERRYPI_POWER=y
>>>  CONFIG_QCOM_SMEM=y
>>>  CONFIG_QCOM_SMD=y
>>
>> I just happened to find there is no SMMUv3 config in arm64 defconfig.
>>
>> Maybe we should add it in defconfig or I miss something.
> 

Hi Arnd,

> It looks fine to me but it's usually the arm-soc guys picking the
> defconfig patches.
> 

It is fine to me.
Is it OK for me to pick up this or will you directly pick up this?
Thanks!

Best Regards,
Wei Xu
Arnd Bergmann Jan. 13, 2017, 4:58 p.m. UTC | #4 
On Friday, January 13, 2017 3:35:54 PM CET Wei Xu wrote:
> On 2017/1/12 9:47, Catalin Marinas wrote:
> > On Thu, Jan 12, 2017 at 08:36:12AM +0800, Zhou Wang wrote:
> >> On 2017/1/9 19:50, Zhou Wang wrote:
> >>> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
> >>> ---
> >>>  arch/arm64/configs/defconfig | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> >>> index 869dded..3520c50 100644
> >>> --- a/arch/arm64/configs/defconfig
> >>> +++ b/arch/arm64/configs/defconfig
> >>> @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
> >>>  CONFIG_BCM2835_MBOX=y
> >>>  CONFIG_HI6220_MBOX=y
> >>>  CONFIG_ARM_SMMU=y
> >>> +CONFIG_ARM_SMMU_V3=y
> >>>  CONFIG_RASPBERRYPI_POWER=y
> >>>  CONFIG_QCOM_SMEM=y
> >>>  CONFIG_QCOM_SMD=y
> >>
> >> I just happened to find there is no SMMUv3 config in arm64 defconfig.
> >>
> >> Maybe we should add it in defconfig or I miss something.
> > 
> 
> Hi Arnd,
> 
> > It looks fine to me but it's usually the arm-soc guys picking the
> > defconfig patches.
> > 
> 
> It is fine to me.
> Is it OK for me to pick up this or will you directly pick up this?
> Thanks!

Either way works for me. I usually recommend platform maintainers
picking up patches and then sending them on to arm@kernel.org,
as that gets our attention quicker. A patch that gets sent with
me personally on Cc might get lost in the email.

	Arnd
Wei Xu Jan. 16, 2017, 4:15 p.m. UTC | #5 
Hi Arnd,

On 2017/1/13 16:58, Arnd Bergmann wrote:
> On Friday, January 13, 2017 3:35:54 PM CET Wei Xu wrote:
>> On 2017/1/12 9:47, Catalin Marinas wrote:
>>> On Thu, Jan 12, 2017 at 08:36:12AM +0800, Zhou Wang wrote:
>>>> On 2017/1/9 19:50, Zhou Wang wrote:
>>>>> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
>>>>> ---
>>>>>  arch/arm64/configs/defconfig | 1 +
>>>>>  1 file changed, 1 insertion(+)
>>>>>
>>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>>>>> index 869dded..3520c50 100644
>>>>> --- a/arch/arm64/configs/defconfig
>>>>> +++ b/arch/arm64/configs/defconfig
>>>>> @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
>>>>>  CONFIG_BCM2835_MBOX=y
>>>>>  CONFIG_HI6220_MBOX=y
>>>>>  CONFIG_ARM_SMMU=y
>>>>> +CONFIG_ARM_SMMU_V3=y
>>>>>  CONFIG_RASPBERRYPI_POWER=y
>>>>>  CONFIG_QCOM_SMEM=y
>>>>>  CONFIG_QCOM_SMD=y
>>>>
>>>> I just happened to find there is no SMMUv3 config in arm64 defconfig.
>>>>
>>>> Maybe we should add it in defconfig or I miss something.
>>>
>>
>> Hi Arnd,
>>
>>> It looks fine to me but it's usually the arm-soc guys picking the
>>> defconfig patches.
>>>
>>
>> It is fine to me.
>> Is it OK for me to pick up this or will you directly pick up this?
>> Thanks!
> 
> Either way works for me. I usually recommend platform maintainers
> picking up patches and then sending them on to arm@kernel.org,
> as that gets our attention quicker. A patch that gets sent with
> me personally on Cc might get lost in the email.

In this case I will pick up it.
Thanks!

Best Regards,
Wei

> 
> 	Arnd
> 
> 
> .
>
Zhou Wang Jan. 17, 2017, 12:56 a.m. UTC | #6 
On 2017/1/17 0:15, Wei Xu wrote:
> Hi Arnd,
> 
> On 2017/1/13 16:58, Arnd Bergmann wrote:
>> On Friday, January 13, 2017 3:35:54 PM CET Wei Xu wrote:
>>> On 2017/1/12 9:47, Catalin Marinas wrote:
>>>> On Thu, Jan 12, 2017 at 08:36:12AM +0800, Zhou Wang wrote:
>>>>> On 2017/1/9 19:50, Zhou Wang wrote:
>>>>>> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>
>>>>>> ---
>>>>>>  arch/arm64/configs/defconfig | 1 +
>>>>>>  1 file changed, 1 insertion(+)
>>>>>>
>>>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>>>>>> index 869dded..3520c50 100644
>>>>>> --- a/arch/arm64/configs/defconfig
>>>>>> +++ b/arch/arm64/configs/defconfig
>>>>>> @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
>>>>>>  CONFIG_BCM2835_MBOX=y
>>>>>>  CONFIG_HI6220_MBOX=y
>>>>>>  CONFIG_ARM_SMMU=y
>>>>>> +CONFIG_ARM_SMMU_V3=y
>>>>>>  CONFIG_RASPBERRYPI_POWER=y
>>>>>>  CONFIG_QCOM_SMEM=y
>>>>>>  CONFIG_QCOM_SMD=y
>>>>>
>>>>> I just happened to find there is no SMMUv3 config in arm64 defconfig.
>>>>>
>>>>> Maybe we should add it in defconfig or I miss something.
>>>>
>>>
>>> Hi Arnd,
>>>
>>>> It looks fine to me but it's usually the arm-soc guys picking the
>>>> defconfig patches.
>>>>
>>>
>>> It is fine to me.
>>> Is it OK for me to pick up this or will you directly pick up this?
>>> Thanks!
>>
>> Either way works for me. I usually recommend platform maintainers
>> picking up patches and then sending them on to arm@kernel.org,
>> as that gets our attention quicker. A patch that gets sent with
>> me personally on Cc might get lost in the email.
> 
> In this case I will pick up it.

Many thanks,

Regards,
Zhou

> Thanks!
> 
> Best Regards,
> Wei
> 
>>
>> 	Arnd
>>
>>
>> .
>>
> 
> 
> .
>
Wei Xu Jan. 20, 2017, 4:57 p.m. UTC | #7 
Hi Zhou Wang,

On 2017/1/9 11:50, Zhou Wang wrote:
> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>

Applied to the hisilicon soc tree.
Thanks!

Best Regards,
Wei

> ---
>  arch/arm64/configs/defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 869dded..3520c50 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -440,6 +440,7 @@ CONFIG_PLATFORM_MHU=y
>  CONFIG_BCM2835_MBOX=y
>  CONFIG_HI6220_MBOX=y
>  CONFIG_ARM_SMMU=y
> +CONFIG_ARM_SMMU_V3=y
>  CONFIG_RASPBERRYPI_POWER=y
>  CONFIG_QCOM_SMEM=y
>  CONFIG_QCOM_SMD=y
>
diff mbox

Patch

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 869dded..3520c50 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -440,6 +440,7 @@  CONFIG_PLATFORM_MHU=y
 CONFIG_BCM2835_MBOX=y
 CONFIG_HI6220_MBOX=y
 CONFIG_ARM_SMMU=y
+CONFIG_ARM_SMMU_V3=y
 CONFIG_RASPBERRYPI_POWER=y
 CONFIG_QCOM_SMEM=y
 CONFIG_QCOM_SMD=y