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 README.md to follow the latest version linux #430

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yskelg
Copy link

@yskelg yskelg commented Aug 17, 2024

Updated the documentation to follow the latest version,
vivado 2022.1, shenki's buildroot on microwatt-2022.08 and latest Linux kernel on v6.11.0-rc3.
Also, I've added an image to make it easier to understand.

2024-08-1722-08-09-ezgif com-video-to-gif-converter

Updated the documentation to follow the latest version, 
microwatt-2022.08 buildroot and latest Linux kernel v6.11.0-rc3.

Signed-off-by: Yunseong Kim <[email protected]>
Copy link
Collaborator

@paulusmack paulusmack left a comment

Choose a reason for hiding this comment

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

I have various comments, mostly around advising people to do things that require sudo. It shouldn't be necessary for people to use sudo.

microwatt/openocd/flash-arty -f a100 build/microwatt_0/arty_a7-100-vivado/microwatt_0.bit
microwatt/openocd/flash-arty -f a100 dtbImage.microwatt.elf -t bin -a $FLASH_ADDRESS
sudo microwatt/openocd/flash-arty -f a100 build/microwatt_0/arty_a7-100-vivado/microwatt_0.bit
sudo microwatt/openocd/flash-arty -f a100 dtbImage.microwatt.elf -t bin -a $FLASH_ADDRESS
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmmm, instead of telling people to use sudo, I think it would be better to tell people about making sure their userid is a member of the dialout group.

@@ -101,10 +101,16 @@ sudo dnf install fusesoc
This is needed to be able to pull down fussoc library components referenced
by microwatt. Run

- If you use artix as a reference, like the uart16550 example below, you need to install the corresponding vivado package.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Which package is that? Where would one get it?

- If you use artix as a reference, like the uart16550 example below, you need to install the corresponding vivado package.

<p align="center">
<img src="https://github.com/user-attachments/assets/0390507c-cdea-4fce-b2f6-d58830bea176" alt="MicroPython running on Microwatt"/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't seem to be an image of Micropython running on Microwatt, more like Linux booting on Microwatt, so the alt tag is confusing.

```
If the cable was plugged in, unplug it when you're done and you'll see the serial console on the minicomputer. If you can't see the serial console, see below.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This statement confuses me. If you unplug the cable, how are you going to see the serial console?


```
$ sudo ifconfig enp6s0 192.168.0.1
$ sudo /etc/init.d/isc-dhcp-server restart
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmmm, it seems dangerous to me to tell people to set up a DHCP server without at least explaining when this might be necessary and what potential problems it might cause. I would rather just tell people to query the DHCP server on their network to find out what address was assigned, or if that is not possible, log in to the microwatt system on the console and run ifconfig (or ip addr list) to find out.

It should be possible for people to build and run microwatt without ever using sudo.

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.

2 participants