public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] show_boot_progress
@ 2014-02-19  9:44 Abdullah YILDIZ
  2014-02-19 10:32 ` Albert ARIBAUD
  0 siblings, 1 reply; 6+ messages in thread
From: Abdullah YILDIZ @ 2014-02-19  9:44 UTC (permalink / raw)
  To: u-boot

I use u-boot version 1.3.3. The function call __show_boot_progress is
defined in common/main.c as

void inline __show_boot_progress (int val) {}

There are many calls to this function in particular locations with
specific parameters. But it does nothing. What is the purpose of this?

Since I'm not a very good programmer, I also wonder the meaning of
void inline show_boot_progress (int val) __attribute__((weak,
alias("__show_boot_progress")));

Is it a directive to the compiler?

abdullah

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

* [U-Boot] show_boot_progress
  2014-02-19  9:44 [U-Boot] show_boot_progress Abdullah YILDIZ
@ 2014-02-19 10:32 ` Albert ARIBAUD
  2014-02-19 13:14   ` Abdullah YILDIZ
  0 siblings, 1 reply; 6+ messages in thread
From: Albert ARIBAUD @ 2014-02-19 10:32 UTC (permalink / raw)
  To: u-boot

Hi Abdullah,

On Wed, 19 Feb 2014 11:44:26 +0200, Abdullah YILDIZ
<abdullah.mails@gmail.com> wrote:

> I use u-boot version 1.3.3.

Any reason to use a five-years-old release?

> The function call __show_boot_progress is
> defined in common/main.c as
> 
> void inline __show_boot_progress (int val) {}
> 
> There are many calls to this function in particular locations with
> specific parameters. But it does nothing. What is the purpose of this?

See below.

> Since I'm not a very good programmer, I also wonder the meaning of
> void inline  (int val) __attribute__((weak,
> alias("__show_boot_progress")));
> 
> Is it a directive to the compiler?

Yes, it is.

It tells the compiler that the function name 'show_boot_progress'
actually refers to function '__show_boot_progress', but that if
someother part of the program defines an actual function called
'show_boot_progress', then that definition is "strong" and will
replace the "weak" definition.

This is a common way to define a generic functionality where some
functions should be adapted on specific architectures or even boards.

Here, the boot progress framework defines a function to call to show
boot progress, but each board will have its own way of presenting that
progress to the board operator, and will implement is own "strong"
show_boot_progress function for this.

> abdullah

Amicalement,
-- 
Albert.

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

* [U-Boot] show_boot_progress
  2014-02-19 10:32 ` Albert ARIBAUD
@ 2014-02-19 13:14   ` Abdullah YILDIZ
  2014-02-19 14:16     ` Albert ARIBAUD
  0 siblings, 1 reply; 6+ messages in thread
From: Abdullah YILDIZ @ 2014-02-19 13:14 UTC (permalink / raw)
  To: u-boot

Hi Albert,

> On Wed, Feb 19, 2014 at 12:32 PM, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
>
> Hi Abdullah,
>
> > I use u-boot version 1.3.3.
>
> Any reason to use a five-years-old release?
>

The development board I use supports u-boot version 1.3.3 and I
couldn't manage to port the functions to newer releases of u-boot.

abdullah

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

* [U-Boot] show_boot_progress
  2014-02-19 13:14   ` Abdullah YILDIZ
@ 2014-02-19 14:16     ` Albert ARIBAUD
  2014-02-19 14:57       ` Abdullah YILDIZ
  0 siblings, 1 reply; 6+ messages in thread
From: Albert ARIBAUD @ 2014-02-19 14:16 UTC (permalink / raw)
  To: u-boot

Hi Abdullah,

On Wed, 19 Feb 2014 15:14:42 +0200, Abdullah YILDIZ
<abdullah.mails@gmail.com> wrote:

> Hi Albert,
> 
> > On Wed, Feb 19, 2014 at 12:32 PM, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
> >
> > Hi Abdullah,
> >
> > > I use u-boot version 1.3.3.
> >
> > Any reason to use a five-years-old release?
> >
> 
> The development board I use supports u-boot version 1.3.3 and I
> couldn't manage to port the functions to newer releases of u-boot.

What board is that?

> abdullah

Amicalement,
-- 
Albert.

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

* [U-Boot] show_boot_progress
  2014-02-19 14:16     ` Albert ARIBAUD
@ 2014-02-19 14:57       ` Abdullah YILDIZ
  2014-02-19 15:37         ` Albert ARIBAUD
  0 siblings, 1 reply; 6+ messages in thread
From: Abdullah YILDIZ @ 2014-02-19 14:57 UTC (permalink / raw)
  To: u-boot

Hi Albert,

>
> What board is that?
>

The board is DevKit3250 from Embest. Actually, there is limited support
of the board in the latest version of u-boot. There is no NAND or
ethernet support, for example.

abdullah


On Wed, Feb 19, 2014 at 4:16 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Abdullah,
>
> On Wed, 19 Feb 2014 15:14:42 +0200, Abdullah YILDIZ
> <abdullah.mails@gmail.com> wrote:
>
>> Hi Albert,
>>
>> > On Wed, Feb 19, 2014 at 12:32 PM, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
>> >
>> > Hi Abdullah,
>> >
>> > > I use u-boot version 1.3.3.
>> >
>> > Any reason to use a five-years-old release?
>> >
>>
>> The development board I use supports u-boot version 1.3.3 and I
>> couldn't manage to port the functions to newer releases of u-boot.
>
> What board is that?
>
>> abdullah
>
> Amicalement,
> --
> Albert.

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

* [U-Boot] show_boot_progress
  2014-02-19 14:57       ` Abdullah YILDIZ
@ 2014-02-19 15:37         ` Albert ARIBAUD
  0 siblings, 0 replies; 6+ messages in thread
From: Albert ARIBAUD @ 2014-02-19 15:37 UTC (permalink / raw)
  To: u-boot

Hi Abdullah,

On Wed, 19 Feb 2014 16:57:37 +0200, Abdullah YILDIZ
<abdullah.mails@gmail.com> wrote:

> Hi Albert,
> 
> >
> > What board is that?
> >
> 
> The board is DevKit3250 from Embest. Actually, there is limited support
> of the board in the latest version of u-boot. There is no NAND or
> ethernet support, for example.

If support for booting to the console is already there, then the most
critical parts of (sup)porting the board are there, and I would suggest
exploring and adding the missing NAND and Ethernet support.
  
> abdullah

Amicalement,
-- 
Albert.

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

end of thread, other threads:[~2014-02-19 15:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-19  9:44 [U-Boot] show_boot_progress Abdullah YILDIZ
2014-02-19 10:32 ` Albert ARIBAUD
2014-02-19 13:14   ` Abdullah YILDIZ
2014-02-19 14:16     ` Albert ARIBAUD
2014-02-19 14:57       ` Abdullah YILDIZ
2014-02-19 15:37         ` Albert ARIBAUD

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