On 2018年06月11日 16:29, Pan, Xiuli wrote:
On 6/11/2018 15:58, Lauda, Tomasz wrote:
FW sets MW0 to different address than ROM We are setting status 0x05 from bootloader, so it's the correct assumption, that ROM jumped out to FW correctly.
So could we disable the MW0 setting in the boot loader? This seems to broke the status check as we want to check the ROM code.
And BTW, I checked the memory window data and found they are fulled with some data that I can find in the rodata. Should this happen? If the memory data can be some none zero value, i suggest that we add a memset for the memory we used for memory windows
We previously memset all MW datas to 0s at MW setup, except those registers reserved for ROM status, does anything of that changed?
Thanks, ~Keyon
Thanks Xiuli
Tomek
-----Original Message----- From: sound-open-firmware-bounces@alsa-project.org [mailto:sound-open-firmware-bounces@alsa-project.org] On Behalf Of Pan, Xiuli Sent: Monday, June 11, 2018 7:11 AM To: Liam Girdwood liam.r.girdwood@linux.intel.com; sound-open-firmware@alsa-project.org Cc: Kamil Kulesza kamil.kulesza@linux.intel.com Subject: Re: [Sound-open-firmware] [PATCH] arch: xtensa: set SRAM window error codes during bootloader
On 6/9/2018 03:06, Liam Girdwood wrote:
On Fri, 2018-06-08 at 19:42 +0100, Liam Girdwood wrote:
From: Kamil Kulesza kamil.kulesza@linux.intel.com
Set status (0x05) and error (0x00) code in Memory Window 0 when the bootloader starts. boot_entry.S - set status (0x05) and error (0x00) code before wnd0 reprogram platform/memory.h - increase bootloader size
Signed-off-by: Kamil Kulesza kamil.kulesza@linux.intel.com
src/arch/xtensa/boot_entry.S | 22 +++++++++++++++++++ .../apollolake/include/platform/memory.h | 2 +- .../cannonlake/include/platform/memory.h | 2 +- 3 files changed, 24 insertions(+), 2 deletions(-)
This patch seems to be a workaround for the firmware boot up. The status (0x05) should be set by the rom. With the rmbox and dump data from status registers. It seems all memory window is fulled with rodata of the firmware. I think there is some potential bugs for memory window. If this is not a bug maybe we need to add memset for memory windows.
Thanks Xiuli
Applied.
Thanks
Liam _______________________________________________ Sound-open-firmware mailing list Sound-open-firmware@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/sound-open-firmware
Sound-open-firmware mailing list Sound-open-firmware@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/sound-open-firmware
Intel Technology Poland sp. z o.o. ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione. This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
Sound-open-firmware mailing list Sound-open-firmware@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/sound-open-firmware
Sound-open-firmware mailing list Sound-open-firmware@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/sound-open-firmware