public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Andrei Konovalov <akonovalov@ru•mvista.com>
To: Jakob Viketoft <jakob.viketoft@bitsim•se>
Cc: Jon Masters <jonathan@jonmasters•org>,
	Tom Rini <trini@kernel•crashing.org>,
	Sylvain Munaut <tnt@246tNt•com>,
	Linux PPC Embedded list <linuxppc-embedded@ozlabs•org>
Subject: Re: Flat OF Device Tree for ppc32 [was: Platform bus/ppc sys	model...]
Date: Fri, 22 Apr 2005 21:33:07 +0400	[thread overview]
Message-ID: <42693553.8080902@ru.mvista.com> (raw)
In-Reply-To: <425E3DC6.7030007@bitsim.se>

Hi,

I've tested the Jon's patch with the changes by Jakob on ML300 board
couple days ago (just before 2.6.12-rc3) - works OK.
(got few rejects for some lite5200 files)

Thanks,
Andrei

Jakob Viketoft wrote:
> Has any more happened on this off-list, or it is just in testing-mode 
> right now?
> 
> I applied the changes to a 2.6.12-rc2 tree (with some minor rejects to 
> sort out) and it compiles and boots fine on my Memec FF1152 board with 
> the Virtex-II Pro chip (Xilinx ML300 config).
> 
> A couple of small changes to be able to use the simple boot-loader is 
> below.
> 
> Cheers!
> 
>     /Jakob
> 
> diff -ruN org-nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/embed_config.c 
> nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/embed_config.c
> --- org-nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/embed_config.c 
> 2005-04-04 18:38:03.000000000 +0200
> +++ nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/embed_config.c 2005-04-14 
> 11:36:49.000000000 +0200
> @@ -23,6 +23,8 @@
>  #endif
>  extern unsigned long timebase_period_ns;
> 
> +#include <syslib/fw_bdt.h>
> +
>  /* For those boards that don't provide one.
>  */
>  #if !defined(CONFIG_MBX)
> @@ -776,8 +778,7 @@
>         bd->bi_intfreq = XPAR_CORE_CLOCK_FREQ_HZ;
>         bd->bi_busfreq = XPAR_PLB_CLOCK_FREQ_HZ;
>         bd->bi_pci_busfreq = XPAR_PCI_0_CLOCK_FREQ_HZ;
> -       timebase_period_ns = 1000000000 / bd->bi_tbfreq;
> -       /* see bi_tbfreq definition in 
> arch/ppc/platforms/4xx/xilinx_ml300.h */
> +       timebase_period_ns = 1000000000 / bd->bi_intfreq;
>  }
>  #endif /* CONFIG_XILINX_ML300 */
> 
> diff -ruN org-nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/misc-embedded.c 
> nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/misc-embedded.c
> --- org-nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/misc-embedded.c 
> 2005-04-04 18:38:04.000000000 +0200
> +++ nobd-linux-2.6.12-rc2/arch/ppc/boot/simple/misc-embedded.c 
> 2005-04-14 10:22:38.000000000 +0200
> @@ -22,6 +22,7 @@
>  #endif
> 
>  #include "nonstdio.h"
> +#include <syslib/fw_bdt.h>
> 
>  /* The linker tells us where the image is. */
>  extern char __image_begin, __image_end;
> 
> 
> Jon Loeliger wrote:
> 
>> On Thu, 2005-04-07 at 12:49, Tom Rini wrote:
>>
>>
>>> Please post to the list as an RFC.  Thanks.
>>
>>
>>
>> Folks,
>>
>> Apologies to those who have received this notice twice.
>> I first sent it to the list where it was summarily
>> denied for size reasons.
>>
>> I have now posted a tgz file here instead:
>>
>>     http://www.jdl.com/bdt_cleanup.tgz
>>
>> What follows is my original email content.
>>
>>
>> OK, for those keeping score over on the U-Boot list, _this_
>> message is really intended to be sent to the public list. :-)
>>
>> Here (as noted above) are my diffs against a clone of
>>     http://linux.bkbits.net/linux-2.6
>> as last pulled a couple days ago. (*)
>>
>> Please consider this a Request For Comments for now.  Feel free
>> to apply it and test it out as well, of course. :-)  I'll sign-off
>> on it when it has ripened some more and gets a nod or two...
>>
>> These changes relocate almost all of the ppcboot.h definitions
>> into new files, arc/ppc/syslib/fw_bdt.[ch] and front them
>> with a new, shim interface in include/asm-ppc/firmware.h.
>>
>> There is a Kconfig option that allows you to select between
>> a "bd_t" interface and a non-existant "OF Flat dev tree"
>> interface.  Choose wisely (default) for now.  It adds the
>> new fw_bdt.o file.
>>
>> Some potential gotchas or points to note:
>>
>> - As of this patch, no interface to any board should change.
>>   This just isolates the scattered bd_t references to one place.
>>
>> - On Kumar's advice arch/ppc/boot/simple was totally ignored.
>>   For now; this round.  It can be next on the hit parade easily.
>>   It holds the last reference to ppcboot.h still.
>>
>> - Earlier cut-n-paste of some devices led to me changing a few
>>   files over in 68k land to remove some now-obsolete ppcboot.h
>>   related comments justs for completeness.
>>
>> - I've compiled uImage using defconfigs for the following boards:
>>
>>         8560_ads        cpci690         ash             redwood5
>>         8540_ads        ads_8272        TQM8260         est8260
>>         rpx8260         bseip           mbx             mpc834x_sys
>>         ocotea          lite5200        hdpu            lopec
>>         TQM860L         rpxlite         radstone_ppc7d  walnut
>>
>> - The file include/asm-ppc/ppcboot.h used to contain a nested
>>   include of linux/types.h that was tacitly used by many files.
>>   In ripping ppcboot.h out of many platform .h files, I've had
>>   to add direct #include <linux/types.h> in several places.
>>   If your board isn't listed above and it looks really broken,
>>   consider if it needs a <linux/types.h>?
>>
>>
>> Enjoy,
>> jdl
>>
>> -- 
>> (*) -- BK is dead.  Long live BK!
>>
> 
> 

      parent reply	other threads:[~2005-04-22 17:26 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-30  8:00 Platform bus/ppc sys model Jakob Viketoft
2005-03-30  9:54 ` Sylvain Munaut
2005-03-30 13:52   ` Andrei Konovalov
2005-03-30 15:06     ` Kumar Gala
2005-03-30 16:12       ` Jakob Viketoft
2005-03-30 17:26         ` Kumar Gala
2005-03-31 12:33           ` Jon Masters
2005-03-31 15:55             ` Flat OF Device Tree for ppc32 [was: Platform bus/ppc sys model...] Jon Loeliger
2005-04-04  7:20               ` Jakob Viketoft
2005-04-04  7:31                 ` Jon Masters
2005-04-04 10:56                 ` Andrei Konovalov
2005-04-04 11:01                   ` Jon Masters
2005-04-04 11:08                   ` Jakob Viketoft
2005-04-04 16:45                   ` Wolfgang Denk
2005-04-04 16:58                     ` Andrei Konovalov
2005-04-04 16:56                       ` Jon Masters
2005-04-07 17:20                   ` Tom Rini
2005-04-07 17:35                     ` Jon Loeliger
2005-04-07 17:49                       ` Tom Rini
2005-04-11 15:58                         ` Jon Loeliger
2005-04-14  9:54                           ` Jakob Viketoft
2005-04-15 14:22                             ` Jon Loeliger
2005-04-22 17:33                             ` Andrei Konovalov [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=42693553.8080902@ru.mvista.com \
    --to=akonovalov@ru$(echo .)mvista.com \
    --cc=jakob.viketoft@bitsim$(echo .)se \
    --cc=jonathan@jonmasters$(echo .)org \
    --cc=linuxppc-embedded@ozlabs$(echo .)org \
    --cc=tnt@246tNt$(echo .)com \
    --cc=trini@kernel$(echo .)crashing.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox