Dear Reinhard Meyer,

In message <4ca99373.7020...@emk-elektronik.de> you wrote:
>
> > Patch 1 will change the #if and introduce CONFIG_AT91SAM9_FAMILY
> > instead.
> > 
> > Patch 2 will fix at91/clock.c+timer.c
> 
> Maybe there is a misunderstanding here. I see three patches
> 
> 1. introduce CONFIG_AT91SAM9_FAMILY to common.h
> 
> 2. add an #ifdef CONFIG_AT91SAM9_FAMILY to global_data.h with the
> required variables
> 
> 3. change clock and timer.
> 
> 2+3 combined in one patch? OK

No.  1 + 2 belong together; they deal with the same topic and should
be applied as a single commit.

3 is a completely different story and goes in a separate patch.

> And I bet, this must be a patch series ;)

Not necessarily. The 1+2 combo is independent from 3 (but needed as a
prerequisite, so posting this in a series is fine as well).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Lots of people drink from the wrong bottle sometimes.
        -- Edith Keeler, "The City on the Edge of Forever",
           stardate unknown
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to