Skip to content

Conversation

douniwan5788
Copy link
Contributor

@douniwan5788 douniwan5788 commented Aug 19, 2024

I don't have the NXP HitagS, only tested on 8211.

Copy link

You are welcome to add an entry to the CHANGELOG.md as well

@iceman1001
Copy link
Collaborator

Nice!

As I look at the changes it feels like this should have been three PR's

  1. darvin handling of mkversion
  2. hitagS
  3. generic lfops / em4x50 fiddling

We dislike mixed PR's since its hard to follow changes.

If you wouldn't mind splitting up ?

@douniwan5788
Copy link
Contributor Author

Of course, then how should I proceed? Do I have to cherry-pick to different branches?

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