Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1213045

Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node

Show all headers | View raw

On Tue, 2015-08-25 at 11:42 +0100, Mark Rutland wrote:
> > > Are you then going to hack GRUB, release a special HiKey version of
> > > GRUB, not support any other versions, and still can your firmware
> > > UEFI?
> > 
> > I don't need to hack GRUB at all.
> 
> Then it is working for you by pure chance alone.
> 
> Please listen to the advice you are being given here; we're trying to
> ensure that your platform functions (and continues to function) as best
> it can.

Since we discussed a lot on this, let's make a conclusion on it.

1. UEFI could append the reserved buffer in it's memory mapping.
2. These reserved buffer must be declared in DT, since we also need to
   support non-UEFI (uboot) at the same time.
3. Mailbox node should reference reserved buffer by phandle in DT. Then
   map the buffer as non-cacheable in driver.
4. These reserved buffer must use "no-map" property since it should be
   non-cacheable in driver.
5. A patch is necessary in kernel. If efi stub feature is enabled,
   arm kernel should not parse memory node or reserved memory buffer in
   DT any more. Arm kernel should either fetch memory information from 
   efi or DT. Currently arm kernel fetch both efi memory information and
   reserved buffer from DTB at the same time.

Do you agree on these points?

Regards
Haojian

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-21 20:50 +0200
  Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leo Yan <leo.yan@linaro.org> - 2015-08-22 15:40 +0200
    Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leo Yan <leo.yan@linaro.org> - 2015-08-24 05:30 +0200
  Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leo Yan <leo.yan@linaro.org> - 2015-08-24 11:20 +0200
    Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-24 12:00 +0200
      Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-24 12:30 +0200
        Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leif Lindholm <leif.lindholm@linaro.org> - 2015-08-24 13:50 +0200
          Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-25 10:20 +0200
            Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leif Lindholm <leif.lindholm@linaro.org> - 2015-08-25 11:50 +0200
              Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-25 12:20 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leif Lindholm <leif.lindholm@linaro.org> - 2015-08-25 12:50 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-25 12:50 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-25 15:50 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leif Lindholm <leif.lindholm@linaro.org> - 2015-08-25 16:30 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-08-25 17:00 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leif Lindholm <leif.lindholm@linaro.org> - 2015-08-25 17:40 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-08-25 17:50 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-26 04:50 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leo Yan <leo.yan@linaro.org> - 2015-08-25 18:10 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-26 03:30 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leo Yan <leo.yan@linaro.org> - 2015-08-26 09:10 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-27 18:40 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Leo Yan <leo.yan@linaro.org> - 2015-08-28 08:40 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Daniel Thompson <daniel.thompson@linaro.org> - 2015-08-27 18:00 +0200
                Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-27 18:50 +0200
        Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-24 14:50 +0200
          Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Haojian Zhuang <haojian.zhuang@linaro.org> - 2015-08-25 10:20 +0200
            Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node Mark Rutland <mark.rutland@arm.com> - 2015-08-25 13:10 +0200

csiph-web