Skip to content

drivers: mediatek: fix build error - #5808

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
tinghan-shen:fix-ipc_cmd_hdr-build-error
May 12, 2022
Merged

lgirdwood merged 1 commit into
thesofproject:mainfrom
tinghan-shen:fix-ipc_cmd_hdr-build-error

Conversation

@tinghan-shen

Copy link
Copy Markdown
Contributor

fix mediatek driver build error after ipc_cmd_hdr is updated.

/proj/mtk15399/cros/src/third_party/sof/src/drivers/mediatek/mt8186/ipc.c: In function ‘ipc_platform_do_cmd’:
/proj/mtk15399/cros/src/third_party/sof/src/drivers/mediatek/mt8186/ipc.c:77: error: ‘ipc_cmd_hdr’ undeclared (first use in this function)
/proj/mtk15399/cros/src/third_party/sof/src/drivers/mediatek/mt8186/ipc.c:77: error: (Each undeclared identifier is reported only once
/proj/mtk15399/cros/src/third_party/sof/src/drivers/mediatek/mt8186/ipc.c:77: error: for each function it appears in.)
/proj/mtk15399/cros/src/third_party/sof/src/drivers/mediatek/mt8186/ipc.c:77: error: ‘hdr’ undeclared (first use in this function)
CMakeFiles/sof.dir/build.make:714: recipe for target 'CMakeFiles/sof.dir/src/drivers/mediatek/mt8186/ipc.c.o' failed

Signed-off-by: Tinghan Shen tinghan.shen@mediatek.com

fix mediatek driver build error after ipc_cmd_hdr is updated.

sof/src/drivers/mediatek/mt8186/ipc.c: In function ‘ipc_platform_do_cmd’:
sof/src/drivers/mediatek/mt8186/ipc.c:77: error: ‘ipc_cmd_hdr’ undeclared (first use in this function)
sof/src/drivers/mediatek/mt8186/ipc.c:77: error: (Each undeclared identifier is reported only once
sof/src/drivers/mediatek/mt8186/ipc.c:77: error: for each function it appears in.)
sof/src/drivers/mediatek/mt8186/ipc.c:77: error: ‘hdr’ undeclared (first use in this function)
CMakeFiles/sof.dir/build.make:714: recipe for target 'CMakeFiles/sof.dir/src/drivers/mediatek/mt8186/ipc.c.o' failed

Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
@kuanhsuncheng
kuanhsuncheng requested a review from afq984 May 12, 2022 12:49

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marc-hb fyi mt8186 target missing from CI build (maybe missing from build all and falling thru?)

@lgirdwood
lgirdwood merged commit 42e0d5e into thesofproject:main May 12, 2022
@marc-hb

marc-hb commented May 12, 2022

Copy link
Copy Markdown
Collaborator

The m8186 is still missing from the build because we are still waiting for Mediatek to provide the toolchain. I re-opened #5388, let's discuss there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants