mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] Fix duplicates and change default endian for microblaze
@ 2017-09-18 16:53 Babu Moger
  2017-09-18 16:53 ` [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc Babu Moger
  2017-09-18 16:53 ` [PATCH 2/2] arch: change default endian for microblaze Babu Moger
  0 siblings, 2 replies; 6+ messages in thread
From: Babu Moger @ 2017-09-18 16:53 UTC (permalink / raw)
  To: monstr, jejb, deller, davem, akpm
  Cc: sparclinux, linux-kernel, linux-parisc, babu.moger

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(-)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc
  2017-09-18 16:53 [PATCH 0/2] Fix duplicates and change default endian for microblaze Babu Moger
@ 2017-09-18 16:53 ` Babu Moger
  2017-09-18 16:53 ` [PATCH 2/2] arch: change default endian for microblaze Babu Moger
  1 sibling, 0 replies; 6+ messages in thread
From: Babu Moger @ 2017-09-18 16:53 UTC (permalink / raw)
  To: monstr, jejb, deller, davem, akpm
  Cc: sparclinux, linux-kernel, linux-parisc, babu.moger

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/2] arch: change default endian for microblaze
  2017-09-18 16:53 [PATCH 0/2] Fix duplicates and change default endian for microblaze Babu Moger
  2017-09-18 16:53 ` [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc Babu Moger
@ 2017-09-18 16:53 ` Babu Moger
  2017-09-18 17:49   ` Michal Simek
  1 sibling, 1 reply; 6+ messages in thread
From: Babu Moger @ 2017-09-18 16:53 UTC (permalink / raw)
  To: monstr, jejb, deller, davem, akpm
  Cc: sparclinux, linux-kernel, linux-parisc, babu.moger

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] arch: change default endian for microblaze
  2017-09-18 16:53 ` [PATCH 2/2] arch: change default endian for microblaze Babu Moger
@ 2017-09-18 17:49   ` Michal Simek
  2017-09-18 18:05     ` Babu Moger
  0 siblings, 1 reply; 6+ messages in thread
From: Michal Simek @ 2017-09-18 17:49 UTC (permalink / raw)
  To: Babu Moger, jejb, deller, davem, akpm
  Cc: sparclinux, linux-kernel, linux-parisc


[-- Attachment #1.1: Type: text/plain, Size: 1368 bytes --]

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



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] arch: change default endian for microblaze
  2017-09-18 17:49   ` Michal Simek
@ 2017-09-18 18:05     ` Babu Moger
  2017-09-25 13:37       ` Michal Simek
  0 siblings, 1 reply; 6+ messages in thread
From: Babu Moger @ 2017-09-18 18:05 UTC (permalink / raw)
  To: monstr, jejb, deller, davem, akpm; +Cc: sparclinux, linux-kernel, linux-parisc


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
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] arch: change default endian for microblaze
  2017-09-18 18:05     ` Babu Moger
@ 2017-09-25 13:37       ` Michal Simek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2017-09-25 13:37 UTC (permalink / raw)
  To: Babu Moger, jejb, deller, davem, akpm
  Cc: sparclinux, linux-kernel, linux-parisc


[-- Attachment #1.1: Type: text/plain, Size: 1723 bytes --]

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



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-09-25 13:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-18 16:53 [PATCH 0/2] Fix duplicates and change default endian for microblaze Babu Moger
2017-09-18 16:53 ` [PATCH 1/2] arch: Fix duplicates in Kconfig for parisc and sparc Babu Moger
2017-09-18 16:53 ` [PATCH 2/2] arch: change default endian for microblaze Babu Moger
2017-09-18 17:49   ` Michal Simek
2017-09-18 18:05     ` Babu Moger
2017-09-25 13:37       ` Michal Simek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®