Skip to content

Conversation

@xiulipan
Copy link
Contributor

@xiulipan xiulipan commented Dec 9, 2019

Need to disable asrc support for BYT and CHT for xcc build.

Signed-off-by: Pan Xiuli xiuli.pan@linux.intel.com

Fix missed change from #1908 to #2024 for xcc build

Need to disable asrc support for BYT and CHT for xcc build.

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
@jajanusz
Copy link
Contributor

jajanusz commented Dec 9, 2019

@singalsu @xiulipan Why is ASRC anyway 'yes' by default? Shouldn't we have just src+vol in default components and all other optional?

@jajanusz jajanusz merged commit 8f51e6b into thesofproject:master Dec 9, 2019
@xiulipan
Copy link
Contributor Author

@singalsu @xiulipan Why is ASRC anyway 'yes' by default? Shouldn't we have just src+vol in default components and all other optional?

@jajanusz Good point. @singalsu any idea here?

@singalsu
Copy link
Collaborator

In my view the idea has been in generic builds to enable everything by default (make sure the features do not break due to other development) and disable only features those are not feasible for a platform.

@singalsu
Copy link
Collaborator

@xiulipan Should the "CONFIG_COMP_ASRC=n" be done for HSW and BDW too? When I did the ASRC PR I didn't know that xcc build uses separate defconfig files.

@xiulipan
Copy link
Contributor Author

@singalsu For bdw/hsw, they share the same defconfig.
But for BYT/CHT the xcc have special optimization level for the FW size issue.

@singalsu
Copy link
Collaborator

Thanks for explanation @xiulipan !

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.

4 participants