Skip to content

qcom-multimedia-image: install thermald only on aarch64 - #418

Closed
Jose Quaresma (quaresmajose) wants to merge 1 commit into
qualcomm-linux:mainfrom
quaresmajose:qcom-multimedia-image
Closed

Jose Quaresma (quaresmajose) wants to merge 1 commit into
qualcomm-linux:mainfrom
quaresmajose:qcom-multimedia-image

Conversation

@quaresmajose

Copy link
Copy Markdown
Member

There are currently build problems in qcom-armv7a. Since thermald is not tested in no other than aarch64, there's no point in installing in other arch.

@quaresmajose

Copy link
Copy Markdown
Member Author

Required for qualcomm-linux/meta-qcom#2881

There are currently build problems in qcom-armv7a.
Since thermald is not tested in no other than aarch64,
there's no point in installing in other arch.

Signed-off-by: Jose Quaresma <jose.quaresma@oss.qualcomm.com>
@lumag

Copy link
Copy Markdown
Contributor

@quaresmajose

Copy link
Copy Markdown
Member Author

Yes, your approach is technically what we should do. But this PR proposal is still valid, we haven't tested it in armv7a and I believe no one else will either.

@quaresmajose

Copy link
Copy Markdown
Member Author

Can this one be merged?

@lumag

Copy link
Copy Markdown
Contributor

Can this one be merged?

Didn't we fix thermald on armv7a? I"ve been onsite last week so I might have lost a track of it. Let me check it.

@quaresmajose

Copy link
Copy Markdown
Member Author

Can this one be merged?

Didn't we fix thermald on armv7a? I"ve been onsite last week so I might have lost a track of it. Let me check it.

Yes, we have armv7a fixed now with your patch qualcomm-linux/meta-qcom@faad3ec but what is the value of installing it? As far as I know, nobody uses that in armv7a.

@lumag

Copy link
Copy Markdown
Contributor

Can this one be merged?

Didn't we fix thermald on armv7a? I"ve been onsite last week so I might have lost a track of it. Let me check it.

Yes, we have armv7a fixed now with your patch qualcomm-linux/meta-qcom@faad3ec but what is the value of installing it? As far as I know, nobody uses that in armv7a.

I'd say, we don't use a lot of tools on armv7a. However building them for armv7a ensures that they are buildable and useable.

@quaresmajose

Copy link
Copy Markdown
Member Author

Can this one be merged?

Didn't we fix thermald on armv7a? I"ve been onsite last week so I might have lost a track of it. Let me check it.

Yes, we have armv7a fixed now with your patch qualcomm-linux/meta-qcom@faad3ec but what is the value of installing it? As far as I know, nobody uses that in armv7a.

I'd say, we don't use a lot of tools on armv7a. However building them for armv7a ensures that they are buildable and useable.

I agree

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants