Resurrect the unused function to print the imx25 silicon revision at
startup. Add support for revision 1.2.
Martin Kaiser (3):
ARM: imx25: support silicon revision 1.2
clk: imx25: print silicon revision during init
clk: imx25: make __mx25_clocks_init return void
arch/arm/mach-imx/cpu-imx25.c | 2 ++
drivers/clk/imx/clk-imx25.c | 5 +++--
include/soc/imx/revision.h | 1 +
3 files changed, 6 insertions(+), 2 deletions(-)
--
2.30.2
Resurrect the unused function to print the imx25 silicon revision at
startup.
changes in v2:
- drop the patch about silicon revision 1.2. It has already been merged
by Shawn Guo.
- send the changeset to the clk maintainers
Martin Kaiser (2):
clk: imx25: print silicon revision during init
clk: imx25: make __mx25_clocks_init return void
drivers/clk/imx/clk-imx25.c | 5 +++--
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+), 2 deletions(-)
--
2.30.2
Print the imx25 silicon revision when the clocks are initialised.
Use the same mechanism as for imx27, i.e. call mx25_revision.
This function is unused at the moment.
Signed-off-by: Martin Kaiser <[email protected]>
---
changes in v2:
- send the patch to the clk maintainers
drivers/clk/imx/clk-imx25.c | 3 +++
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+)
diff --git a/drivers/clk/imx/clk-imx25.c b/drivers/clk/imx/clk-imx25.c
index 66192fe0a898..263409fca1eb 100644
--- a/drivers/clk/imx/clk-imx25.c
+++ b/drivers/clk/imx/clk-imx25.c
@@ -13,6 +13,7 @@
#include <linux/of.h>
#include <linux/of_address.h>
#include <linux/of_irq.h>
+#include <soc/imx/revision.h>
#include "clk.h"
@@ -220,6 +221,8 @@ static int __init __mx25_clocks_init(void __iomem *ccm_base)
imx_register_uart_clocks(6);
+ imx_print_silicon_rev("i.MX25", mx25_revision());
+
return 0;
}
diff --git a/include/soc/imx/revision.h b/include/soc/imx/revision.h
index b2a55dafaf0a..b122d2fc8881 100644
--- a/include/soc/imx/revision.h
+++ b/include/soc/imx/revision.h
@@ -22,6 +22,7 @@
#define IMX_CHIP_REVISION_3_3 0x33
#define IMX_CHIP_REVISION_UNKNOWN 0xff
+int mx25_revision(void);
int mx27_revision(void);
int mx31_revision(void);
int mx35_revision(void);
--
2.30.2
The __mx25_clocks_init function always returns 0 and its only
caller does not check the return value. Let's remove it.
Signed-off-by: Martin Kaiser <[email protected]>
---
changes in v2:
- send the patch to the clk maintainers
drivers/clk/imx/clk-imx25.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/clk/imx/clk-imx25.c b/drivers/clk/imx/clk-imx25.c
index 263409fca1eb..72b30dd7f05f 100644
--- a/drivers/clk/imx/clk-imx25.c
+++ b/drivers/clk/imx/clk-imx25.c
@@ -74,7 +74,7 @@ enum mx25_clks {
static struct clk *clk[clk_max];
-static int __init __mx25_clocks_init(void __iomem *ccm_base)
+static void __init __mx25_clocks_init(void __iomem *ccm_base)
{
BUG_ON(!ccm_base);
@@ -222,8 +222,6 @@ static int __init __mx25_clocks_init(void __iomem *ccm_base)
imx_register_uart_clocks(6);
imx_print_silicon_rev("i.MX25", mx25_revision());
-
- return 0;
}
static void __init mx25_clocks_init_dt(struct device_node *np)
--
2.30.2
Resurrect the unused function to print the imx25 silicon revision at
startup.
It's been a month since I sent v2. I'm resending the series, hoping to
get a response this time. Thanks in advance.
changes in v3:
- rebase against today's linux-next
changes in v2:
- drop the patch about silicon revision 1.2. It has already been merged
by Shawn Guo.
- send the changeset to the clk maintainers
Martin Kaiser (2):
clk: imx25: print silicon revision during init
clk: imx25: make __mx25_clocks_init return void
drivers/clk/imx/clk-imx25.c | 5 +++--
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+), 2 deletions(-)
--
2.30.2
Resurrect the unused function to print the imx25 silicon revision at
startup.
Now that 6.0-rc1 is out, I hope that the clk maintainers have a moment
to review this simple patch set ;-)
changes in v4:
- rebase against today's linux-next
- add Fabio's Reviewed-by tags
changes in v3:
- rebase against today's linux-next
changes in v2:
- drop the patch about silicon revision 1.2. It has already been merged
by Shawn Guo.
- send the changeset to the clk maintainers
Martin Kaiser (2):
clk: imx25: print silicon revision during init
clk: imx25: make __mx25_clocks_init return void
drivers/clk/imx/clk-imx25.c | 5 +++--
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+), 2 deletions(-)
--
2.30.2
The __mx25_clocks_init function always returns 0 and its only
caller does not check the return value. Let's remove it.
Signed-off-by: Martin Kaiser <[email protected]>
Reviewed-by: Fabio Estevam <[email protected]>
---
changes in v4:
- rebase against today's linux-next
- add Fabio's Reviewed-by tag
changes in v3:
- rebase against today's linux-next
changes in v2:
- send the patch to the clk maintainers
drivers/clk/imx/clk-imx25.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/clk/imx/clk-imx25.c b/drivers/clk/imx/clk-imx25.c
index 263409fca1eb..72b30dd7f05f 100644
--- a/drivers/clk/imx/clk-imx25.c
+++ b/drivers/clk/imx/clk-imx25.c
@@ -74,7 +74,7 @@ enum mx25_clks {
static struct clk *clk[clk_max];
-static int __init __mx25_clocks_init(void __iomem *ccm_base)
+static void __init __mx25_clocks_init(void __iomem *ccm_base)
{
BUG_ON(!ccm_base);
@@ -222,8 +222,6 @@ static int __init __mx25_clocks_init(void __iomem *ccm_base)
imx_register_uart_clocks(6);
imx_print_silicon_rev("i.MX25", mx25_revision());
-
- return 0;
}
static void __init mx25_clocks_init_dt(struct device_node *np)
--
2.30.2
Resurrect the unused function to print the imx25 silicon revision at
startup.
This patchset has been mentioned in a discussion today. Let's bring it to the
attention of the clk maintainers again.
Thanks & best regards,
Martin
changes in v5:
- rebase against today's linux-next
changes in v4:
- rebase against today's linux-next
- add Fabio's Reviewed-by tags
changes in v3:
- rebase against today's linux-next
changes in v2:
- drop the patch about silicon revision 1.2. It has already been merged
by Shawn Guo.
- send the changeset to the clk maintainers
Martin Kaiser (2):
clk: imx25: print silicon revision during init
clk: imx25: make __mx25_clocks_init return void
drivers/clk/imx/clk-imx25.c | 5 +++--
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+), 2 deletions(-)
--
2.30.2
On Wed, May 17, 2023, at 17:39, Martin Kaiser wrote:
> Resurrect the unused function to print the imx25 silicon revision at
> startup.
>
> This patchset has been mentioned in a discussion today. Let's bring it to the
> attention of the clk maintainers again.
Looks good to me, and fixes the warning about the function having
no prototype, so
Acked-by: Arnd Bergmann <[email protected]>
Resurrect the unused function to print the imx25 silicon revision at
startup.
This patch set is blocking Arnd's work on enabling -Werror=missing-prototypes.
I'd really appreciate a response from the clk maintainers if these (trivial)
patches are ok.
Thanks,
Martin
changes in v6:
- rebase against today's linux-next
- add Acked-by from Arnd
changes in v5:
- rebase against today's linux-next
changes in v4:
- rebase against today's linux-next
- add Fabio's Reviewed-by tags
changes in v3:
- rebase against today's linux-next
changes in v2:
- drop the patch about silicon revision 1.2. It has already been merged
by Shawn Guo.
- send the changeset to the clk maintainers
Martin Kaiser (2):
clk: imx25: print silicon revision during init
clk: imx25: make __mx25_clocks_init return void
drivers/clk/imx/clk-imx25.c | 5 +++--
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+), 2 deletions(-)
--
2.39.2
The __mx25_clocks_init function always returns 0 and its only
caller does not check the return value. Let's remove it.
Signed-off-by: Martin Kaiser <[email protected]>
Reviewed-by: Fabio Estevam <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
---
drivers/clk/imx/clk-imx25.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/clk/imx/clk-imx25.c b/drivers/clk/imx/clk-imx25.c
index bee3da2e21e1..c566be848c2d 100644
--- a/drivers/clk/imx/clk-imx25.c
+++ b/drivers/clk/imx/clk-imx25.c
@@ -74,7 +74,7 @@ enum mx25_clks {
static struct clk *clk[clk_max];
-static int __init __mx25_clocks_init(void __iomem *ccm_base)
+static void __init __mx25_clocks_init(void __iomem *ccm_base)
{
BUG_ON(!ccm_base);
@@ -222,8 +222,6 @@ static int __init __mx25_clocks_init(void __iomem *ccm_base)
imx_register_uart_clocks();
imx_print_silicon_rev("i.MX25", mx25_revision());
-
- return 0;
}
static void __init mx25_clocks_init_dt(struct device_node *np)
--
2.39.2
Print the imx25 silicon revision when the clocks are initialised.
Use the same mechanism as for imx27, i.e. call mx25_revision.
This function is unused at the moment.
Signed-off-by: Martin Kaiser <[email protected]>
Reviewed-by: Fabio Estevam <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
---
drivers/clk/imx/clk-imx25.c | 3 +++
include/soc/imx/revision.h | 1 +
2 files changed, 4 insertions(+)
diff --git a/drivers/clk/imx/clk-imx25.c b/drivers/clk/imx/clk-imx25.c
index cc013b343e62..bee3da2e21e1 100644
--- a/drivers/clk/imx/clk-imx25.c
+++ b/drivers/clk/imx/clk-imx25.c
@@ -13,6 +13,7 @@
#include <linux/of.h>
#include <linux/of_address.h>
#include <linux/of_irq.h>
+#include <soc/imx/revision.h>
#include "clk.h"
@@ -220,6 +221,8 @@ static int __init __mx25_clocks_init(void __iomem *ccm_base)
imx_register_uart_clocks();
+ imx_print_silicon_rev("i.MX25", mx25_revision());
+
return 0;
}
diff --git a/include/soc/imx/revision.h b/include/soc/imx/revision.h
index b2a55dafaf0a..b122d2fc8881 100644
--- a/include/soc/imx/revision.h
+++ b/include/soc/imx/revision.h
@@ -22,6 +22,7 @@
#define IMX_CHIP_REVISION_3_3 0x33
#define IMX_CHIP_REVISION_UNKNOWN 0xff
+int mx25_revision(void);
int mx27_revision(void);
int mx31_revision(void);
int mx35_revision(void);
--
2.39.2
Quoting Martin Kaiser (2023-07-30 09:39:26)
> Resurrect the unused function to print the imx25 silicon revision at
> startup.
>
> This patch set is blocking Arnd's work on enabling -Werror=missing-prototypes.
>
> I'd really appreciate a response from the clk maintainers if these (trivial)
> patches are ok.
You need to Cc [email protected] on imx clk patches. I think Abel has
some filter that looks for linux-clk@vger and linux-imx.
Also, please don't send series replies to previous rounds. It buries the
patches deep in my MUA and my scripts don't work well to apply patches
when they're sent like that.
I presume that Abel will pick the patches up? Or do you want to land
them through Arnd's tree? Either way feel free to add my acked-by
Acked-by: Stephen Boyd <[email protected]>
Stephen Boyd ([email protected]) wrote:
> You need to Cc [email protected] on imx clk patches. I think Abel has
> some filter that looks for linux-clk@vger and linux-imx.
> Also, please don't send series replies to previous rounds. It buries the
> patches deep in my MUA and my scripts don't work well to apply patches
> when they're sent like that.
> I presume that Abel will pick the patches up? Or do you want to land
> them through Arnd's tree? Either way feel free to add my acked-by
> Acked-by: Stephen Boyd <[email protected]>
Thanks for responding, Stephen.
Ok, understood. I'll resend with Cc [email protected] and without the
reply to previous versions.
Arnd, are you ok with taking these patches through your tree as
suggested by Stephen?
Thanks,
Martin
On Wed, Aug 02, 2023 at 08:38:49PM +0200, Martin Kaiser wrote:
> Stephen Boyd ([email protected]) wrote:
>
> > You need to Cc [email protected] on imx clk patches. I think Abel has
> > some filter that looks for linux-clk@vger and linux-imx.
>
> > Also, please don't send series replies to previous rounds. It buries the
> > patches deep in my MUA and my scripts don't work well to apply patches
> > when they're sent like that.
>
> > I presume that Abel will pick the patches up? Or do you want to land
> > them through Arnd's tree? Either way feel free to add my acked-by
>
> > Acked-by: Stephen Boyd <[email protected]>
>
> Thanks for responding, Stephen.
>
> Ok, understood. I'll resend with Cc [email protected] and without the
> reply to previous versions.
>
> Arnd, are you ok with taking these patches through your tree as
> suggested by Stephen?
I prefer to have Abel apply the patches through clk tree rather than
arm-soc tree, unless there is a good reason for them to go via the
latter.
Shawn
Shawn Guo ([email protected]) wrote:
> I prefer to have Abel apply the patches through clk tree rather than
> arm-soc tree, unless there is a good reason for them to go via the
> latter.
ok with me. I'd be happy to get this sorted (one way or another) so that
Arnd can enable -Werror=missing-prototypes for 6.6.
Abel, could you pick up these patches?
Thanks,
Martin