Refactor sysfs usage in HAL about xrt

[stsoe]

I agree, this looks like some dangerous implicit assumptions about sysfs. Absolutely should be factored into one access point of some sort.

[maxzhen]

I'll clean it up.
What is the danger you're seeing here other than duplicate code?

[rradjabi]

Max I’m working on it. This is why we need the ability to assign issues. Right now you need Write permission to do this.

…

Sent from my iPhone On Jul 18, 2018, at 5:25 PM, Max Zhen <[email protected]<mailto:[email protected]>> wrote: EXTERNAL EMAIL I'll clean it up. What is the danger you're seeing here other than duplicate code? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#104 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/Ae94RJDhGUCdbgdNBk0Qc_MY-MKYPb3eks5uH9H2gaJpZM4VVgl_>.

[maxzhen]

Oops, I just removed myself from the assignee. Maybe all of us, in this team, should have write permission?

[sonals]

Please use C++ templatized methods to modularize this code. The same templatized function should be used to read data types of different types and will be type safe.

[hcneema]

post 2018.2_XDF

[maxzhen]

Implemented in PR#399.