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

Update description files for RK322X #1718

Merged
merged 3 commits into from
Jul 26, 2017

Conversation

TonyXie06
Copy link
Contributor

Update .travis.yml, MAINTAINERS.md and README.md for RK322X #1666

@jforissier
Copy link
Contributor

Minor comment: you should probably drop the word "description" from ".travis.yml: Add Rockchip RK322X description".
Thanks for the updates and for adding yourself as a maintainer.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

@TonyXie06
Copy link
Contributor Author

Agreed, I will drop it.

jforissier, I discussed with my teammates and come to an advice: may I use a public email account as maintainer , just like "Rockchip rockchip-op-tee@rock-chips.com" ? Because there will be more Rockchip SoCs supported by me or my teammates in the future, many of us play different roles for porting and maintainance work.

Signed-off-by: Joseph Chen <chenjh@rock-chips.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
@jforissier
Copy link
Contributor

@TonyXie06 I think a "generic" maintainer address is fine. So, if it is more convenient for you please go ahead and update it.

Signed-off-by: Joseph Chen <chenjh@rock-chips.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Joseph Chen <chenjh@rock-chips.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
@TonyXie06
Copy link
Contributor Author

  1. tags applied;
  2. drop word "description" from ".travis.yml: Add Rockchip RK322X description";
  3. update maintainer as "Rockchip op-tee@rock-chips.com".

@jforissier jforissier merged commit d4c2757 into OP-TEE:master Jul 26, 2017
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.

3 participants