-
Notifications
You must be signed in to change notification settings - Fork 197
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
Added the OEM Module to the Lepton Interface. #41
Conversation
Only Power down and Power on have been tested. Power down works, Power on does not work.
Thanks for sending this, Ben. I have a couple questions/comments.
|
Well scratch comment 2, I took a second look at the one in the purethermal source and see that there each register access adds the 0x4000 offset (OEM bit). It does still beg the question why these are different. |
Hi,
Sorry, I didn't realise that you had already implemented these two files.
1. What do you mean with 'come from a FLIR SDK'? I used the Software ID
document to create the file. I suppose that means I should include
the copyright information? If so I'll add a commit now.
2. Ah, well, the only change I made was add the Power Register.
Ben
…On 22.03.2017 15:37, Kurt Kiefer wrote:
Thanks for sending this, Ben. I have a couple questions/comments.
1.
Where did this modules come from? Are you the author, or did they
come from a FLIR SDK? If it's the latter, we should to maintain
FLIR's copyright information.
2.
Why does the OEM module's base address differ from the one that
came from FLIR that is included in the purethermal source? (which
you can find here:
https://github.com/groupgets/purethermal1-firmware/blob/master/Middlewares/lepton_sdk/Inc/LEPTON_OEM.h)
3.
Don't commit trivial changes to |LEPTON_I2C_Reg.h|, this makes it
difficult to understand the diff of the file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#41 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAcd2dh2XgR3QlEF5VuIpS34JzsPvmvLks5roUA9gaJpZM4MlXcp>.
|
There is no point in pulling this in if it is already in the purethermal1 firmware, but it might be worth copying over the LEPTON_OEM.{c,h} and LEPTON_RAD.{c,h} files to here. I changed the base address because it was a simpler method than adding it to every command id. Technically, it does not make a difference because you either obscure the Base ID or the Command ID. So adding a comment why a 0x4000 offset is there makes alot more sense than asking why the offset is there in the first place. |
Thanks Ben, since you did implement these, then there was no need to include FLIR's copyright info. That was the question! That being said, it sounds like you might as well just pull in the files from the purethermal firmware as they came from FLIR. Maybe the power on command will behave better? If you want to submit a new pull request that includes those files I'll merge it in. |
I tried. Unfortunately, the purethermal1 firmware does exactly the same as my code did, i.e. write 0x0 to the power register. In the Software IDD it says: |
Only Power down and Power on have been tested.
Power down works, Power on does not work.