diff mbox

ARM: keystone: remove redundant depends on ARM_PATCH_PHYS_VIRT

Message ID 1454694041-14990-1-git-send-email-yamada.masahiro@socionext.com (mailing list archive)
State New, archived
Headers show

Commit Message

Masahiro Yamada Feb. 5, 2016, 5:40 p.m. UTC
KeyStone is Multi-platform.  ARM_PATCH_PHYS_VIRT is already select'ed
by ARCH_MULTIPLATFORM.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 arch/arm/mach-keystone/Kconfig | 1 -
 1 file changed, 1 deletion(-)

Comments

Santosh Shilimkar Feb. 5, 2016, 5:56 p.m. UTC | #1
On 2/5/2016 9:40 AM, Masahiro Yamada wrote:
> KeyStone is Multi-platform.  ARM_PATCH_PHYS_VIRT is already select'ed
> by ARCH_MULTIPLATFORM.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
>
There is also Keystone config so its nit redundant as you think.

P.S: arch/arm/configs/keystone_defconfig

Regards,
Santosh
Masahiro Yamada Feb. 5, 2016, 6:13 p.m. UTC | #2
2016-02-06 2:56 GMT+09:00 santosh shilimkar <santosh.shilimkar@oracle.com>:
> On 2/5/2016 9:40 AM, Masahiro Yamada wrote:
>>
>> KeyStone is Multi-platform.  ARM_PATCH_PHYS_VIRT is already select'ed
>> by ARCH_MULTIPLATFORM.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>> ---
>>
> There is also Keystone config so its nit redundant as you think.
>
> P.S: arch/arm/configs/keystone_defconfig
>

It *is* redundant.

keystone_defconfig also creates Multiplatform configuration.
Santosh Shilimkar Feb. 5, 2016, 6:31 p.m. UTC | #3
On 2/5/2016 10:13 AM, Masahiro Yamada wrote:
> 2016-02-06 2:56 GMT+09:00 santosh shilimkar <santosh.shilimkar@oracle.com>:
>> On 2/5/2016 9:40 AM, Masahiro Yamada wrote:
>>>
>>> KeyStone is Multi-platform.  ARM_PATCH_PHYS_VIRT is already select'ed
>>> by ARCH_MULTIPLATFORM.
>>>
>>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>>> ---
>>>
>> There is also Keystone config so its nit redundant as you think.
>>
>> P.S: arch/arm/configs/keystone_defconfig
>>
>
> It *is* redundant.
>
> keystone_defconfig also creates Multiplatform configuration.
>
Fair enough. Should I checked it.
For patch,
Acked-by: Santosh Shilimkar<ssantosh@kernel.org>
Masahiro Yamada April 5, 2016, 2:14 a.m. UTC | #4
2016-02-06 3:31 GMT+09:00 santosh shilimkar <santosh.shilimkar@oracle.com>:
> On 2/5/2016 10:13 AM, Masahiro Yamada wrote:
>>
>> 2016-02-06 2:56 GMT+09:00 santosh shilimkar
>> <santosh.shilimkar@oracle.com>:
>>>
>>> On 2/5/2016 9:40 AM, Masahiro Yamada wrote:
>>>>
>>>>
>>>> KeyStone is Multi-platform.  ARM_PATCH_PHYS_VIRT is already select'ed
>>>> by ARCH_MULTIPLATFORM.
>>>>
>>>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>>>> ---
>>>>
>>> There is also Keystone config so its nit redundant as you think.
>>>
>>> P.S: arch/arm/configs/keystone_defconfig
>>>
>>
>> It *is* redundant.
>>
>> keystone_defconfig also creates Multiplatform configuration.
>>
> Fair enough. Should I checked it.
> For patch,
> Acked-by: Santosh Shilimkar<ssantosh@kernel.org>
>

If it is OK with you, please apply this patch.
Masahiro Yamada June 14, 2016, 6:17 a.m. UTC | #5
Hi Santosh

Ping again.

It is taking so long
for this apparently correct patch.



2016-04-05 11:14 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>:
> 2016-02-06 3:31 GMT+09:00 santosh shilimkar <santosh.shilimkar@oracle.com>:
>> On 2/5/2016 10:13 AM, Masahiro Yamada wrote:
>>>
>>> 2016-02-06 2:56 GMT+09:00 santosh shilimkar
>>> <santosh.shilimkar@oracle.com>:
>>>>
>>>> On 2/5/2016 9:40 AM, Masahiro Yamada wrote:
>>>>>
>>>>>
>>>>> KeyStone is Multi-platform.  ARM_PATCH_PHYS_VIRT is already select'ed
>>>>> by ARCH_MULTIPLATFORM.
>>>>>
>>>>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>>>>> ---
>>>>>
>>>> There is also Keystone config so its nit redundant as you think.
>>>>
>>>> P.S: arch/arm/configs/keystone_defconfig
>>>>
>>>
>>> It *is* redundant.
>>>
>>> keystone_defconfig also creates Multiplatform configuration.
>>>
>> Fair enough. Should I checked it.
>> For patch,
>> Acked-by: Santosh Shilimkar<ssantosh@kernel.org>
>>
>
> If it is OK with you, please apply this patch.
>
>
> --
> Best Regards
> Masahiro Yamada
Santosh Shilimkar June 14, 2016, 3:36 p.m. UTC | #6
On 6/13/2016 11:17 PM, Masahiro Yamada wrote:
> Hi Santosh
>
> Ping again.
>
> It is taking so long
> for this apparently correct patch.
>
I thought it was already picked up. Will apply it
for next merge window.

Regards,
Santosh
diff mbox

Patch

diff --git a/arch/arm/mach-keystone/Kconfig b/arch/arm/mach-keystone/Kconfig
index ea955f6db..75bcb7f 100644
--- a/arch/arm/mach-keystone/Kconfig
+++ b/arch/arm/mach-keystone/Kconfig
@@ -1,7 +1,6 @@ 
 config ARCH_KEYSTONE
 	bool "Texas Instruments Keystone Devices"
 	depends on ARCH_MULTI_V7
-	depends on ARM_PATCH_PHYS_VIRT
 	select ARM_GIC
 	select HAVE_ARM_ARCH_TIMER
 	select CLKSRC_MMIO