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

[centec][arm64] fix tsingma bsp compile error #12774

Conversation

LuiSzee
Copy link
Contributor

@LuiSzee LuiSzee commented Nov 21, 2022

Why I did it

fix centec arm64 tsingma bsp compile error caused by linux kernel api change

How I did it

fix centec arm64 tsingma bsp compile error caused by linux kernel api change

How to verify it

compile centec arm64 image and verify on centec arm64 E530 series device

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@LuiSzee LuiSzee requested a review from lguohan as a code owner November 21, 2022 01:30
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 21, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@LuiSzee
Copy link
Contributor Author

LuiSzee commented Dec 1, 2022

@liushilongbuaa Please help to review this PR, this PR fixs centec tsingma-bsp compile error.

@liushilongbuaa
Copy link
Contributor

The change and error message seems not match. @LuiSzee , are you fixing the following error?

dh_install: warning: Cannot find (any matches for) "src/ctc_wdt/ctc_wdt.ko" (tried in ., debian/tmp)
dh_install: warning: tsingma-bsp missing files: src/ctc_wdt/ctc_wdt.ko
dh_install: error: missing files, aborting

Copy link
Contributor

@liushilongbuaa liushilongbuaa left a comment

Choose a reason for hiding this comment

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

I can approve it because change only affect centec-arm64.

@liushilongbuaa
Copy link
Contributor

Hi @LuiSzee , if you are sure with the failure, just merge.
Let's wait for the build result.

@lguohan lguohan merged commit cd12486 into sonic-net:master Dec 4, 2022
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Dec 5, 2022
fix centec arm64 tsingma bsp compile error caused by linux kernel api change
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #12944

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #12957

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