0
\$\begingroup\$

This is the first circuit I have ever designed. The circuit is altered from the 'Typical Application' of the IP5310 datasheet to allow the type-C port behave as both Input and Output.

I plan on PCB printing this so I'm looking for any feedback on anything I need to be aware of(Issues suggestions).

Here's the link to the IC datasheet: https://www.skytech.ir/DownLoad/File/2381_IP5310.pdf

Edit: I altered section related to the PMOS, because in the datasheet on how to use the IP5310 with a type-C port. In the datasheet, the Chip could only act as output. I altered it so that the transistors allowed the circuit to allow input of voltage to the IC for battery charging

The Circuit

\$\endgroup\$
5
  • 1
    \$\begingroup\$ INFINIYOS KD - Hi, (a) Please edit the question & link to the datasheet containing the original design which you adapted. (b) Please also explain what you changed from that typical application and why. (c) Asking for "any feedback on anything" is very broad and discussions on meta have shown that we don't want such broad review questions (e.g. it can be impossible to choose a "best" answer to accept, if different answers address different parts of the design, and such broad questions can actually put-off people from responding). Can you focus the question to ask about a specific concern? \$\endgroup\$ Commented Aug 8 at 13:53
  • 1
    \$\begingroup\$ why are two same-net lines running in parallel? ... you only need one \$\endgroup\$ Commented Aug 8 at 16:31
  • 1
    \$\begingroup\$ Some tips for the schematic: draw your wires on-grid, leave space for your reference designators and symbol labels so they don't have wires cutting through them, always have power pointing up and ground pointing down, use net labels to avoid spaghetti traces (your CC1/CC2 traces are a clear candidate for that approach), use consistent positioning for components (C1 and C2 could be drawn next to each other in parallel rather than having C1 looping round at the top), don't hide the refdes labels (LED at the bottom), and add function labels next to each LED and switch to explain what they do. \$\endgroup\$ Commented Aug 8 at 20:06
  • 1
    \$\begingroup\$ This looks like Altium, so you should also use the Electrical Rules Check (ERC) feature to detect mistakes. Configure your symbol pins as the correct type (power, input, output, I/O, etc.) and it'll warn you when you bridge things together improperly, like you did here with all the pins on the right side of the IC. If you name each net properly with a net label, it'll also tell you when you accidentally bridge two nets together (multiple name net warning). Naming your nets is also helpful during PCB layout because you can see you're routing USB_CC2 rather than NetU5_B5. \$\endgroup\$ Commented Aug 8 at 20:22
  • 1
    \$\begingroup\$ I don't know what you edited and why, but the errors are still there and the datasheet does include a charge/discharge from Type-C example and you are not following it, and we have no clue why you did not follow the example or did not draw it correctly. \$\endgroup\$ Commented Aug 11 at 11:03

1 Answer 1

Reset to default
1
\$\begingroup\$

It will not work as there are several errors. Even without looking at what the circuit does, the different signal wires are erroneously shorted together.

\$\endgroup\$
3
  • 1
    \$\begingroup\$ R8 is also supposed to be the NTC inside the battery pack, rather than just a regular 100K resistor. \$\endgroup\$ Commented Aug 8 at 20:15
  • \$\begingroup\$ Could You clarify because I followed the Datasheet: skytech.ir/DownLoad/File/2381_IP5310.pdf Though with adjustments to allow dual role of the type c port \$\endgroup\$ Commented Aug 9 at 10:55
  • 1
    \$\begingroup\$ @INFINIYOSKD Look at the chip on the side where LX and CC pins are located - all those red dots mean the wires are connected together. I highly doubt your connections are made in a way you want or what the datasheet says. \$\endgroup\$ Commented Aug 9 at 11:47

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.