将 max31855 热电偶的 bitbang spi 驱动程序添加到数字转换器… #55

不幸的是，我们的 .comp 构建系统中的一个烦人的问题阻止了该组件在 uspace 配置上构建。我猜你是在 RTAI 机器上开发？

我将 comp 移动到 src/hal/components，在那里它将为 rtai 和 uspace 配置构建。如果 buildbot 没有揭示任何其他问题，我会把它推给 master。它现在在 sleepybishop-max31855 分支中：https ://github.com/LinuxCNC/linuxcnc/commits/sleepybishop-max31855

感谢您的公关。

Seb 的分支在我们的 CI（buildbot）系统中的某些系统上遇到错误：http:
//buildbot.linuxcnc.org/buildbot/builders/1301.rip-precise-rtai-i386/builds/3408/steps/compile/日志/stdio
http://buildbot.linuxcnc.org/buildbot/builders/1902.clang-wheezy-rtai-i386/builds/1351/steps/compile/logs/stdio

hal/components/max31855.comp: In function ‘read_celsius’:
hal/components/max31855.comp:70:12: error: ‘NAN’ undeclared (first use in this function)
hal/components/max31855.comp:70:12: note: each undeclared identifier is reported only once for each function it appears in
...

C99nan在<math.h> 中指定了该函数，但我们在<rtapi_math.h> 中不支持它。看起来在那里添加是微不足道的，只是使用__builtin_nan()，未经测试：

// add to the if defined(__KERNEL__) block
#define nan(x) __builtin_nan(x)

然后更改您的代码以使用nan("")而不是NAN. 我已经做到了这一点，并将分支重新定位在我们当前的 master 之上，让 buildbot 再次咀嚼它。

你是说在 rtapi_math.h 中？

max31855 组件同时使用 isnan() 和 NAN 宏，isnan() 似乎在 rtapi_math.h 中处理，NAN 宏不是，gnu c 似乎将此宏定义为：
#define NAN __builtin_nanf(""):

我对这两种方式都没有偏好，但在 rtapi_math.h 中类似地定义它是否有意义？

乐于做出任何需要的改变。

是的，rtapi_math.h。之后，下一个问题是 exp() 函数。我会仔细研究一下。

http://buildbot.linuxcnc.org/buildbot/builders/1401.rip-wheezy-rtai-i386/builds/2025/steps/compile/logs/stdio

hal/components/max31855.comp: In function ‘read_celsius_adjusted’:
hal/components/max31855.comp:129:5: error: implicit declaration of function ‘exp’ [-Werror=implicit-function-declaration]

对于 exp() 函数，类似的问题，pow() 已定义但 exp() 未定义，我认为
extern double exp(double);
可以解决它

是的，现在基于 rtai 的系统很高兴。 @SebKuzminsky在将其合并到 master 之前还需要发生什么？

合并是@mozmck电话，他是 2.7+1 的发布经理，但我觉得不错。

我唯一的评论是，是否应该将 nan() 和 exp() 添加到测试/实时数学中？还是会在我们的 RTAI 平台的某些子集上出现问题？

75c2329之后我没有保留。

@mozmck在 IRC 上给出了 OK，所以现在它被合并到我们的 master 分支中。它很快就会出现在github上。