I'm making a portable speaker setup that can be powered and charged at the same time by a regular 19.5v laptop charger. I'm kinda noob with electronics, so I'm not sure if this setup will work as expected. Anyway, the system runs on 4s 18650 batteries, the (protected) batteries are charged via a stepdown converter to 16.8v@2A (suitable for charging). There is a diode to prevent the current from bypassing this buck converter when charging. The other diode is to prevent current coming from the battery to protect the buck charger (I'm not sure if this is necessary though). Does this circuit look alright? will it work? Any suggestions, and/or improvements are much appreciated. Thanks in advance!
Here's the schematic:

Here's the schematic:
