* [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards
@ 2011-10-31 10:07 Valentin Longchamp
2011-10-31 10:48 ` Igor Grinberg
0 siblings, 1 reply; 5+ messages in thread
From: Valentin Longchamp @ 2011-10-31 10:07 UTC (permalink / raw)
To: u-boot
As this is a mandatory CONFIG for ARM boards now.
Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
---
include/configs/km/km_arm.h | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/include/configs/km/km_arm.h b/include/configs/km/km_arm.h
index eaf6017..f30088d 100644
--- a/include/configs/km/km_arm.h
+++ b/include/configs/km/km_arm.h
@@ -48,6 +48,8 @@
#define CONFIG_KW88F6281 /* SOC Name */
#define CONFIG_MACH_KM_KIRKWOOD /* Machine type */
+#define CONFIG_MACH_TYPE MACH_TYPE_KM_KIRKWOOD
+
/* include common defines/options for all Keymile boards */
#include "keymile-common.h"
--
1.7.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards
2011-10-31 10:07 [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards Valentin Longchamp
@ 2011-10-31 10:48 ` Igor Grinberg
2011-10-31 11:50 ` Prafulla Wadaskar
2011-10-31 18:28 ` Mike Frysinger
0 siblings, 2 replies; 5+ messages in thread
From: Igor Grinberg @ 2011-10-31 10:48 UTC (permalink / raw)
To: u-boot
Hi Valentin,
On 10/31/11 12:07, Valentin Longchamp wrote:
> As this is a mandatory CONFIG for ARM boards now.
>
> Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
> Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
> ---
> include/configs/km/km_arm.h | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/include/configs/km/km_arm.h b/include/configs/km/km_arm.h
> index eaf6017..f30088d 100644
> --- a/include/configs/km/km_arm.h
> +++ b/include/configs/km/km_arm.h
> @@ -48,6 +48,8 @@
> #define CONFIG_KW88F6281 /* SOC Name */
> #define CONFIG_MACH_KM_KIRKWOOD /* Machine type */
>
> +#define CONFIG_MACH_TYPE MACH_TYPE_KM_KIRKWOOD
> +
Probably, you also want to remove the gd->bd->bi_arch_number
assignment from board files using this mach id?
--
Regards,
Igor.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards
2011-10-31 10:48 ` Igor Grinberg
@ 2011-10-31 11:50 ` Prafulla Wadaskar
2011-10-31 18:28 ` Mike Frysinger
1 sibling, 0 replies; 5+ messages in thread
From: Prafulla Wadaskar @ 2011-10-31 11:50 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 Igor Grinberg
> Sent: Monday, October 31, 2011 4:18 PM
> To: Valentin Longchamp
> Cc: u-boot at lists.denx.de; Holger Brunck
> Subject: Re: [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for
> all keymile boards
>
> Hi Valentin,
>
> On 10/31/11 12:07, Valentin Longchamp wrote:
> > As this is a mandatory CONFIG for ARM boards now.
> >
> > Signed-off-by: Valentin Longchamp
> <valentin.longchamp@keymile.com>
> > Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
> > ---
> > include/configs/km/km_arm.h | 2 ++
> > 1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/include/configs/km/km_arm.h
> b/include/configs/km/km_arm.h
> > index eaf6017..f30088d 100644
> > --- a/include/configs/km/km_arm.h
> > +++ b/include/configs/km/km_arm.h
> > @@ -48,6 +48,8 @@
> > #define CONFIG_KW88F6281 /* SOC Name */
> > #define CONFIG_MACH_KM_KIRKWOOD /* Machine type */
> >
> > +#define CONFIG_MACH_TYPE MACH_TYPE_KM_KIRKWOOD
> > +
>
> Probably, you also want to remove the gd->bd->bi_arch_number
> assignment from board files using this mach id?
Hi Valentin
Please post one patch for all this
(including above, MACH_TYPE_KM_KIRKDOOD definition and CONFIG_MACH_TYPE definition)
Regards...
Prafulla . . .
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards
2011-10-31 10:48 ` Igor Grinberg
2011-10-31 11:50 ` Prafulla Wadaskar
@ 2011-10-31 18:28 ` Mike Frysinger
2011-11-01 11:48 ` Igor Grinberg
1 sibling, 1 reply; 5+ messages in thread
From: Mike Frysinger @ 2011-10-31 18:28 UTC (permalink / raw)
To: u-boot
On Monday 31 October 2011 06:48:06 Igor Grinberg wrote:
> On 10/31/11 12:07, Valentin Longchamp wrote:
> > --- a/include/configs/km/km_arm.h
> > +++ b/include/configs/km/km_arm.h
> > @@ -48,6 +48,8 @@
> >
> > #define CONFIG_KW88F6281 /* SOC Name */
> > #define CONFIG_MACH_KM_KIRKWOOD /* Machine type */
> >
> > +#define CONFIG_MACH_TYPE MACH_TYPE_KM_KIRKWOOD
> > +
>
> Probably, you also want to remove the gd->bd->bi_arch_number
> assignment from board files using this mach id?
if possible, i'd like to punt bi_arch_number from everyone ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111031/6bc9bf6c/attachment.pgp
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards
2011-10-31 18:28 ` Mike Frysinger
@ 2011-11-01 11:48 ` Igor Grinberg
0 siblings, 0 replies; 5+ messages in thread
From: Igor Grinberg @ 2011-11-01 11:48 UTC (permalink / raw)
To: u-boot
On 10/31/11 20:28, Mike Frysinger wrote:
> On Monday 31 October 2011 06:48:06 Igor Grinberg wrote:
>> On 10/31/11 12:07, Valentin Longchamp wrote:
>>> --- a/include/configs/km/km_arm.h
>>> +++ b/include/configs/km/km_arm.h
>>> @@ -48,6 +48,8 @@
>>>
>>> #define CONFIG_KW88F6281 /* SOC Name */
>>> #define CONFIG_MACH_KM_KIRKWOOD /* Machine type */
>>>
>>> +#define CONFIG_MACH_TYPE MACH_TYPE_KM_KIRKWOOD
>>> +
>>
>> Probably, you also want to remove the gd->bd->bi_arch_number
>> assignment from board files using this mach id?
>
> if possible, i'd like to punt bi_arch_number from everyone ...
Apart from those who have the mach id runtime discoverable,
it would be great idea, but we cannot request this from
someone who fixes his/her boards.
Patches are welcome ;)
--
Regards,
Igor.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-11-01 11:48 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-31 10:07 [U-Boot] [PATCH] ARM: define CONFIG_MACH_TYPE for all keymile boards Valentin Longchamp
2011-10-31 10:48 ` Igor Grinberg
2011-10-31 11:50 ` Prafulla Wadaskar
2011-10-31 18:28 ` Mike Frysinger
2011-11-01 11:48 ` Igor Grinberg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox