Fix make empty variable warnings#3260
Conversation
…with proper conditional. Remove t_on/t_off/t_p dependencies because they got component'ized and renamed in 3775816. Comment out the terminal clutter option --warn-undefined-variables.
|
I wonder why those (mainly) drivers do not exist in uspace? They seem to be a mix of ISA and PCI boards. |
|
It looks like Motenc-Lite is still a current product, though it looks like an expensive choice. |
That is indeed a good question. But, it looks like this code is 15+ years (and some older). Has anybody seen or heard someone complaining that there was no uspace support for these boards?
Lack of testing would be an obvious reason. But the time this code has been untouched also suggests that nobody has needed it and nobody cared. I imagine that using these drivers is for someone wanting to "fix" an existing install, but to use it as new with the cost you mention... And even then, when has it last been tested as a kernel module? Therefore, I see no point in updating the code for uspace. That leaves us with fixing the make problem, which the warnings indicated. |
|
If the code is 15 years old then I think that it is not related to uspace (which is younger), but rather to the old "sim" build target which explicitly excluded hardware drivers. |
|
Well, for example the hal_motenc.c driver (for the one you mentioned) was added between 21 and 19 years ago. The last fix was fixing a PCI API deprecation 13 years ago. With uspace, the commit 1d6c0d0 just moved the 'sim' exclusions into 'uspace' exclusions. I guess nobody cared about these drivers being ported to uspace functionality. That said, this PR does not change current behavior. I merely fixes the rules such that no warnings are generated. If someone wants these drivers to become active in uspace, they are welcome to do the port, but that is a different problem. I don't have/run RTAI, so I can't comment on the specific problems on that build. But, maybe it should receive some caring attention, reading the required out-of-tree efforts to make it work. But this is also beyond the scope of this PR. |
Indeed, I was just wondering if we need to look at this. I will merge this, and then raise an issue. |
Fixed in this PR: