All of lore.kernel.org
 help / color / mirror / Atom feed
From: Florian Fainelli <f.fainelli@gmail.com>
To: Jonas Gorski <jogo@openwrt.org>, linux-mips@linux-mips.org
Cc: devicetree@vger.kernel.org, Ralf Baechle <ralf@linux-mips.org>,
	John Crispin <blogic@openwrt.org>,
	Kevin Cernekee <cernekee@gmail.com>,
	Aaro Koskinen <aaro.koskinen@iki.fi>,
	Markos Chandras <markos.chandras@imgtec.com>,
	Andrew Bresticker <abrestic@chromium.org>,
	Daniel Schwierzeck <daniel.schwierzeck@gmail.com>,
	Paul Burton <paul.burton@imgtec.com>,
	James Hartley <James.Hartley@imgtec.com>
Subject: Re: [PATCH RFC v3 4/4] MIPS: BMIPS: accept UHI interface for passing a dtb
Date: Fri, 17 Apr 2015 20:09:35 -0700	[thread overview]
Message-ID: <5531CAEF.9090606@gmail.com> (raw)
In-Reply-To: <1428834301-12721-5-git-send-email-jogo@openwrt.org>

Le 12/04/2015 03:25, Jonas Gorski a écrit :
> Detect and use passed dtb address using the UHI interface. This allows for
> booting with a vmlinux.bin appended dtb instead of using a built-in one.

No particular objections to this change, current platforms support by
this change fall under 3 categories:

- BCM7xxx using a CFE providing a firmware interface you call into using
code from arch/mips/fw/cfe/
- BCM63xx CFE
- BCM33xx using Aeolus [1]

The first two are likely not going to be changed since these are
currently deployed products/systems, however the latter could be changed
to match UHI as Kevin proposed a while ago.

> 
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>
> ---
>  arch/mips/bmips/setup.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/mips/bmips/setup.c b/arch/mips/bmips/setup.c
> index fae800e..526ec27 100644
> --- a/arch/mips/bmips/setup.c
> +++ b/arch/mips/bmips/setup.c
> @@ -149,6 +149,8 @@ void __init plat_mem_setup(void)
>  	/* intended to somewhat resemble ARM; see Documentation/arm/Booting */
>  	if (fw_arg0 == 0 && fw_arg1 == 0xffffffff)
>  		dtb = phys_to_virt(fw_arg2);
> +	else if (fw_arg0 == -2) /* UHI interface */
> +		dtb = (void *)fw_arg1;
>  	else if (__dtb_start != __dtb_end)
>  		dtb = (void *)__dtb_start;
>  	else
> 

WARNING: multiple messages have this Message-ID (diff)
From: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Jonas Gorski <jogo-p3rKhJxN3npAfugRpC6u6w@public.gmane.org>,
	linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Ralf Baechle <ralf-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org>,
	John Crispin <blogic-p3rKhJxN3npAfugRpC6u6w@public.gmane.org>,
	Kevin Cernekee <cernekee-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Aaro Koskinen <aaro.koskinen-X3B1VOXEql0@public.gmane.org>,
	Markos Chandras
	<markos.chandras-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>,
	Andrew Bresticker
	<abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	Daniel Schwierzeck
	<daniel.schwierzeck-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Paul Burton <paul.burton-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>,
	James Hartley
	<James.Hartley-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH RFC v3 4/4] MIPS: BMIPS: accept UHI interface for passing a dtb
Date: Fri, 17 Apr 2015 20:09:35 -0700	[thread overview]
Message-ID: <5531CAEF.9090606@gmail.com> (raw)
In-Reply-To: <1428834301-12721-5-git-send-email-jogo-p3rKhJxN3npAfugRpC6u6w@public.gmane.org>

Le 12/04/2015 03:25, Jonas Gorski a écrit :
> Detect and use passed dtb address using the UHI interface. This allows for
> booting with a vmlinux.bin appended dtb instead of using a built-in one.

No particular objections to this change, current platforms support by
this change fall under 3 categories:

- BCM7xxx using a CFE providing a firmware interface you call into using
code from arch/mips/fw/cfe/
- BCM63xx CFE
- BCM33xx using Aeolus [1]

The first two are likely not going to be changed since these are
currently deployed products/systems, however the latter could be changed
to match UHI as Kevin proposed a while ago.

> 
> Signed-off-by: Jonas Gorski <jogo-p3rKhJxN3npAfugRpC6u6w@public.gmane.org>
> ---
>  arch/mips/bmips/setup.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/mips/bmips/setup.c b/arch/mips/bmips/setup.c
> index fae800e..526ec27 100644
> --- a/arch/mips/bmips/setup.c
> +++ b/arch/mips/bmips/setup.c
> @@ -149,6 +149,8 @@ void __init plat_mem_setup(void)
>  	/* intended to somewhat resemble ARM; see Documentation/arm/Booting */
>  	if (fw_arg0 == 0 && fw_arg1 == 0xffffffff)
>  		dtb = phys_to_virt(fw_arg2);
> +	else if (fw_arg0 == -2) /* UHI interface */
> +		dtb = (void *)fw_arg1;
>  	else if (__dtb_start != __dtb_end)
>  		dtb = (void *)__dtb_start;
>  	else
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2015-04-18  3:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-12 10:24 [PATCH RFC v3 0/4] MIPS: add vmlinu{x,z}.bin appended dtb support Jonas Gorski
2015-04-12 10:24 ` Jonas Gorski
2015-04-12 10:24 ` [PATCH RFC v3 1/4] MIPS: add support for vmlinux.bin appended dtb Jonas Gorski
2015-04-12 10:24   ` Jonas Gorski
2015-04-12 10:24 ` [PATCH RFC v3 2/4] MIPS: add support for vmlinuz.bin " Jonas Gorski
2015-04-12 10:24   ` Jonas Gorski
2015-04-12 10:25 ` [PATCH RFC v3 3/4] MIPS: BMIPS: build all dtbs if no builtin dtb Jonas Gorski
2015-04-12 10:25   ` Jonas Gorski
2015-04-12 10:25 ` [PATCH RFC v3 4/4] MIPS: BMIPS: accept UHI interface for passing a dtb Jonas Gorski
2015-04-12 10:25   ` Jonas Gorski
2015-04-18  3:09   ` Florian Fainelli [this message]
2015-04-18  3:09     ` Florian Fainelli

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=5531CAEF.9090606@gmail.com \
    --to=f.fainelli@gmail.com \
    --cc=James.Hartley@imgtec.com \
    --cc=aaro.koskinen@iki.fi \
    --cc=abrestic@chromium.org \
    --cc=blogic@openwrt.org \
    --cc=cernekee@gmail.com \
    --cc=daniel.schwierzeck@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=jogo@openwrt.org \
    --cc=linux-mips@linux-mips.org \
    --cc=markos.chandras@imgtec.com \
    --cc=paul.burton@imgtec.com \
    --cc=ralf@linux-mips.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.