please correct my circuit design. mainly the power supply factors.

 

What is it?

 

Nothing looks "wrong" on the face of it. I did not check the connections to the breakout boards. Is the crystal 20 MHz and the load capacitors 22 pf?

 

crystal is 16mhz and load cp is 22pf. And input voltage is 12V 3A does this much of voltage and current suites atmega328p -pu. Mosfet sometimes does not switch On.

 

The input voltage and available current are irrelevant to the processor. But what if a motor stalls and draws a lot of current, will the voltage stay high enough to keep the processor alive?

I think there should be a good-sized electrolytic capacitor on the input side of the voltage regulator--I'd use something like 470uF.

Is there some good reason to have your controls connected to Vcc? It's more common to have the pushbuttons connected to Gnd, and if you did that you could use internal pullup resistors in the processor, and save a couple of components.

How is the FTDI chip getting powered? The diagram shows its Vcc pin unconnected. And is that circuit with C2 and R6 really doing the right thing?

 

I have designed this circuit on basis of this link
https://www.circuito.io/app?compone...7,641455,942700,942700,956215,1000328,2631981

main component spec:
1. motor with 12v 3.2 a
2. water flow sensor yf-s201
3. bluetooth hc 05
4. lcd 20x4 I2C
5 atmega328p -pu

circuit flow goes in sequese like this:
1.push button_1 - motor_1 - motor_2 - motor_3- motor_4 - and cycle repeats as per the program.
note: In above sequence motor_1 and motor_2 ON/OFF is depend on water flow sensor
2.push button button_2 - motor_5

when I build circuit as per the link then the mosfet doest switch on .

I need some simple circuit to execute these tasks.

some good reason to have your controls connected to Vcc? It's more common to hav

 

for clear view of schematic

 

n channel mosfet switch on for miloseconds, its not holding up.