[U-Boot] [PATCH 1/2] openrisc/bitops: add hweightX defines

Signed-off-by: Stefan Kristiansson stefan.kristiansson@saunalahti.fi --- arch/openrisc/include/asm/bitops.h | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/openrisc/include/asm/bitops.h b/arch/openrisc/include/asm/bitops.h index c001a5d..c76a409 100644 --- a/arch/openrisc/include/asm/bitops.h +++ b/arch/openrisc/include/asm/bitops.h @@ -25,4 +25,8 @@ #define PLATFORM_FFS #include <asm/bitops/ffs.h>
+#define hweight32(x) generic_hweight32(x) +#define hweight16(x) generic_hweight16(x) +#define hweight8(x) generic_hweight8(x) + #endif /* __ASM_GENERIC_BITOPS_H */

Signed-off-by: Stefan Kristiansson stefan.kristiansson@saunalahti.fi --- arch/openrisc/lib/timer.c | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/arch/openrisc/lib/timer.c b/arch/openrisc/lib/timer.c index 4e92a31..89e644b 100644 --- a/arch/openrisc/lib/timer.c +++ b/arch/openrisc/lib/timer.c @@ -86,6 +86,16 @@ void set_timer(ulong t) timestamp = t; }
+unsigned long long get_ticks(void) +{ + return get_timer(0); +} + +ulong get_tbclk(void) +{ + return CONFIG_SYS_HZ; +} + void __udelay(ulong usec) { ulong elapsed = 0;

On Wed, Feb 22, 2012 at 07:10:10AM -0000, Stefan Kristiansson wrote:
Signed-off-by: Stefan Kristiansson stefan.kristiansson@saunalahti.fi
Applied to u-boot/master, thanks!

On Wed, Feb 22, 2012 at 07:10:09PM +0200, Stefan Kristiansson wrote:
Signed-off-by: Stefan Kristiansson stefan.kristiansson@saunalahti.fi
arch/openrisc/include/asm/bitops.h | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/openrisc/include/asm/bitops.h b/arch/openrisc/include/asm/bitops.h index c001a5d..c76a409 100644 --- a/arch/openrisc/include/asm/bitops.h +++ b/arch/openrisc/include/asm/bitops.h @@ -25,4 +25,8 @@ #define PLATFORM_FFS #include <asm/bitops/ffs.h>
+#define hweight32(x) generic_hweight32(x) +#define hweight16(x) generic_hweight16(x) +#define hweight8(x) generic_hweight8(x)
#endif /* __ASM_GENERIC_BITOPS_H */
1.7.5.4
Sorry, screwed up when I format-patched this, there is no part 2 to it.
Stefan

On Wed, Feb 22, 2012 at 07:10:09PM +0200, Stefan Kristiansson wrote:
Signed-off-by: Stefan Kristiansson stefan.kristiansson@saunalahti.fi
arch/openrisc/include/asm/bitops.h | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/openrisc/include/asm/bitops.h b/arch/openrisc/include/asm/bitops.h index c001a5d..c76a409 100644 --- a/arch/openrisc/include/asm/bitops.h +++ b/arch/openrisc/include/asm/bitops.h @@ -25,4 +25,8 @@
#define PLATFORM_FFS #include <asm/bitops/ffs.h>
+#define hweight32(x) generic_hweight32(x) +#define hweight16(x) generic_hweight16(x) +#define hweight8(x) generic_hweight8(x)
#endif /* __ASM_GENERIC_BITOPS_H */
Sorry, screwed up when I format-patched this, there is no part 2 to it.
Hey,
looks like get_tbclk() and get_ticks() looks all the same just about everywhere. What about introducing a weak alias for these functions so in case weird platform pops up, these can be overridden.
M

On Wednesday 22 February 2012 17:15:50 Marek Vasut wrote:
looks like get_tbclk() and get_ticks() looks all the same just about everywhere. What about introducing a weak alias for these functions so in case weird platform pops up, these can be overridden.
the unified timer core that Graeme/Simon were doing takes care of that implicitly -mike

On Wednesday 22 February 2012 17:15:50 Marek Vasut wrote:
looks like get_tbclk() and get_ticks() looks all the same just about everywhere. What about introducing a weak alias for these functions so in case weird platform pops up, these can be overridden.
the unified timer core that Graeme/Simon were doing takes care of that implicitly
Unified timer core?
M

Ho Marek,
On Thu, Feb 23, 2012 at 10:24 AM, Marek Vasut marek.vasut@gmail.com wrote:
On Wednesday 22 February 2012 17:15:50 Marek Vasut wrote:
looks like get_tbclk() and get_ticks() looks all the same just about everywhere. What about introducing a weak alias for these functions so in case weird platform pops up, these can be overridden.
the unified timer core that Graeme/Simon were doing takes care of that implicitly
Unified timer core?
I don't know what this has to do with openrisc/bitops but a while ago there was a discussion regarding the implementation of new timer API. I did some initial tidy-up work but we kind of got stuck on the finer details. It still in my todo list, but several other items have taken priority
Regards,
Graeme

On Wed, Feb 22, 2012 at 07:10:09AM -0000, Stefan Kristiansson wrote:
Signed-off-by: Stefan Kristiansson stefan.kristiansson@saunalahti.fi
Applied to u-boot/master, thanks!
participants (5)
-
Graeme Russ
-
Marek Vasut
-
Mike Frysinger
-
Stefan Kristiansson
-
Tom Rini