Earlier fix removed unused phase, but that changed the values for other phases. Since these are exposed to userspace through ppdev, it is safer not to change them. Restore the unused phase value.
Unified · Split
| @@ -236,12 +236,14 @@ struct pardevice { · 235 unmodified lines | ||
| 236 | 236 | |
| 237 | 237 | /* IEEE1284 information */ |
| 238 | 238 | |
| 239 | −/* IEEE1284 phases */ | |
| 239 | +/* IEEE1284 phases. These are exposed to userland through ppdev IOCTL | |
| 240 | + * PP[GS]ETPHASE, so do not change existing values. */ | |
| 240 | 241 | enum ieee1284_phase { |
| 241 | 242 | IEEE1284_PH_FWD_DATA, |
| 242 | 243 | IEEE1284_PH_FWD_IDLE, |
| 243 | 244 | IEEE1284_PH_TERMINATE, |
| 244 | 245 | IEEE1284_PH_NEGOTIATION, |
| 246 | + IEEE1284_PH_HBUSY_DNA, | |
| 245 | 247 | IEEE1284_PH_REV_IDLE, |
| 246 | 248 | IEEE1284_PH_HBUSY_DAVAIL, |
| 247 | 249 | IEEE1284_PH_REV_DATA, |