All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sandy Patterson <apatterson@sightlogix.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] Ability to load linux kernel on rock2 RK3288
Date: Fri, 15 Jul 2016 09:40:48 -0400	[thread overview]
Message-ID: <CABCGj3bPY7bgQY6KFZeXW_Xwp3jKnqxH9b8BRfQwytfNRJO-sQ@mail.gmail.com> (raw)
In-Reply-To: <CAPnjgZ3pPmfZEwtHz6PnLVJ9piTD0qa6_Ts4m3m4hcPD5BOLTg@mail.gmail.com>

Hi Simon,

I think I screwed up submitting, and didn't cc the maintainers for the
reverts. I can resubmit and get patman to behave. What do you suggest?

I still think this is the best patch to getting rock2 to load kernels again.

I just retested with a patch specific to my kernel. I think 3.14 linux has
some quirks in how it wants the dram setup, and I get instability without
some changes to the dram init. I don't have enough of a system to test for
this instability using the latest mainline kernel, so I left that patch out.

Ziyuan had an alternative fix for efi_loader, but it looks like it may
break efi_loader for others, and I don't know how to test the efi
functionality, so I think disabling for RK3288 is best. I think I should
move the CONFIG_EFI_LOADER patch to somehow patching rk3288_common.h
instead (can I #undef config vars in that file?)

Sandy



On Thu, Jul 14, 2016 at 11:19 PM, Simon Glass <sjg@chromium.org> wrote:

> HI Sandy,
>
> On 13 July 2016 at 11:51, Sandy Patterson <apatterson@sightlogix.com>
> wrote:
> > I did a little more on this, and talked to someone else here. It seems
> that
> > my problem with loading the kernel including these patches is specific to
> > our kernel and after applying a local patch we have, it appears to load
> > fine.
> >
> > So this patchset gets me back to the same functionality in v2016.03.
> >
> > We're left with the puzzle of what's wrong on the RK3288 regarding
> caching
> > and memory.
>
> So what is the status of this patch set? Should it be withdrawn?
>
> Regards,
> Simon
>
> >
> > Sandy Patterson
> >
> > On Mon, Jul 11, 2016 at 1:38 PM, Sandy Patterson <
> apatterson at sightlogix.com>
> > wrote:
> >>
> >> I wasn't able to load the linux kernel using a Rock2 board
> >> using the latest master branch. The board hangs after it has
> >> handed executing over to the kernel. I found that the latest release
> >> that worked was v2016.03.
> >>
> >> I did some searching and I suspect the problem may be cache related.
> >>
> >> This patchset allows the kernel to start by reverting two problem
> >> commits and disabling EFI_LOADER which I suspect rubs the caching the
> >> wrong way. We also found that the 512M limit for fdt and initrd is now
> >> 256M.
> >> I'm not sure why this is.
> >>
> >> This still doesn't work 100%. I think it's not initializing the SD card
> >> volages correctly, but at least the Kernel is loading.
> >>
> >> I also am not sure changing the caching for all armv7 is the right
> >> answer. I wasn't too sure about the revert. I am not very familiar with
> >> this low level stuff.
> >>
> >> Sandy Patterson
> >>
> >>
> >> Sandy Patterson (4):
> >>   Revert "arm: Replace v7_maint_dcache_all(ARMV7_DCACHE_CLEAN_INVAL_ALL)
> >>     with asm code"
> >>   Revert "arm: Replace v7_maint_dcache_all(ARMV7_DCACHE_INVAL_ALL) with
> >>     asm code"
> >>   Disable CONFIG_EFI_LOADER for rock2.
> >>   RK3288 needs fdt and initrd below 256M now.
> >>
> >>  arch/arm/cpu/armv7/Makefile                  |   2 +-
> >>  arch/arm/cpu/armv7/cache_v7.c                | 135
> >> ++++++++++++++++++++++-
> >>  arch/arm/cpu/armv7/cache_v7_asm.S            | 154
> >> ---------------------------
> >>  arch/arm/mach-uniphier/arm32/lowlevel_init.S |  67 +++++++++++-
> >>  configs/rock2_defconfig                      |   1 +
> >>  include/configs/rk3288_common.h              |   6 +-
> >>  6 files changed, 201 insertions(+), 164 deletions(-)
> >>  delete mode 100644 arch/arm/cpu/armv7/cache_v7_asm.S
> >>
> >> --
> >> 1.9.1
> >>
> >
>

  reply	other threads:[~2016-07-15 13:40 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-11 17:38 [U-Boot] [PATCH 0/4] Ability to load linux kernel on rock2 RK3288 Sandy Patterson
2016-07-11 17:38 ` [U-Boot] [PATCH 1/4] Revert "arm: Replace v7_maint_dcache_all(ARMV7_DCACHE_CLEAN_INVAL_ALL) with asm code" Sandy Patterson
2016-07-11 17:38 ` [U-Boot] [PATCH 2/4] Revert "arm: Replace v7_maint_dcache_all(ARMV7_DCACHE_INVAL_ALL) " Sandy Patterson
2016-07-11 17:38 ` [U-Boot] [PATCH 3/4] Disable CONFIG_EFI_LOADER for rock2 Sandy Patterson
2016-07-11 17:38 ` [U-Boot] [PATCH 4/4] RK3288 needs fdt and initrd below 256M now Sandy Patterson
2016-08-27 16:06   ` Simon Glass
2016-08-27 17:03     ` Sandy Patterson
2016-10-15 13:10       ` Simon Glass
2016-07-13 17:51 ` [U-Boot] [PATCH 0/4] Ability to load linux kernel on rock2 RK3288 Sandy Patterson
2016-07-15  3:19   ` Simon Glass
2016-07-15 13:40     ` Sandy Patterson [this message]
2016-07-22  3:21       ` Simon Glass
2016-07-22 13:49         ` Sandy Patterson
2016-07-22 15:02           ` Ziyuan Xu
2016-07-22  3:13   ` Ziyuan Xu

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=CABCGj3bPY7bgQY6KFZeXW_Xwp3jKnqxH9b8BRfQwytfNRJO-sQ@mail.gmail.com \
    --to=apatterson@sightlogix.com \
    --cc=u-boot@lists.denx.de \
    /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.