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

Add 7250/7450 Serial Pinout Details #5

Open
wants to merge 99 commits into
base: master
Choose a base branch
from

Conversation

eduncan911
Copy link

@eduncan911 eduncan911 commented Feb 3, 2020

Putting the serial cable details in one location, easy to reference going forward.

Also added some sub-sub-sub ### headers to the Prerequisites area.

docs/icx7250.md Outdated

**Connect** to the switches serial/console port on the front using a program like Putty (9600 8N1), and connect any of the normal switch ports to your network (do NOT use the dedicated management port).
Copy link
Author

Choose a reason for hiding this comment

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

this was not removed... it just moved up to line 16 above (damn Github)

@eduncan911
Copy link
Author

Ping @Fohdeesha for review. :)

@Fohdeesha
Copy link
Owner

Hey sorry didn't mean to miss this. I've been planning (among other additions) to just add an extra doc page under the brocade section specifically for all the serial/console connection details like this, including the easiest way to buy or make the required weird USB cables for the 7 series. Hopefully I'll get around to it soon

@eduncan911
Copy link
Author

Would you like me to fix the conflicts? Then when you get around to making that sub-page, you can remove these?

@Fohdeesha
Copy link
Owner

indeed if you re-base so there's no conflicts I'll merge then organize everything. There will be a major update to the guide with the latest firmware and some license info etc soon

@Fohdeesha Fohdeesha force-pushed the master branch 5 times, most recently from b1015ef to 8033717 Compare October 1, 2024 03:36
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.

4 participants