u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [PATCH] serial: a37xx: Remove CONFIG_DEBUG_UART_SHIFT options
@ 2021-08-13 11:58 Pali Rohár
  2021-08-16  6:59 ` Stefan Roese
  2021-09-01  9:10 ` Stefan Roese
  0 siblings, 2 replies; 3+ messages in thread
From: Pali Rohár @ 2021-08-13 11:58 UTC (permalink / raw)
  To: Stefan Roese, Marek Behún; +Cc: u-boot

Armada 37xx serial driver does not use CONFIG_DEBUG_UART_SHIFT.

So do not define any bogus value for CONFIG_DEBUG_UART_SHIFT option in any
Armada 37xx defconfig file.

Signed-off-by: Pali Rohár <pali@kernel.org>
---
 configs/mvebu_db-88f3720_defconfig          | 1 -
 configs/mvebu_espressobin-88f3720_defconfig | 1 -
 configs/turris_mox_defconfig                | 1 -
 configs/uDPU_defconfig                      | 1 -
 4 files changed, 4 deletions(-)

diff --git a/configs/mvebu_db-88f3720_defconfig b/configs/mvebu_db-88f3720_defconfig
index bc92fdb8ee8c..eb50afc0f381 100644
--- a/configs/mvebu_db-88f3720_defconfig
+++ b/configs/mvebu_db-88f3720_defconfig
@@ -63,7 +63,6 @@ CONFIG_PHY=y
 CONFIG_MVEBU_COMPHY_SUPPORT=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_ARMADA_37XX=y
-CONFIG_DEBUG_UART_SHIFT=2
 CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_MVEBU_A3700_UART=y
 CONFIG_MVEBU_A3700_SPI=y
diff --git a/configs/mvebu_espressobin-88f3720_defconfig b/configs/mvebu_espressobin-88f3720_defconfig
index 4003a25346ce..9641c02d9317 100644
--- a/configs/mvebu_espressobin-88f3720_defconfig
+++ b/configs/mvebu_espressobin-88f3720_defconfig
@@ -79,7 +79,6 @@ CONFIG_MVEBU_COMPHY_SUPPORT=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_ARMADA_37XX=y
 CONFIG_DM_REGULATOR_GPIO=y
-CONFIG_DEBUG_UART_SHIFT=2
 CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_MVEBU_A3700_UART=y
 CONFIG_MVEBU_A3700_SPI=y
diff --git a/configs/turris_mox_defconfig b/configs/turris_mox_defconfig
index fc5e1fc3766f..40f975ead314 100644
--- a/configs/turris_mox_defconfig
+++ b/configs/turris_mox_defconfig
@@ -87,7 +87,6 @@ CONFIG_DM_RTC=y
 CONFIG_RTC_DS1307=y
 CONFIG_SCSI=y
 CONFIG_DM_SCSI=y
-CONFIG_DEBUG_UART_SHIFT=2
 CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_MVEBU_A3700_UART=y
 CONFIG_MVEBU_A3700_SPI=y
diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
index 3e6bb32cb88b..1ea3aad5ff2a 100644
--- a/configs/uDPU_defconfig
+++ b/configs/uDPU_defconfig
@@ -79,7 +79,6 @@ CONFIG_PINCTRL=y
 CONFIG_PINCTRL_ARMADA_37XX=y
 CONFIG_DM_REGULATOR_FIXED=y
 CONFIG_DM_REGULATOR_GPIO=y
-CONFIG_DEBUG_UART_SHIFT=2
 CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_MVEBU_A3700_UART=y
 CONFIG_MVEBU_A3700_SPI=y
-- 
2.20.1


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

* Re: [PATCH] serial: a37xx: Remove CONFIG_DEBUG_UART_SHIFT options
  2021-08-13 11:58 [PATCH] serial: a37xx: Remove CONFIG_DEBUG_UART_SHIFT options Pali Rohár
@ 2021-08-16  6:59 ` Stefan Roese
  2021-09-01  9:10 ` Stefan Roese
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2021-08-16  6:59 UTC (permalink / raw)
  To: Pali Rohár, Marek Behún; +Cc: u-boot

On 13.08.21 13:58, Pali Rohár wrote:
> Armada 37xx serial driver does not use CONFIG_DEBUG_UART_SHIFT.
> 
> So do not define any bogus value for CONFIG_DEBUG_UART_SHIFT option in any
> Armada 37xx defconfig file.
> 
> Signed-off-by: Pali Rohár <pali@kernel.org>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

