Hi Rasmus,
On Fri, 28 Feb 2020 at 16:09, Rasmus Villemoes
wrote:
>
> On 28/02/2020 18.35, Tom Rini wrote:
> > On Fri, Feb 28, 2020 at 05:24:58PM +, Rasmus Villemoes wrote:
>
> >> eliminated, and there's not an #ifdef in sight.
> >
> > That sounds pretty nice actually. If you're so inclined
On 28/02/2020 18.35, Tom Rini wrote:
> On Fri, Feb 28, 2020 at 05:24:58PM +, Rasmus Villemoes wrote:
>> eliminated, and there's not an #ifdef in sight.
>
> That sounds pretty nice actually. If you're so inclined I'd like to see
> it.
>
So I started looking at that, and while it's mostly
On Fri, Feb 28, 2020 at 05:24:58PM +, Rasmus Villemoes wrote:
> On 28/02/2020 16.46, Tom Rini wrote:
> > On Fri, Feb 28, 2020 at 08:42:21AM +, Rasmus Villemoes wrote:
> >> On 28/02/2020 00.40, Simon Glass wrote:
>
> >>> Using if() is preferable to #if if there is no cost.
> >>
> >>
On 28/02/2020 16.46, Tom Rini wrote:
> On Fri, Feb 28, 2020 at 08:42:21AM +, Rasmus Villemoes wrote:
>> On 28/02/2020 00.40, Simon Glass wrote:
>>> Using if() is preferable to #if if there is no cost.
>>
>> Completely agree, and I also prefer to have the linker eliminate unused
>> functions
On Fri, Feb 28, 2020 at 08:42:21AM +, Rasmus Villemoes wrote:
> On 28/02/2020 00.40, Simon Glass wrote:
> > Hi Rasmus,
> >
> > On Thu, 27 Feb 2020 at 00:18, Rasmus Villemoes
> > wrote:
> >>
> >> Some init functions, e.g. print_resetinfo(), are conditionally defined
> >> depending on some
On 28/02/2020 00.40, Simon Glass wrote:
> Hi Rasmus,
>
> On Thu, 27 Feb 2020 at 00:18, Rasmus Villemoes
> wrote:
>>
>> Some init functions, e.g. print_resetinfo(), are conditionally defined
>> depending on some config options, and are correspondingly
>> conditionally included in the
Hi Rasmus,
On Thu, 27 Feb 2020 at 00:18, Rasmus Villemoes
wrote:
>
> Some init functions, e.g. print_resetinfo(), are conditionally defined
> depending on some config options, and are correspondingly
> conditionally included in the init_sequence_f[] array.
>
> Others are unconditionally defined
Some init functions, e.g. print_resetinfo(), are conditionally defined
depending on some config options, and are correspondingly
conditionally included in the init_sequence_f[] array.
Others are unconditionally defined and included in init_sequence_f[],
but have their entire body, sans a
8 matches
Mail list logo