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 NXP i.MX8 SOC family support. #556

Merged
merged 1 commit into from
Feb 20, 2023
Merged

Conversation

unbel13ver
Copy link
Contributor

@unbel13ver unbel13ver commented Feb 13, 2023

Currently there are two devices supported:

  • imx8qm-mek
  • imx8qxp-mek

Signed-off-by: Ivan Nikolaenko ivan.nikolaenko@unikie.com

Description of changes

Changes made in the way it makes very easy to add another mx8-based device or device-specific configuration.

Things done
  • Tested the changes in your own NixOS Configuration
  • Tested the changes end-to-end by using your fork of nixos-hardware and
    importing it via <nixos-hardware> or Flake input

@necrophcodr
Copy link
Contributor

@unbel13ver is the firmware files completely public domain to use, or are they under some license agreement that the user needs to agree to?

@Mic92
Copy link
Member

Mic92 commented Feb 13, 2023

I think in either case we would be fine to have the nix expression in this repository because the binaries are public accessible and we are not hosting repackaged versions of them (hence no copy). Might be nice though to have a license in meta.license if available.

@unbel13ver
Copy link
Contributor Author

@unbel13ver is the firmware files completely public domain to use, or are they under some license agreement that the user needs to agree to?

Thanks for quick reply. Yes, the firmware is publicly available, but one needs to accept NXP's EULA:

Welcome to NXP imx-seco-3.8.6.bin

You need to read and accept the EULA before you can continue.

LA_OPT_NXP_Software_License v34 February 2022

I am not quilte familiar with legal stuff, so I would appreciate a lot if you point me what kind of license information I should add.

@Mic92
Copy link
Member

Mic92 commented Feb 18, 2023

Does not look like it's redistributable so I would go with lib.licenses.unfree

@Mic92 Mic92 marked this pull request as draft February 18, 2023 06:00
Currently there are two devices supported:
* imx8qm-mek
* imx8qxp-mek

Signed-off-by: Ivan Nikolaenko <ivan.nikolaenko@unikie.com>
@unbel13ver
Copy link
Contributor Author

unbel13ver commented Feb 20, 2023

Does not look like it's redistributable so I would go with lib.licenses.unfree

Addressed
https://github.com/NixOS/nixos-hardware/pull/556/files#diff-748168167d9e4a84f5dfbd73518482b51af244e2918fd6d7ed40957e9b63da64R65

@Mic92 Mic92 marked this pull request as ready for review February 20, 2023 15:53
@Mic92
Copy link
Member

Mic92 commented Feb 20, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Feb 20, 2023

Build succeeded:

@bors bors bot merged commit 157e1e4 into NixOS:master Feb 20, 2023
@@ -0,0 +1,3 @@
final: _prev: {
inherit ( final.callPackage ./bsp/u-boot/imx8/imx-uboot.nix { pkgs = final; targetBoard = "imx8qm"; }) ubootImx8 imx-firmware;

Choose a reason for hiding this comment

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

Is ./bsp/u-boot/imx8/imx-uboot.nix correct path?

Copy link
Member

Choose a reason for hiding this comment

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

Maybe some refactoring issues after moving the code to the repository. @unbel13ver can you fix this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi, sorry for this. I'll fix paths and improve README

@@ -0,0 +1,3 @@
final: prev: {
inherit ( final.callPackage ./bsp/u-boot/imx8/imx-uboot.nix { pkgs = final; targetBoard = "imx8qxp"; }) ubootImx8 imx-firmware;
Copy link

@GangaRamX GangaRamX Mar 7, 2023

Choose a reason for hiding this comment

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

./bsp/u-boot/imx8/imx-uboot.nix no u-boot/imx8 directory in bsp. Also relative path of bsp doesn't seem ok.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Relative paths are ok as long as they point within the repo directory structure.

Mic92 added a commit that referenced this pull request May 5, 2023
This was an overside in #556
Mic92 added a commit that referenced this pull request May 5, 2023
This was an overside in #556
@Mic92 Mic92 mentioned this pull request May 5, 2023
2 tasks
Mic92 added a commit that referenced this pull request May 5, 2023
This was an overside in #556
ghostbuster91 pushed a commit to ghostbuster91/nixos-hardware that referenced this pull request May 7, 2023
This was an overside in NixOS#556
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

4 participants