public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] bmips: swapping IO space isn't required
@ 2018-12-09  9:41 Álvaro Fernández Rojas
  2018-12-10 14:50 ` Daniel Schwierzeck
  2018-12-10 18:03 ` Daniel Schwierzeck
  0 siblings, 2 replies; 4+ messages in thread
From: Álvaro Fernández Rojas @ 2018-12-09  9:41 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
---
 arch/mips/mach-bmips/Kconfig | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/arch/mips/mach-bmips/Kconfig b/arch/mips/mach-bmips/Kconfig
index 521996599e..b259a931c9 100644
--- a/arch/mips/mach-bmips/Kconfig
+++ b/arch/mips/mach-bmips/Kconfig
@@ -25,7 +25,6 @@ config SOC_BMIPS_BCM3380
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_WATCHDOG
 	help
 	  This supports BMIPS BCM3380 family.
@@ -36,7 +35,6 @@ config SOC_BMIPS_BCM6318
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6318 family.
@@ -47,7 +45,6 @@ config SOC_BMIPS_BCM6328
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6328 family including BCM63281 and BCM63283.
@@ -58,7 +55,6 @@ config SOC_BMIPS_BCM6338
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6338 family.
@@ -69,7 +65,6 @@ config SOC_BMIPS_BCM6348
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_WATCHDOG
 	help
 	  This supports BMIPS BCM6348 family.
@@ -80,7 +75,6 @@ config SOC_BMIPS_BCM6358
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6358 family including BCM6358 and BCM6359.
@@ -91,7 +85,6 @@ config SOC_BMIPS_BCM6368
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6368 family including BCM6368 and BCM6369.
@@ -102,7 +95,6 @@ config SOC_BMIPS_BCM6362
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6362 family including BCM6361 and BCM6362.
@@ -113,7 +105,6 @@ config SOC_BMIPS_BCM63268
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM63268 family including BCM63168, BCM63169,
@@ -125,7 +116,6 @@ config SOC_BMIPS_BCM6838
 	select MIPS_TUNE_4KC
 	select SUPPORTS_BIG_ENDIAN
 	select SUPPORTS_CPU_MIPS32_R1
-	select SWAP_IO_SPACE
 	select SYSRESET_SYSCON
 	help
 	  This supports BMIPS BCM6838 family including BCM68380, BCM68381,
-- 
2.11.0

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

* [U-Boot] [PATCH] bmips: swapping IO space isn't required
  2018-12-09  9:41 [U-Boot] [PATCH] bmips: swapping IO space isn't required Álvaro Fernández Rojas
@ 2018-12-10 14:50 ` Daniel Schwierzeck
  2018-12-10 16:35   ` Álvaro Fernández Rojas
  2018-12-10 18:03 ` Daniel Schwierzeck
  1 sibling, 1 reply; 4+ messages in thread
From: Daniel Schwierzeck @ 2018-12-10 14:50 UTC (permalink / raw)
  To: u-boot


Am 09.12.18 um 10:41 schrieb Álvaro Fernández Rojas:
> Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
> ---
>  arch/mips/mach-bmips/Kconfig | 10 ----------
>  1 file changed, 10 deletions(-)

Reviewed-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>

I guess this is related to your recent series "serial: bcm6345: switch
to raw I/O functions"? As Tom already applied this series, this patch
should go in 2019.01 too?

> 
> diff --git a/arch/mips/mach-bmips/Kconfig b/arch/mips/mach-bmips/Kconfig
> index 521996599e..b259a931c9 100644
> --- a/arch/mips/mach-bmips/Kconfig
> +++ b/arch/mips/mach-bmips/Kconfig
> @@ -25,7 +25,6 @@ config SOC_BMIPS_BCM3380
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_WATCHDOG
>  	help
>  	  This supports BMIPS BCM3380 family.
> @@ -36,7 +35,6 @@ config SOC_BMIPS_BCM6318
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6318 family.
> @@ -47,7 +45,6 @@ config SOC_BMIPS_BCM6328
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6328 family including BCM63281 and BCM63283.
> @@ -58,7 +55,6 @@ config SOC_BMIPS_BCM6338
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6338 family.
> @@ -69,7 +65,6 @@ config SOC_BMIPS_BCM6348
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_WATCHDOG
>  	help
>  	  This supports BMIPS BCM6348 family.
> @@ -80,7 +75,6 @@ config SOC_BMIPS_BCM6358
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6358 family including BCM6358 and BCM6359.
> @@ -91,7 +85,6 @@ config SOC_BMIPS_BCM6368
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6368 family including BCM6368 and BCM6369.
> @@ -102,7 +95,6 @@ config SOC_BMIPS_BCM6362
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6362 family including BCM6361 and BCM6362.
> @@ -113,7 +105,6 @@ config SOC_BMIPS_BCM63268
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM63268 family including BCM63168, BCM63169,
> @@ -125,7 +116,6 @@ config SOC_BMIPS_BCM6838
>  	select MIPS_TUNE_4KC
>  	select SUPPORTS_BIG_ENDIAN
>  	select SUPPORTS_CPU_MIPS32_R1
> -	select SWAP_IO_SPACE
>  	select SYSRESET_SYSCON
>  	help
>  	  This supports BMIPS BCM6838 family including BCM68380, BCM68381,
> 

-- 
- Daniel

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

* [U-Boot] [PATCH] bmips: swapping IO space isn't required
  2018-12-10 14:50 ` Daniel Schwierzeck
