LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Nathan Whitehorn <nwhitehorn@freebsd.org>
To: Geoff Levand <geoff@infradead.org>,
	Sascha Schroeder <sascha.schroeder@aethyx.com>
Cc: Linuxppc-dev@lists.ozlabs.org, petitboot@lists.ozlabs.org
Subject: Re: Getting kernel 4.14 to run on PS3
Date: Sun, 31 Dec 2017 14:50:14 -0800	[thread overview]
Message-ID: <1a034ec1-5345-a2a1-780a-b5adbe0af332@freebsd.org> (raw)
In-Reply-To: <5c41fd34-5aac-028e-af12-59b6ab576d91@infradead.org>



On 12/12/17 15:05, Geoff Levand wrote:
> Hi Nathan,
>
> On 12/08/2017 01:25 PM, Nathan Whitehorn wrote:
>> I submitted patches to libfdt that resolve this particular ABI breakage yesterday. If the patch gets merged, newer kernels should become bootable again.
> Here's the link:
>
>    https://github.com/dgibson/dtc/pull/12 (Add limited read-only support for older (V2 and V3) device tree to libfdt.)
>
> I rebased your patch and have it in my ps3-queue
> branch, but my ps3 with ps3-petitboot-09.11.30 still
> would not boot with it.  I didn't spend any time yet
> to look into why.
>
>    https://git.kernel.org/pub/scm/linux/kernel/git/geoff/ps3-linux.git/log/?h=ps3-queue
>
> -Geoff
>

I found a bug (and updated the pull request), which might help.

The other potential issue is that the fdt_get_property*() API in libfdt 
is fundamentally incompatible with the way that V2 device trees align 
data accesses since it returns a pointer to a packed structure with both 
the property header and data. However, the data in V2 devtrees may be 
offset by 4 bytes in a way that depends both on the offset within the 
tree and the length of the property and so cannot be represented only as 
the tail of a fixed structure. I don't know if Linux uses those 
functions; I had to replace two uses in FreeBSD with the equivalent 
fdt_getprop*() calls to get things to work.
-Nathan

  reply	other threads:[~2017-12-31 22:50 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-08  9:24 Getting kernel 4.14 to run on PS3 Sascha Schroeder
2017-12-08 17:37 ` Geoff Levand
2017-12-08 20:43   ` Sascha Schroeder
2017-12-08 21:25   ` Nathan Whitehorn
2017-12-12 23:05     ` Geoff Levand
2017-12-31 22:50       ` Nathan Whitehorn [this message]
2017-12-31 23:26         ` Sascha Schroeder

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=1a034ec1-5345-a2a1-780a-b5adbe0af332@freebsd.org \
    --to=nwhitehorn@freebsd.org \
    --cc=Linuxppc-dev@lists.ozlabs.org \
    --cc=geoff@infradead.org \
    --cc=petitboot@lists.ozlabs.org \
    --cc=sascha.schroeder@aethyx.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox