Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1734304 > unrolled thread

[PATCH 0/2] Fix duplicates and change default endian for microblaze

Started byBabu Moger <babu.moger@oracle.com>
First post2017-09-18 19:00 +0200
Last post2017-09-18 19:00 +0200
Articles 6 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] Fix duplicates and change default endian for microblaze Babu Moger <babu.moger@oracle.com> - 2017-09-18 19:00 +0200
    [PATCH 2/2] arch: change default endian for microblaze Babu Moger <babu.moger@oracle.com> - 2017-09-18 19:00 +0200
      Re: [PATCH 2/2] arch: change default endian for microblaze Michal Simek <monstr@monstr.eu> - 2017-09-18 19:50 +0200
        Re: [PATCH 2/2] arch: change default endian for microblaze Babu Moger <babu.moger@oracle.com> - 2017-09-18 20:10 +0200
          Re: [PATCH 2/2] arch: change default endian for microblaze Michal Simek <monstr@monstr.eu> - 2017-09-25 15:40 +0200
    [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc Babu Moger <babu.moger@oracle.com> - 2017-09-18 19:00 +0200

#1734304 — [PATCH 0/2] Fix duplicates and change default endian for microblaze

FromBabu Moger <babu.moger@oracle.com>
Date2017-09-18 19:00 +0200
Subject[PATCH 0/2] Fix duplicates and change default endian for microblaze
Message-ID<ur7yO-mI-13@gated-at.bofh.it>
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(-)

[toc] | [next] | [standalone]

#1734308 — [PATCH 2/2] arch: change default endian for microblaze

FromBabu Moger <babu.moger@oracle.com>
Date2017-09-18 19:00 +0200
Subject[PATCH 2/2] arch: change default endian for microblaze
Message-ID<ur7yO-mI-23@gated-at.bofh.it>
In reply to#1734304
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 <babu.moger@oracle.com>
Cc: Michal Simek <monstr@monstr.eu>
---
 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

[toc] | [prev] | [next] | [standalone]

#1734332 — Re: [PATCH 2/2] arch: change default endian for microblaze

FromMichal Simek <monstr@monstr.eu>
Date2017-09-18 19:50 +0200
SubjectRe: [PATCH 2/2] arch: change default endian for microblaze
Message-ID<ur8lb-TR-3@gated-at.bofh.it>
In reply to#1734308

[Multipart message — attachments visible in raw view] — view raw

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 <babu.moger@oracle.com>
> Cc: Michal Simek <monstr@monstr.eu>
> ---
>  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 <michal.simek@xilinx.com>

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: 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

[toc] | [prev] | [next] | [standalone]

#1734348 — Re: [PATCH 2/2] arch: change default endian for microblaze

FromBabu Moger <babu.moger@oracle.com>
Date2017-09-18 20:10 +0200
SubjectRe: [PATCH 2/2] arch: change default endian for microblaze
Message-ID<ur8Ex-1lQ-1@gated-at.bofh.it>
In reply to#1734332
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 <babu.moger@oracle.com>
>> Cc: Michal Simek <monstr@monstr.eu>
>> ---
>>   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 <michal.simek@xilinx.com>
Thanks.
> If you want me to take this through my microblaze tree please let me know.
Sure. Thanks
>
> Thanks,
> Michal
>

[toc] | [prev] | [next] | [standalone]

#1739057 — Re: [PATCH 2/2] arch: change default endian for microblaze

FromMichal Simek <monstr@monstr.eu>
Date2017-09-25 15:40 +0200
SubjectRe: [PATCH 2/2] arch: change default endian for microblaze
Message-ID<utBM5-6Z8-7@gated-at.bofh.it>
In reply to#1734348

[Multipart message — attachments visible in raw view] — view raw

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 <babu.moger@oracle.com>
>>> Cc: Michal Simek <monstr@monstr.eu>
>>> ---
>>>   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 <michal.simek@xilinx.com>
> 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: 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

[toc] | [prev] | [next] | [standalone]

#1734310 — [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc

FromBabu Moger <babu.moger@oracle.com>
Date2017-09-18 19:00 +0200
Subject[PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc
Message-ID<ur7yP-mI-27@gated-at.bofh.it>
In reply to#1734304
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 <babu.moger@oracle.com>
---
 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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web