s/Option<TxSignatures>/() for return from provide_holder_witnesses

In a following commit we change the state at which a V2 channel is
promoted to a `Channel` (after monitor persistence) to avoid a crash
upon reading a `Channel` with not corresponding monitor persisted.
This means that we will have no need for an optional (based on signing
order) `TxSignatures` being returned when `provide_holder_witnesses` is
called.

The above is also the reason for removing the
`received_commitment_signed` and signing order checks in the body of
`provide_holder_witnesses`.

These checks will only be important when we actually contribute inputs.
Currently, we don't so we always send `tx_signatures` first in
accrodance with the Interactive Transaction Construction specification:

https://github.com/lightning/bolts/blob/aa5207aea/02-peer-protocol.md?plain=1#L404

Author: dunxen

lightning/src/ln/interactivetxs.rs

@@ -342,7 +342,7 @@ impl InteractiveTxSigningSession {
 	/// unsigned transaction.
 	pub fn provide_holder_witnesses(
 		&mut self, channel_id: ChannelId, witnesses: Vec<Witness>,
-	) -> Result<Option<TxSignatures>, ()> {
+	) -> Result<(), ()> {
 		if self.local_inputs_count() != witnesses.len() {
 			return Err(());
 		}
@@ -354,13 +354,8 @@ impl InteractiveTxSigningSession {
 			witnesses: witnesses.into_iter().collect(),
 			shared_input_signature: None,
 		});
-		if self.received_commitment_signed
-			&& (self.holder_sends_tx_signatures_first || self.counterparty_sent_tx_signatures)
-		{
-			Ok(self.holder_tx_signatures.clone())
-		} else {
-			Ok(None)
-		}
+
+		Ok(())
 	}
 
 	pub fn remote_inputs_count(&self) -> usize {