This repository was archived by the owner on Jan 3, 2023. It is now read-only.
Move bfn asic drivers out of platform package#30
Merged
mkbalani merged 3 commits intobarefootnetworks:bfn_kdrvfrom Oct 23, 2018
Merged
Move bfn asic drivers out of platform package#30mkbalani merged 3 commits intobarefootnetworks:bfn_kdrvfrom
mkbalani merged 3 commits intobarefootnetworks:bfn_kdrvfrom
Conversation
krambn
reviewed
Oct 23, 2018
| @@ -0,0 +1,15 @@ | |||
| Copyright (C) 2016 Microsoft, Inc | |||
Author
There was a problem hiding this comment.
Not sure what to use here. Copied this over from license file from previously build bfn-platform-montara deb packaging
|
|
||
| # Runtime dependency on sai is set only for syncd | ||
| $(SYNCD)_RDEPENDS += $(BFN_SAI) $(WNC_OSW1800_PLATFORM) $(BFN_INGRASYS_PLATFORM) $(BFN_PLATFORM) | ||
| $(SYNCD)_RDEPENDS += $(BFN_MODULE) $(BFN_SAI) $(WNC_OSW1800_PLATFORM) $(BFN_INGRASYS_PLATFORM) $(BFN_PLATFORM) |
Author
There was a problem hiding this comment.
Isn't the presence of these modules mandatory at runtime for syncd to run?
krambn
approved these changes
Oct 23, 2018
NStetskovych-zz
left a comment
NStetskovych-zz
left a comment
There was a problem hiding this comment.
Do we have separate package for asic drivers now?
Author
|
@NStetskovych yes, there is a single separate debian package which is used for all platforms |
mkbalani
added a commit
that referenced
this pull request
Oct 31, 2018
…c-net#2183) * Move bfn asic drivers out of platform package (#30) * Move bfn asic drivers out of platform package * Change debian pkg name for bfn modules * Change license file * Fix build breakage due to dependency issue * Minor changes to debian bld files (#31)
akokhan
pushed a commit
that referenced
this pull request
Sep 9, 2021
* 3b330db4a 2021-01-18 | [build]: Fix build error when compiling for armhf (32-bit) (#30) (HEAD, origin/master, origin/HEAD, master) [dflynn-Nokia] * 56aaa225b 2021-01-16 | [ci]: add pipeline for armhf and arm64 (#29) [lguohan] * 90da6141c 2021-01-12 | [ci]: propagate the correct error code the next step (#27) [lguohan] Signed-off-by: Guohan Lu <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.