2017-09-18 16:53:55

by Babu Moger

[permalink] [raw]
Subject: [PATCH 0/2] Fix duplicates and change default endian for microblaze

This series fixes few Kconfig issues.

Babu Moger (2):
arch: Fix duplicates in Kconfig for parisc and sparc
arch: change default endian for microblaze

arch/microblaze/Kconfig | 2 +-
arch/parisc/Kconfig | 3 ---
arch/sparc/Kconfig | 3 ---
3 files changed, 1 insertions(+), 7 deletions(-)


2017-09-18 16:53:53

by Babu Moger

[permalink] [raw]
Subject: [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc

Fix duplicates for sparc and parisc. This was due these following commits.

1. commit 4c97a0c8fee3 ("arch: define CPU_BIG_ENDIAN for all fixed big
endian archs")
2. commit 97d9f969161d ("arch/sparc: Define config parameter
CPU_BIG_ENDIAN")
3. commit 74ad3d28af21 ("parisc: Define CONFIG_CPU_BIG_ENDIAN")

Remove duplicates.

Signed-off-by: Babu Moger <[email protected]>
---
arch/parisc/Kconfig | 3 ---
arch/sparc/Kconfig | 3 ---
2 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/arch/parisc/Kconfig b/arch/parisc/Kconfig
index ba7b7dd..1364851 100644
--- a/arch/parisc/Kconfig
+++ b/arch/parisc/Kconfig
@@ -59,9 +59,6 @@ config PARISC
config CPU_BIG_ENDIAN
def_bool y

-config CPU_BIG_ENDIAN
- def_bool y
-
config MMU
def_bool y

diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig
index 0be3828..a4a6261 100644
--- a/arch/sparc/Kconfig
+++ b/arch/sparc/Kconfig
@@ -97,9 +97,6 @@ config ARCH_PROC_KCORE_TEXT
config CPU_BIG_ENDIAN
def_bool y

-config CPU_BIG_ENDIAN
- def_bool y
-
config ARCH_ATU
bool
default y if SPARC64
--
1.7.1

2017-09-18 16:54:10

by Babu Moger

[permalink] [raw]
Subject: [PATCH 2/2] arch: change default endian for microblaze

Fix the default for microblaze. Michal Simek mentioned default for
microblaze should be CPU_LITTLE_ENDIAN.

Fixes : commit 206d3642d8ee ("arch/microblaze: add choice for endianness
and update Makefile")

Signed-off-by: Babu Moger <[email protected]>
Cc: Michal Simek <[email protected]>
---
arch/microblaze/Kconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
index 9d26abd..4f798aa 100644
--- a/arch/microblaze/Kconfig
+++ b/arch/microblaze/Kconfig
@@ -39,7 +39,7 @@ config MICROBLAZE
# Endianness selection
choice
prompt "Endianness selection"
- default CPU_BIG_ENDIAN
+ default CPU_LITTLE_ENDIAN
help
microblaze architectures can be configured for either little or
big endian formats. Be sure to select the appropriate mode.
--
1.7.1

2017-09-18 17:49:38

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH 2/2] arch: change default endian for microblaze

On 18.9.2017 18:53, Babu Moger wrote:
> Fix the default for microblaze. Michal Simek mentioned default for
> microblaze should be CPU_LITTLE_ENDIAN.
>
> Fixes : commit 206d3642d8ee ("arch/microblaze: add choice for endianness
> and update Makefile")
>
> Signed-off-by: Babu Moger <[email protected]>
> Cc: Michal Simek <[email protected]>
> ---
> arch/microblaze/Kconfig | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
> index 9d26abd..4f798aa 100644
> --- a/arch/microblaze/Kconfig
> +++ b/arch/microblaze/Kconfig
> @@ -39,7 +39,7 @@ config MICROBLAZE
> # Endianness selection
> choice
> prompt "Endianness selection"
> - default CPU_BIG_ENDIAN
> + default CPU_LITTLE_ENDIAN
> help
> microblaze architectures can be configured for either little or
> big endian formats. Be sure to select the appropriate mode.
>

Acked-by: Michal Simek <[email protected]>

If you want me to take this through my microblaze tree please let me know.

Thanks,
Michal

--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs



Attachments:
signature.asc (198.00 B)
OpenPGP digital signature

2017-09-18 18:06:05

by Babu Moger

[permalink] [raw]
Subject: Re: [PATCH 2/2] arch: change default endian for microblaze


On 9/18/2017 12:49 PM, Michal Simek wrote:
> On 18.9.2017 18:53, Babu Moger wrote:
>> Fix the default for microblaze. Michal Simek mentioned default for
>> microblaze should be CPU_LITTLE_ENDIAN.
>>
>> Fixes : commit 206d3642d8ee ("arch/microblaze: add choice for endianness
>> and update Makefile")
>>
>> Signed-off-by: Babu Moger <[email protected]>
>> Cc: Michal Simek <[email protected]>
>> ---
>> arch/microblaze/Kconfig | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
>> index 9d26abd..4f798aa 100644
>> --- a/arch/microblaze/Kconfig
>> +++ b/arch/microblaze/Kconfig
>> @@ -39,7 +39,7 @@ config MICROBLAZE
>> # Endianness selection
>> choice
>> prompt "Endianness selection"
>> - default CPU_BIG_ENDIAN
>> + default CPU_LITTLE_ENDIAN
>> help
>> microblaze architectures can be configured for either little or
>> big endian formats. Be sure to select the appropriate mode.
>>
> Acked-by: Michal Simek <[email protected]>
Thanks.
> If you want me to take this through my microblaze tree please let me know.
Sure. Thanks
>
> Thanks,
> Michal
>

2017-09-25 13:37:14

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH 2/2] arch: change default endian for microblaze

On 18.9.2017 20:05, Babu Moger wrote:
>
> On 9/18/2017 12:49 PM, Michal Simek wrote:
>> On 18.9.2017 18:53, Babu Moger wrote:
>>> Fix the default for microblaze. Michal Simek mentioned default for
>>> microblaze should be CPU_LITTLE_ENDIAN.
>>>
>>> Fixes : commit 206d3642d8ee ("arch/microblaze: add choice for endianness
>>>     and update Makefile")
>>>
>>> Signed-off-by: Babu Moger <[email protected]>
>>> Cc: Michal Simek <[email protected]>
>>> ---
>>>   arch/microblaze/Kconfig |    2 +-
>>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
>>> index 9d26abd..4f798aa 100644
>>> --- a/arch/microblaze/Kconfig
>>> +++ b/arch/microblaze/Kconfig
>>> @@ -39,7 +39,7 @@ config MICROBLAZE
>>>   # Endianness selection
>>>   choice
>>>       prompt "Endianness selection"
>>> -    default CPU_BIG_ENDIAN
>>> +    default CPU_LITTLE_ENDIAN
>>>       help
>>>         microblaze architectures can be configured for either little or
>>>         big endian formats. Be sure to select the appropriate mode.
>>>
>> Acked-by: Michal Simek <[email protected]>
> Thanks.
>> If you want me to take this through my microblaze tree please let me
>> know.
> Sure. Thanks

I can't see this in linux-next yet that's why pplied this one to mb/next
to be the part of my pull request.

Thanks,
Michal

--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs



Attachments:
signature.asc (198.00 B)
OpenPGP digital signature