Skip to content

Ubuntu base 18.04 lts#16

Merged
olajep merged 10 commits into
parallella:masterfrom
xuminready:Ubuntu_Base_18.04_LTS
Jul 26, 2018
Merged

Ubuntu base 18.04 lts#16
olajep merged 10 commits into
parallella:masterfrom
xuminready:Ubuntu_Base_18.04_LTS

Conversation

@xuminready
Copy link
Copy Markdown
Contributor

upgrade to Ubuntu Base 18.04 LTS (Bionic Beaver)

Copy link
Copy Markdown

@ThorodanBrom ThorodanBrom left a comment

Choose a reason for hiding this comment

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

This is still using a very old kernel, ie. the parallella-linux kernel has to also be updated to at least 4.15, right?

@xuminready
Copy link
Copy Markdown
Contributor Author

It's Linux 4.6.0+. The same kernel as Parabuntu 2016.11.1. It works on my Parallella board(zynq 7020).

@olajep
Copy link
Copy Markdown
Member

olajep commented Jul 24, 2018

Thank you @xuminready

Fantastic work!!!

Will try to review / merge ASAP!!

Cheers,
Ola

@olajep
Copy link
Copy Markdown
Member

olajep commented Jul 24, 2018

Please split out the changes to packages*.txt to a separate commit
You need to justify the ones you commented out (and if obsolete remove them).
Many of these have been put in by user requests and will break things for some people if we remove them. Just because you don't need it doesn't mean we can remove it. Hope you'll understand.

// Ola

cd mnt/rootfs/home/parallella
git clone https://github.com/adapteva/epiphany-examples.git --branch 2016.11
git clone https://github.com/parallella/parallella-examples.git --branch 2016.11
git clone --depth=1 https://github.com/adapteva/epiphany-examples.git --branch 2016.11
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No to these two. We might need to ask users to switch branches etc. to troubleshoot things etc.

Copy link
Copy Markdown
Member

@olajep olajep left a comment

Choose a reason for hiding this comment

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

Looking good!

Just some minor things and I'll take it.

@@ -1,5 +0,0 @@
# Parallella PPA
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No biggie but can we please leave this in but replace vivid --> bionic?

There actually is a parallella ppa but we never used it.

Comment thread packages.basic.txt Outdated

# Language Packs (Fix utf8 & get rid of Ubuntu's annoying locale warings)
language-pack-en
#language-pack-en
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This and the other stuff you've commented out you need to justify.
Leave it in here even if it is already installed in the base ubuntu image.
Things might change in the future and then we'll have regressions that can be easily avoided by just leaving the package in here.

Comment thread rootfs.sh Outdated
echo Renaming linaro user to parallella
[ -d ${top}/mnt/rootfs/home/parallella ] || mv ${top}/mnt/rootfs/home/linaro ${top}/mnt/rootfs/home/parallella
sed -i 's/linaro/parallella/g' ${top}/mnt/rootfs/etc/{group,shadow,passwd}
#echo Renaming linaro user to parallella
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a 'ubuntu' user in the ubuntu base image? If yes, rename that user. If no, remove these lines instead of just commenting them out.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't think there's a 'ubuntu' user in the base image.
https://wiki.ubuntu.com/Base/InstallationExample

@ThorodanBrom
Copy link
Copy Markdown

ThorodanBrom commented Jul 24, 2018

Are there any advantages to updating parallella-linux to 4.15? I understand that @xuminready said that it works fine with the existing kernel, but 4.15 is the recommended kernel for Bionic.

If there is, I can try to make the necessary changes

@olajep
Copy link
Copy Markdown
Member

olajep commented Jul 24, 2018

@ThorodanBrom wrote.

This is still using a very old kernel, ie. the parallella-linux kernel has to also be updated to at least 4.15, right?

Yes. We use https://github.com/analogdevicesinc/linux.git as upstream so you should merge their latest stable release. Believe it's 2018_R1.

Also need to rebuild the fpga bitstreams

HDMI bitstreams:
Need to switch to the corresponding hdl branch here:
https://github.com/parallella/parallella-fpga

Headless bitstreams:
Rebuild
https://github.com/parallella/oh/tree/stable/parallella/fpga

Correct vivado version can be found here:
https://github.com/analogdevicesinc/hdl/blob/hdl_2018_r1/library/scripts/adi_ip.tcl#L6

// Ola

@xuminready
Copy link
Copy Markdown
Contributor Author

@ThorodanBrom Please upgrade Linux kernel if you know how to do it.

@ThorodanBrom
Copy link
Copy Markdown

@xuminready @olajep I will try to upgrade the kernel in some time. Honestly, I am new to the parallella board, so I'll have to find some instructions on how to do the bitstream building etc.

@olajep olajep merged commit 7066403 into parallella:master Jul 26, 2018
@olajep
Copy link
Copy Markdown
Member

olajep commented Jul 27, 2018 via email

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