public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [PATCH 1/2] mx7ulp: clock: Remove unuseful information
@ 2020-10-07 11:09 Fabio Estevam
  2020-10-07 11:09 ` [PATCH 2/2] mx7ulp: clock: Align the PLL_USB frequency Fabio Estevam
  2020-11-02  8:09 ` [PATCH 1/2] mx7ulp: clock: Remove unuseful information sbabic at denx.de
  0 siblings, 2 replies; 4+ messages in thread
From: Fabio Estevam @ 2020-10-07 11:09 UTC (permalink / raw)
  To: u-boot

The command 'clocks' shows the following output:

=> clocks
PLL_A7_SPLL         528 MHz
PLL_A7_APLL         529 MHz
PLL_USB           0 MHz

....

[do_mx7_showclocks] addr = 0x9FFB61F1

The last line is not useful at all, so just remove it.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/mach-imx/mx7ulp/clock.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/arch/arm/mach-imx/mx7ulp/clock.c b/arch/arm/mach-imx/mx7ulp/clock.c
index a987ff22df..1a504d0b9a 100644
--- a/arch/arm/mach-imx/mx7ulp/clock.c
+++ b/arch/arm/mach-imx/mx7ulp/clock.c
@@ -333,7 +333,7 @@ void hab_caam_clock_enable(unsigned char enable)
 int do_mx7_showclocks(struct cmd_tbl *cmdtp, int flag, int argc,
 		      char *const argv[])
 {
-	u32 addr = 0;
+
 	u32 freq;
 	freq = decode_pll(PLL_A7_SPLL);
 	printf("PLL_A7_SPLL    %8d MHz\n", freq / 1000000);
@@ -356,8 +356,6 @@ int do_mx7_showclocks(struct cmd_tbl *cmdtp, int flag, int argc,
 	printf("USDHC2     %8d kHz\n", mxc_get_clock(MXC_ESDHC2_CLK) / 1000);
 	printf("I2C4       %8d kHz\n", mxc_get_clock(MXC_I2C_CLK) / 1000);
 
-	addr = (u32) clock_init;
-	printf("[%s] addr = 0x%08X\r\n", __func__, addr);
 	scg_a7_info();
 
 	return 0;
-- 
2.17.1

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

* [PATCH 2/2] mx7ulp: clock: Align the PLL_USB frequency
  2020-10-07 11:09 [PATCH 1/2] mx7ulp: clock: Remove unuseful information Fabio Estevam
@ 2020-10-07 11:09 ` Fabio Estevam
  2020-11-02  8:08   ` sbabic at denx.de
  2020-11-02  8:09 ` [PATCH 1/2] mx7ulp: clock: Remove unuseful information sbabic at denx.de
  1 sibling, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2020-10-07 11:09 UTC (permalink / raw)
  To: u-boot

The command 'clocks' shows the following output:

=> clocks
PLL_A7_SPLL         528 MHz
PLL_A7_APLL         529 MHz
PLL_USB           0 MHz

Add some extra spaces so that the PLL_USB information gets aligned with
the previous reported frequencies.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/mach-imx/mx7ulp/clock.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-imx/mx7ulp/clock.c b/arch/arm/mach-imx/mx7ulp/clock.c
index 1a504d0b9a..51aaa5001e 100644
--- a/arch/arm/mach-imx/mx7ulp/clock.c
+++ b/arch/arm/mach-imx/mx7ulp/clock.c
@@ -342,7 +342,7 @@ int do_mx7_showclocks(struct cmd_tbl *cmdtp, int flag, int argc,
 	printf("PLL_A7_APLL    %8d MHz\n", freq / 1000000);
 
 	freq = decode_pll(PLL_USB);
-	printf("PLL_USB    %8d MHz\n", freq / 1000000);
+	printf("PLL_USB        %8d MHz\n", freq / 1000000);
 
 	printf("\n");
 
-- 
2.17.1

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

* [PATCH 2/2] mx7ulp: clock: Align the PLL_USB frequency
  2020-10-07 11:09 ` [PATCH 2/2] mx7ulp: clock: Align the PLL_USB frequency Fabio Estevam
@ 2020-11-02  8:08   ` sbabic at denx.de
  0 siblings, 0 replies; 4+ messages in thread
From: sbabic at denx.de @ 2020-11-02  8:08 UTC (permalink / raw)
  To: u-boot

> The command 'clocks' shows the following output:
> => clocks
> PLL_A7_SPLL         528 MHz
> PLL_A7_APLL         529 MHz
> PLL_USB           0 MHz
> Add some extra spaces so that the PLL_USB information gets aligned with
> the previous reported frequencies.
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

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

* [PATCH 1/2] mx7ulp: clock: Remove unuseful information
  2020-10-07 11:09 [PATCH 1/2] mx7ulp: clock: Remove unuseful information Fabio Estevam
  2020-10-07 11:09 ` [PATCH 2/2] mx7ulp: clock: Align the PLL_USB frequency Fabio Estevam
@ 2020-11-02  8:09 ` sbabic at denx.de
  1 sibling, 0 replies; 4+ messages in thread
From: sbabic at denx.de @ 2020-11-02  8:09 UTC (permalink / raw)
  To: u-boot

> The command 'clocks' shows the following output:
> => clocks
> PLL_A7_SPLL         528 MHz
> PLL_A7_APLL         529 MHz
> PLL_USB           0 MHz
> ....
> [do_mx7_showclocks] addr = 0x9FFB61F1
> The last line is not useful at all, so just remove it.
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

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

end of thread, other threads:[~2020-11-02  8:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-10-07 11:09 [PATCH 1/2] mx7ulp: clock: Remove unuseful information Fabio Estevam
2020-10-07 11:09 ` [PATCH 2/2] mx7ulp: clock: Align the PLL_USB frequency Fabio Estevam
2020-11-02  8:08   ` sbabic at denx.de
2020-11-02  8:09 ` [PATCH 1/2] mx7ulp: clock: Remove unuseful information sbabic at denx.de

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