camcontrol(8): remove unnecessary CCB zeroing

[trasz]

Summary

After 3e404b8c53d, cam_getccb(3) clears the returned CCB, making
a number of calls to CCB_CLEAR_ALL_EXCEPT_HDR(3) unnecessary.

[trasz]

Squash.

[trasz]

(Tinderboxed.)

[imp]

All but one of these are easily seen to be redundant now.

sbin/camcontrol/camcontrol.c

1828

This one I'm not sure of since I've not traced all callers of ata_do_pass_16

[imp]

I'd commit all but the flagged one.
The flagged one, if someone can confirm my code tracing, changes things. Don't know if the change is a breakage or just a benign change, but I think it requires additional study (and maybe a comment about why it's needed, or the flow needs to change to call it after the flagged ata_do_identify, but even then I've not chased down all branches.

sbin/camcontrol/camcontrol.c

1828

I'm pretty sure this isn't good.

atahpa() gets a ccb
Then calls ata_do_identify (which fills in the ccb)
And later atahpa can call atahpa_password when action is set to ATA_HPA_ACTION_SET_PWD
which calls ata_do_cmd
which will call ata_do_pass_16 if ata_try_pass_16
which used to call CCB_CLEAR_ALL_EXCEPT_HDR to zero everything out, but now does not do so after the identify already stomped on things in this ccb. Don't know if the stomping is OK or not, but this is a change.

[trasz]

Add back chunk that's required, as advised by imp@.

[imp]

Filter the unrelated things out, please, and we're good.

sbin/camcontrol/camcontrol.c

114

This is a good change, but unrelated.

250–251

ditto

10236

ditto