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

[no ci] Overlay: export soc environment variable #947

Merged
merged 4 commits into from
Aug 12, 2023
Merged

[no ci] Overlay: export soc environment variable #947

merged 4 commits into from
Aug 12, 2023

Conversation

viktorxda
Copy link
Collaborator

No description provided.

@viktorxda viktorxda merged commit 1ac67dc into OpenIPC:master Aug 12, 2023
@viktorxda viktorxda deleted the profile branch August 12, 2023 11:45
@flyrouter
Copy link
Member

Hi !
When you get a chance, please give a little more explanation

  • I understand that now there are only two cores for i6b, the rest are symlinks.
  • What are the options changes in ssc338 core related to, just wondering.
  • As far as I understand, the firmware package for mt7601 is not used anymore.
  • What is the reason for removing the SOC=$(fw_printenv -n soc) variable?

I have no objections to the actions but would like to be kept up to date on the details, thanks ;)

@viktorxda
Copy link
Collaborator Author

Hello, I checked the differences between the families on the kernel configs and they are basically identical with minor differences. For the SSC335 and SSC337 is only boils down to the used device tree file.

The changes made on the SSC338 config are mostly intended to have both families as similar as possible in settings and options.

The external MT7601U driver needs the MT7601USTA.dat file to read some of the configuration data from it, this is not needed for the built-in kernel driver.
The firmware file mt7601u.bin is needed by both, the package is still available but was simplified, as we only need three small files from a ~500MB archive: 73dd5a6

The SOC variable was introduced to identify the Sigmastar chipsets with the environment variable instead of the ipctool output.
However, if microbe-web uses ipcinfo/ipctool, it cannot read the environment variable and outputs the default guessed chipsets based on the family id (SSC336X/SSC338X/SSC339X).
This was changed on ipctool itself, so it can now properly read the uboot 'soc' variable.

@flyrouter
Copy link
Member

Thank you so much for your lovely comment.
It is very nice to see not only your work, but also explanations that are easy to understand.
Good luck with your experiments and research !

@viktorxda
Copy link
Collaborator Author

One thing I forgot to mention, the MT7601U is currently not used by Sigmastar, if this changes in the future we can enable the driver and add the firmware back in.

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