@ 2018-12-10 16:35   ` Álvaro Fernández Rojas
  0 siblings, 0 replies; 4+ messages in thread
From: Álvaro Fernández Rojas @ 2018-12-10 16:35 UTC (permalink / raw)
  To: u-boot

Hi Daniel,

El 10/12/2018 a las 15:50, Daniel Schwierzeck escribió:
> Am 09.12.18 um 10:41 schrieb Álvaro Fernández Rojas:
>> Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
>> ---
>>   arch/mips/mach-bmips/Kconfig | 10 ----------
>>   1 file changed, 10 deletions(-)
> Reviewed-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
>
> I guess this is related to your recent series "serial: bcm6345: switch
> to raw I/O functions"? As Tom already applied this series, this patch
> should go in 2019.01 too?

Yes, I completely forgot I had copied SWAP_IO_SPACE from linux target 
without really needing it.

Without this patch serial driver isn't working for bmips boards, so yeah 
this should go in 2019.01.

>
>> diff --git a/arch/mips/mach-bmips/Kconfig b/arch/mips/mach-bmips/Kconfig
>> index 521996599e..b259a931c9 100644
>> --- a/arch/mips/mach-bmips/Kconfig
>> +++ b/arch/mips/mach-bmips/Kconfig
>> @@ -25,7 +25,6 @@ config SOC_BMIPS_BCM3380
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_WATCHDOG
>>   	help
>>   	  This supports BMIPS BCM3380 family.
>> @@ -36,7 +35,6 @@ config SOC_BMIPS_BCM6318
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6318 family.
>> @@ -47,7 +45,6 @@ config SOC_BMIPS_BCM6328
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6328 family including BCM63281 and BCM63283.
>> @@ -58,7 +55,6 @@ config SOC_BMIPS_BCM6338
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6338 family.
>> @@ -69,7 +65,6 @@ config SOC_BMIPS_BCM6348
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_WATCHDOG
>>   	help
>>   	  This supports BMIPS BCM6348 family.
>> @@ -80,7 +75,6 @@ config SOC_BMIPS_BCM6358
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6358 family including BCM6358 and BCM6359.
>> @@ -91,7 +85,6 @@ config SOC_BMIPS_BCM6368
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6368 family including BCM6368 and BCM6369.
>> @@ -102,7 +95,6 @@ config SOC_BMIPS_BCM6362
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6362 family including BCM6361 and BCM6362.
>> @@ -113,7 +105,6 @@ config SOC_BMIPS_BCM63268
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM63268 family including BCM63168, BCM63169,
>> @@ -125,7 +116,6 @@ config SOC_BMIPS_BCM6838
>>   	select MIPS_TUNE_4KC
>>   	select SUPPORTS_BIG_ENDIAN
>>   	select SUPPORTS_CPU_MIPS32_R1
>> -	select SWAP_IO_SPACE
>>   	select SYSRESET_SYSCON
>>   	help
>>   	  This supports BMIPS BCM6838 family including BCM68380, BCM68381,
>>

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

* [U-Boot] [PATCH] bmips: swapping IO space isn't required
  2018-12-09  9:41 [U-Boot] [PATCH] bmips: swapping IO space isn't required Álvaro Fernández Rojas
  2018-12-10 14:50 ` Daniel Schwierzeck
@ 2018-12-10 18:03 ` Daniel Schwierzeck
  1 sibling, 0 replies; 4+ messages in thread
From: Daniel Schwierzeck @ 2018-12-10 18:03 UTC (permalink / raw)
  To: u-boot



Am 09.12.18 um 10:41 schrieb Álvaro Fernández Rojas:
> Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
> ---
>  arch/mips/mach-bmips/Kconfig | 10 ----------
>  1 file changed, 10 deletions(-)
> 

applied to u-boot-mips/fixes, thanks.

-- 
- Daniel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181210/8a40c690/attachment.sig>

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

end of thread, other threads:[~2018-12-10 18:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-09  9:41 [U-Boot] [PATCH] bmips: swapping IO space isn't required Álvaro Fernández Rojas
2018-12-10 14:50 ` Daniel Schwierzeck
2018-12-10 16:35   ` Álvaro Fernández Rojas
2018-12-10 18:03 ` Daniel Schwierzeck

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