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

Support for using Linux as firmware on HDMI2USB-litex-firmware. #372

Merged
merged 43 commits into from Jan 11, 2018

Conversation

mithro
Copy link
Member

@mithro mithro commented Nov 3, 2017

No description provided.

Copy link
Contributor

@stffrdhrn stffrdhrn left a comment

Choose a reason for hiding this comment

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

Looks good, a few points.


(
cd third_party/litex
git checkout or1k-linux
Copy link
Contributor

Choose a reason for hiding this comment

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

Why use or1k-linux here, but litex-minimal below?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is checking out in third_party/litex directory - below is checking out the Linux repo into third_party/linux.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, litex/linux look almost the same to me.

if [ ! -d "$LINUX_SRC_DIR" ]; then
(
cd $(dirname $LINUX_SRC_DIR)
git clone https://github.com/mithro/linux-litex.git -b litex-minimal linux
Copy link
Contributor

Choose a reason for hiding this comment

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

I am not sure if people will really do this? It takes a long time to clone linux, but I don't see many other options.

Maybe we can offer users to specify their linux home to symlink in?

)

make tftp
make gateware-load
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want make firmware-connect too?

mithro and others added 27 commits November 17, 2017 16:48
Fix platform.build() needs builddir
Everyone who has a recent version of the ip tool can use it to create
tap0 --- and it's standard on recent Debian, Ubuntu and CentOS.
Not everyone has UML-tools (tunctl) or openvpn installed.  So use ip
instead of openvpn to instantiate the tun devices.
Use ip not openvpn to create tap0.
 * litex changed from v0.1-191-gaa6dadf0 to v0.1-211-gb463b216

Full submodule status
--
 f56f329ed23a25d002352dedba1e8f092a47286f edid-decode (heads/master)
 070d8b29acf1445c08ebf3073d04206461819bbe flash_proxies (heads/master)
 f31f8a03ff6911befa6d07bf347f8478c30448d8 litedram (heads/master)
 937c2407276371d9f8c1bb19c4ae8e97e581da83 liteeth (heads/master)
 c836c467b8d086c7ade23a45557644e4620a4bdc litepcie (heads/master)
 c3d491552cc77312bbadf6488c08e1e3df9612ef litesata (heads/master)
 d5887a3eb0de105cfc55e52fbeb9de9da035e7db litescope (heads/master)
 7a17876bc7f8943a2a0b500e4c1cb1e454c680b6 liteusb (heads/master)
 cd44e3f14cd3042a948170d43c21919b00c3c35d litevideo (heads/master)
 b463b2169b33bd5861bfa94dcb2c8c896dd36cf7 litex (v0.1-211-gb463b216)
@mithro mithro force-pushed the litex-linux branch 3 times, most recently from fd1f3fc to bc5e831 Compare December 31, 2017 14:05
@mithro mithro changed the title WIP - scripts: Working on script to build Linux. Support for using Linux as firmware on HDMI2USB-litex-firmware. Jan 1, 2018
@mithro mithro force-pushed the litex-linux branch 3 times, most recently from 245ae41 to d851073 Compare January 4, 2018 11:15
See the following page - https://discourse.drone.io/t/github-claims-that-merge-refs-are-undocumented-feature/1100

> There appears to be a bug in Github where upon pull request creation,
> or pushing to an existing pull request, the PR webhook is fired before
> the merge ref is created.
>
> This results in dangerous bugs in CI systems where the code being
> tested in a CI job is not the code that the PR author thinks is being
> tested.
>
> Specifically when this happens, one of two things happens:
>
> 1) The CI job fails because the merge ref cannot be found (if this is
>    right when the PR is opened)
>
> 2) The CI job runs but uses the wrong commit (if this is a subsequent
>    push to an existing PR) - This is because the FETCH_HEAD is
>    pointing to an older commit.
@mithro mithro merged commit 0cd842f into timvideos:master Jan 11, 2018
@mithro mithro deleted the litex-linux branch January 11, 2018 09:16
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.

None yet

3 participants