I would suggest to add an activation function after self.fc1, since otherwise you are basically just applying a single linear layer.
Besides that your architecture looks alright. Do you see any other issues using it?