> ---
>   configs/mvebu_db-88f3720_defconfig          | 1 -
>   configs/mvebu_espressobin-88f3720_defconfig | 1 -
>   configs/turris_mox_defconfig                | 1 -
>   configs/uDPU_defconfig                      | 1 -
>   4 files changed, 4 deletions(-)
> 
> diff --git a/configs/mvebu_db-88f3720_defconfig b/configs/mvebu_db-88f3720_defconfig
> index bc92fdb8ee8c..eb50afc0f381 100644
> --- a/configs/mvebu_db-88f3720_defconfig
> +++ b/configs/mvebu_db-88f3720_defconfig
> @@ -63,7 +63,6 @@ CONFIG_PHY=y
>   CONFIG_MVEBU_COMPHY_SUPPORT=y
>   CONFIG_PINCTRL=y
>   CONFIG_PINCTRL_ARMADA_37XX=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> diff --git a/configs/mvebu_espressobin-88f3720_defconfig b/configs/mvebu_espressobin-88f3720_defconfig
> index 4003a25346ce..9641c02d9317 100644
> --- a/configs/mvebu_espressobin-88f3720_defconfig
> +++ b/configs/mvebu_espressobin-88f3720_defconfig
> @@ -79,7 +79,6 @@ CONFIG_MVEBU_COMPHY_SUPPORT=y
>   CONFIG_PINCTRL=y
>   CONFIG_PINCTRL_ARMADA_37XX=y
>   CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> diff --git a/configs/turris_mox_defconfig b/configs/turris_mox_defconfig
> index fc5e1fc3766f..40f975ead314 100644
> --- a/configs/turris_mox_defconfig
> +++ b/configs/turris_mox_defconfig
> @@ -87,7 +87,6 @@ CONFIG_DM_RTC=y
>   CONFIG_RTC_DS1307=y
>   CONFIG_SCSI=y
>   CONFIG_DM_SCSI=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
> index 3e6bb32cb88b..1ea3aad5ff2a 100644
> --- a/configs/uDPU_defconfig
> +++ b/configs/uDPU_defconfig
> @@ -79,7 +79,6 @@ CONFIG_PINCTRL=y
>   CONFIG_PINCTRL_ARMADA_37XX=y
>   CONFIG_DM_REGULATOR_FIXED=y
>   CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> 


Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de

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

* Re: [PATCH] serial: a37xx: Remove CONFIG_DEBUG_UART_SHIFT options
  2021-08-13 11:58 [PATCH] serial: a37xx: Remove CONFIG_DEBUG_UART_SHIFT options Pali Rohár
  2021-08-16  6:59 ` Stefan Roese
@ 2021-09-01  9:10 ` Stefan Roese
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2021-09-01  9:10 UTC (permalink / raw)
  To: Pali Rohár, Marek Behún; +Cc: u-boot

On 13.08.21 13:58, Pali Rohár wrote:
> Armada 37xx serial driver does not use CONFIG_DEBUG_UART_SHIFT.
> 
> So do not define any bogus value for CONFIG_DEBUG_UART_SHIFT option in any
> Armada 37xx defconfig file.
> 
> Signed-off-by: Pali Rohár <pali@kernel.org>

Applied to u-boot-marvell/master

Thanks,
Stefan

> ---
>   configs/mvebu_db-88f3720_defconfig          | 1 -
>   configs/mvebu_espressobin-88f3720_defconfig | 1 -
>   configs/turris_mox_defconfig                | 1 -
>   configs/uDPU_defconfig                      | 1 -
>   4 files changed, 4 deletions(-)
> 
> diff --git a/configs/mvebu_db-88f3720_defconfig b/configs/mvebu_db-88f3720_defconfig
> index bc92fdb8ee8c..eb50afc0f381 100644
> --- a/configs/mvebu_db-88f3720_defconfig
> +++ b/configs/mvebu_db-88f3720_defconfig
> @@ -63,7 +63,6 @@ CONFIG_PHY=y
>   CONFIG_MVEBU_COMPHY_SUPPORT=y
>   CONFIG_PINCTRL=y
>   CONFIG_PINCTRL_ARMADA_37XX=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> diff --git a/configs/mvebu_espressobin-88f3720_defconfig b/configs/mvebu_espressobin-88f3720_defconfig
> index 4003a25346ce..9641c02d9317 100644
> --- a/configs/mvebu_espressobin-88f3720_defconfig
> +++ b/configs/mvebu_espressobin-88f3720_defconfig
> @@ -79,7 +79,6 @@ CONFIG_MVEBU_COMPHY_SUPPORT=y
>   CONFIG_PINCTRL=y
>   CONFIG_PINCTRL_ARMADA_37XX=y
>   CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> diff --git a/configs/turris_mox_defconfig b/configs/turris_mox_defconfig
> index fc5e1fc3766f..40f975ead314 100644
> --- a/configs/turris_mox_defconfig
> +++ b/configs/turris_mox_defconfig
> @@ -87,7 +87,6 @@ CONFIG_DM_RTC=y
>   CONFIG_RTC_DS1307=y
>   CONFIG_SCSI=y
>   CONFIG_DM_SCSI=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig
> index 3e6bb32cb88b..1ea3aad5ff2a 100644
> --- a/configs/uDPU_defconfig
> +++ b/configs/uDPU_defconfig
> @@ -79,7 +79,6 @@ CONFIG_PINCTRL=y
>   CONFIG_PINCTRL_ARMADA_37XX=y
>   CONFIG_DM_REGULATOR_FIXED=y
>   CONFIG_DM_REGULATOR_GPIO=y
> -CONFIG_DEBUG_UART_SHIFT=2
>   CONFIG_DEBUG_UART_ANNOUNCE=y
>   CONFIG_MVEBU_A3700_UART=y
>   CONFIG_MVEBU_A3700_SPI=y
> 


Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de

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

end of thread, other threads:[~2021-09-01  9:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-13 11:58 [PATCH] serial: a37xx: Remove CONFIG_DEBUG_UART_SHIFT options Pali Rohár
2021-08-16  6:59 ` Stefan Roese
2021-09-01  9:10 ` Stefan Roese

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).