Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MAINTAINERS: Update Corstone1000 maintainer #6237

Conversation

xueliang-zhong-arm
Copy link
Contributor

Update maintainer of ARM Corstone1000 to Emekcan Aras.

@jenswi-linaro
Copy link
Contributor

@vishnu-banavath, OK with you?
BTW, why isn't the new maintainer doing this PR?

@xueliang-zhong-arm
Copy link
Contributor Author

Emekcan Aras is currently active developer of ARM Corstone-1000 board in various projects, including supporting Corstone-1000 optee recipes & patches in Yocto meta-arm, e.g.:
https://git.yoctoproject.org/meta-arm/log/?qt=grep&q=Emekcan+Aras

If the OP-TEE maintainers require Emekcan himself to raise such PR, that is fine, we'll raise a new PR. Please confirm.

@jforissier
Copy link
Contributor

I suppose we could do with an Acked-by: from both @ememarar and @vishnu-banavath

@ememarar
Copy link

Sorry, we talked to Xueliang about this, but I was a bit busy, so he helped :) looks good to me.

@xueliang-zhong-arm
Copy link
Contributor Author

Vishnu is on vacation this week, I'll ask him to ack this PR next week. Thanks all.

@vishnu-banavath
Copy link
Contributor

I'm ok with this change and sorry for the delayed response.

Thanks

@jforissier
Copy link
Contributor

Thanks @vishnu-banavath
@xueliang-zhong-arm can you please add Acked-by: Vishnu Banavath <vishnu.banavath@arm.com>? Thanks!

Update maintainer of ARM Corstone1000 to Emekcan Aras.

Signed-off-by: Xueliang Zhong <xueliang.zhong@arm.com>
Acked-by: Vishnu Banavath <vishnu.banavath@arm.com>
@xueliang-zhong-arm
Copy link
Contributor Author

Hi, I've just added
Acked-by: Vishnu Banavath <vishnu.banavath@arm.com>
in the comment message.

@jforissier
Copy link
Contributor

I have added Emeckan's Acked-by too. Merged, thanks.

@jforissier jforissier closed this Sep 5, 2023
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.

5 participants