* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
@ 2011-11-24 13:09 Holger Brunck
2011-11-25 8:52 ` Marek Vasut
0 siblings, 1 reply; 9+ messages in thread
From: Holger Brunck @ 2011-11-24 13:09 UTC (permalink / raw)
To: u-boot
commit f31a911fe (arm, post: add missing post_time_ms for arm)
enables get_ticks and get_tbclk for all arm based boards,
but kirkwood has currently no implementation for this. So
undefine this for kirkwood boards.
Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
cc: Heiko Schocher <hs@denx.de>
cc: Prafulla Wadaskar <prafulla@marvell.com>
cc: Valentin Longchamp <valentin.longchamp@keymile.com>
---
post/post.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/post/post.c b/post/post.c
index 0e67ad7..422a819 100644
--- a/post/post.c
+++ b/post/post.c
@@ -494,7 +494,7 @@ void post_reloc(void)
*/
unsigned long post_time_ms(unsigned long base)
{
-#if defined(CONFIG_PPC) || defined(CONFIG_ARM)
+#if defined(CONFIG_PPC) || defined(CONFIG_ARM) && !defined(CONFIG_KIRKWOOD)
return (unsigned long)(get_ticks() / (get_tbclk() / CONFIG_SYS_HZ))
- base;
#else
--
1.7.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-24 13:09 [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood Holger Brunck
@ 2011-11-25 8:52 ` Marek Vasut
2011-11-25 9:16 ` Prafulla Wadaskar
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2011-11-25 8:52 UTC (permalink / raw)
To: u-boot
> commit f31a911fe (arm, post: add missing post_time_ms for arm)
> enables get_ticks and get_tbclk for all arm based boards,
> but kirkwood has currently no implementation for this. So
> undefine this for kirkwood boards.
So this means the kirkwood timer doesn't conform to current timer api? Why work
it around and not fix it then?
M
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-25 8:52 ` Marek Vasut
@ 2011-11-25 9:16 ` Prafulla Wadaskar
2011-11-25 9:22 ` Marek Vasut
0 siblings, 1 reply; 9+ messages in thread
From: Prafulla Wadaskar @ 2011-11-25 9:16 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
> On Behalf Of Marek Vasut
> Sent: 25 November 2011 14:22
> To: u-boot at lists.denx.de
> Cc: Valentin at theia.denx.de; Longchamp; Heiko Schocher; Holger Brunck
> Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests on
> kirkwood
>
> > commit f31a911fe (arm, post: add missing post_time_ms for arm)
> > enables get_ticks and get_tbclk for all arm based boards,
> > but kirkwood has currently no implementation for this. So
> > undefine this for kirkwood boards.
>
> So this means the kirkwood timer doesn't conform to current timer api? Why
> work
> it around and not fix it then?
I think this is better approach
Regards..
Prafulla . . .
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-25 9:16 ` Prafulla Wadaskar
@ 2011-11-25 9:22 ` Marek Vasut
2011-11-26 10:29 ` Prafulla Wadaskar
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2011-11-25 9:22 UTC (permalink / raw)
To: u-boot
> > -----Original Message-----
> > From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
> > On Behalf Of Marek Vasut
> > Sent: 25 November 2011 14:22
> > To: u-boot at lists.denx.de
> > Cc: Valentin at theia.denx.de; Longchamp; Heiko Schocher; Holger Brunck
> > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests on
> > kirkwood
> >
> > > commit f31a911fe (arm, post: add missing post_time_ms for arm)
> > > enables get_ticks and get_tbclk for all arm based boards,
> > > but kirkwood has currently no implementation for this. So
> > > undefine this for kirkwood boards.
> >
> > So this means the kirkwood timer doesn't conform to current timer api?
> > Why work
> > it around and not fix it then?
>
> I think this is better approach
>
Right? :) So now the question is ... who's gonna do it ? ;-)
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-25 9:22 ` Marek Vasut
@ 2011-11-26 10:29 ` Prafulla Wadaskar
2011-11-26 11:45 ` Marek Vasut
0 siblings, 1 reply; 9+ messages in thread
From: Prafulla Wadaskar @ 2011-11-26 10:29 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut at gmail.com]
> Sent: 25 November 2011 14:52
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Valentin at theia.denx.de; Longchamp; Heiko
> Schocher; Holger Brunck
> Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests on
> kirkwood
>
> > > -----Original Message-----
> > > From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> bounces at lists.denx.de]
> > > On Behalf Of Marek Vasut
> > > Sent: 25 November 2011 14:22
> > > To: u-boot at lists.denx.de
> > > Cc: Valentin at theia.denx.de; Longchamp; Heiko Schocher; Holger Brunck
> > > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests
> on
> > > kirkwood
> > >
> > > > commit f31a911fe (arm, post: add missing post_time_ms for arm)
> > > > enables get_ticks and get_tbclk for all arm based boards,
> > > > but kirkwood has currently no implementation for this. So
> > > > undefine this for kirkwood boards.
> > >
> > > So this means the kirkwood timer doesn't conform to current timer api?
> > > Why work
> > > it around and not fix it then?
> >
> > I think this is better approach
> >
> Right? :) So now the question is ... who's gonna do it ? ;-)
Patch from any one is welcomed. :-D
Regards..
Prafulla . . .
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-26 10:29 ` Prafulla Wadaskar
@ 2011-11-26 11:45 ` Marek Vasut
2011-11-27 5:56 ` Prafulla Wadaskar
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2011-11-26 11:45 UTC (permalink / raw)
To: u-boot
> > -----Original Message-----
> > From: Marek Vasut [mailto:marek.vasut at gmail.com]
> > Sent: 25 November 2011 14:52
> > To: Prafulla Wadaskar
> > Cc: u-boot at lists.denx.de; Valentin at theia.denx.de; Longchamp; Heiko
> > Schocher; Holger Brunck
> > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests on
> > kirkwood
> >
> > > > -----Original Message-----
> > > > From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> >
> > bounces at lists.denx.de]
> >
> > > > On Behalf Of Marek Vasut
> > > > Sent: 25 November 2011 14:22
> > > > To: u-boot at lists.denx.de
> > > > Cc: Valentin at theia.denx.de; Longchamp; Heiko Schocher; Holger Brunck
> > > > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests
> >
> > on
> >
> > > > kirkwood
> > > >
> > > > > commit f31a911fe (arm, post: add missing post_time_ms for arm)
> > > > > enables get_ticks and get_tbclk for all arm based boards,
> > > > > but kirkwood has currently no implementation for this. So
> > > > > undefine this for kirkwood boards.
> > > >
> > > > So this means the kirkwood timer doesn't conform to current timer
> > > > api? Why work
> > > > it around and not fix it then?
> > >
> > > I think this is better approach
> >
> > Right? :) So now the question is ... who's gonna do it ? ;-)
>
> Patch from any one is welcomed. :-D
Anyone, eh? Anyone ... anyone like the maintainer maybe ? >:-D
M
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-26 11:45 ` Marek Vasut
@ 2011-11-27 5:56 ` Prafulla Wadaskar
2011-11-27 7:07 ` Marek Vasut
0 siblings, 1 reply; 9+ messages in thread
From: Prafulla Wadaskar @ 2011-11-27 5:56 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut at gmail.com]
> Sent: 26 November 2011 17:15
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Valentin at theia.denx.de; Longchamp; Heiko
> Schocher; Holger Brunck
> Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests on
> kirkwood
>
> > > -----Original Message-----
> > > From: Marek Vasut [mailto:marek.vasut at gmail.com]
> > > Sent: 25 November 2011 14:52
> > > To: Prafulla Wadaskar
> > > Cc: u-boot at lists.denx.de; Valentin at theia.denx.de; Longchamp; Heiko
> > > Schocher; Holger Brunck
> > > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests
> on
> > > kirkwood
> > >
> > > > > -----Original Message-----
> > > > > From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> > >
> > > bounces at lists.denx.de]
> > >
> > > > > On Behalf Of Marek Vasut
> > > > > Sent: 25 November 2011 14:22
> > > > > To: u-boot at lists.denx.de
> > > > > Cc: Valentin at theia.denx.de; Longchamp; Heiko Schocher; Holger
> Brunck
> > > > > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post
> tests
> > >
> > > on
> > >
> > > > > kirkwood
> > > > >
> > > > > > commit f31a911fe (arm, post: add missing post_time_ms for arm)
> > > > > > enables get_ticks and get_tbclk for all arm based boards,
> > > > > > but kirkwood has currently no implementation for this. So
> > > > > > undefine this for kirkwood boards.
> > > > >
> > > > > So this means the kirkwood timer doesn't conform to current timer
> > > > > api? Why work
> > > > > it around and not fix it then?
> > > >
> > > > I think this is better approach
> > >
> > > Right? :) So now the question is ... who's gonna do it ? ;-)
> >
> > Patch from any one is welcomed. :-D
>
> Anyone, eh? Anyone ... anyone like the maintainer maybe ? >:-D
I would have loved to do it, but I am very busy with some other very important assignment.
May be I will post a patch in feb2012
Regards..
Prafulla . .
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-27 5:56 ` Prafulla Wadaskar
@ 2011-11-27 7:07 ` Marek Vasut
2011-11-28 8:51 ` Holger Brunck
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2011-11-27 7:07 UTC (permalink / raw)
To: u-boot
> > -----Original Message-----
> > From: Marek Vasut [mailto:marek.vasut at gmail.com]
> > Sent: 26 November 2011 17:15
> > To: Prafulla Wadaskar
> > Cc: u-boot at lists.denx.de; Valentin at theia.denx.de; Longchamp; Heiko
> > Schocher; Holger Brunck
> > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests on
> > kirkwood
> >
> > > > -----Original Message-----
> > > > From: Marek Vasut [mailto:marek.vasut at gmail.com]
> > > > Sent: 25 November 2011 14:52
> > > > To: Prafulla Wadaskar
> > > > Cc: u-boot at lists.denx.de; Valentin at theia.denx.de; Longchamp; Heiko
> > > > Schocher; Holger Brunck
> > > > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post tests
> >
> > on
> >
> > > > kirkwood
> > > >
> > > > > > -----Original Message-----
> > > > > > From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> > > >
> > > > bounces at lists.denx.de]
> > > >
> > > > > > On Behalf Of Marek Vasut
> > > > > > Sent: 25 November 2011 14:22
> > > > > > To: u-boot at lists.denx.de
> > > > > > Cc: Valentin at theia.denx.de; Longchamp; Heiko Schocher; Holger
> >
> > Brunck
> >
> > > > > > Subject: Re: [U-Boot] [PATCH] post: fix compile issue for post
> >
> > tests
> >
> > > > on
> > > >
> > > > > > kirkwood
> > > > > >
> > > > > > > commit f31a911fe (arm, post: add missing post_time_ms for arm)
> > > > > > > enables get_ticks and get_tbclk for all arm based boards,
> > > > > > > but kirkwood has currently no implementation for this. So
> > > > > > > undefine this for kirkwood boards.
> > > > > >
> > > > > > So this means the kirkwood timer doesn't conform to current timer
> > > > > > api? Why work
> > > > > > it around and not fix it then?
> > > > >
> > > > > I think this is better approach
> > > >
> > > > Right? :) So now the question is ... who's gonna do it ? ;-)
> > >
> > > Patch from any one is welcomed. :-D
> >
> > Anyone, eh? Anyone ... anyone like the maintainer maybe ? >:-D
>
> I would have loved to do it, but I am very busy with some other very
> important assignment. May be I will post a patch in feb2012
>
I'll keep an eye on this ;-)
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood
2011-11-27 7:07 ` Marek Vasut
@ 2011-11-28 8:51 ` Holger Brunck
0 siblings, 0 replies; 9+ messages in thread
From: Holger Brunck @ 2011-11-28 8:51 UTC (permalink / raw)
To: u-boot
Hi,
On 11/27/2011 08:07 AM, Marek Vasut wrote:
>>>>>>>
>>>>>>>> commit f31a911fe (arm, post: add missing post_time_ms for arm)
>>>>>>>> enables get_ticks and get_tbclk for all arm based boards,
>>>>>>>> but kirkwood has currently no implementation for this. So
>>>>>>>> undefine this for kirkwood boards.
>>>>>>>
>>>>>>> So this means the kirkwood timer doesn't conform to current timer
>>>>>>> api? Why work
>>>>>>> it around and not fix it then?
>>>>>>
>>>>>> I think this is better approach
>>>>>
>>>>> Right? :) So now the question is ... who's gonna do it ? ;-)
>>>>
>>>> Patch from any one is welcomed. :-D
>>>
>>> Anyone, eh? Anyone ... anyone like the maintainer maybe ? >:-D
>>
>> I would have loved to do it, but I am very busy with some other very
>> important assignment. May be I will post a patch in feb2012
>>
>
> I'll keep an eye on this ;-)
I do also agree that implementing the functions would be better then adding an
#ifdef as my patch does. But I currently got also no time to do this.
So could someone please pick this patch up that the keymile ARM boards are
compilable again?
Best regards
Holger
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2011-11-28 8:51 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-24 13:09 [U-Boot] [PATCH] post: fix compile issue for post tests on kirkwood Holger Brunck
2011-11-25 8:52 ` Marek Vasut
2011-11-25 9:16 ` Prafulla Wadaskar
2011-11-25 9:22 ` Marek Vasut
2011-11-26 10:29 ` Prafulla Wadaskar
2011-11-26 11:45 ` Marek Vasut
2011-11-27 5:56 ` Prafulla Wadaskar
2011-11-27 7:07 ` Marek Vasut
2011-11-28 8:51 ` Holger Brunck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox