This is an example review we did for our intern who is designing a Multi Battery Charger for Frogwatch
Algemeen
Opamp
We kunnen het beste op zoek naar een zgn 'rail to rail' opamp. Bijvoorbeeld TLV9001 . Het symbool daarvoor zit al in kicad libraries.
Fuel Gauge
Q6+R19 zijn niet echt nodig, dit bespaart maar ~6uA volgens datasheet. Ik zou het weglaten (SW ongebruikt laten).D2 zit verkeerd om. Dit is een zener diode (die laat vanaf een bepaalde spanning stroom door in 'omgekeerde richting').
De BZX84C4V7 lijkt me een geschikte variant (specifieker part nummer voor de 4.7V variant).DNP weerstand van +3.3V naar U2 pin 6 plaatsen. Mocht het nodig zijn kunnen we daar met het prototype een 0-ohm solderen (in plaats van regulator circuit Q5/D2/R18).5V DC-DC
[x] 5V DC/DC: aansluitschema even nakijken. Volgens mij is de voltage divider voor de FB pin niet juist. Vergelijk even met datasheet 9.2

[x] Vin condensatoren: als C18-C20 inderdaad in 0603 verkrijgbaar zijn kan C21 wat mij betreft weg (zo niet C18-C20 footprints updaten en in layout de kleinste footprint het dichtst bij U3 plaatsen).
Battery charger
[x] Vref is een output, dus niet aan 3.3v verbinden (zie dat dit een klein foutje in het symbool is, vref zou ideaal gezien als power output gedefinieerd horen te zijn)
[x] PG pin zal in praktijk altijd laag zijn, dus D5 kan uiteindelijk misschien weggelaten worden. Eventueel laten zitten voor nu (debug?)
[x] aansluitschema even nakijken. Vergelijk met datasheet 10.2.2 of 10.2.1 en let op de SRN, SRP lijnen en component RSR1:
Ik zou in het schema net even meer ruimte pakken (zet hem alvast in een subsheet) en de componenten wat overzichtelijker proberen te plaatsen zoals in de datasheet.
[x] CE pin als het kan aan de overige microcontroller GPIO's hangen ipv aan vref. dan hebben we nog de optie om in software charger(s) aan/uit te zetten. Als er niet genoeg pinnen zijn kun je ze ook per 2 packs delen.
Connectors in overleg nog aanpassen, zie issue #2
Charger
CE. Volgens mij hoort die bij VREF:

Battery connector
A_DIAG1 zit nu met 10K aan ground. Dit wordt een voltage divider samen met de 2M serieweerstand in de pack, dus weglaten!
Voor EMC zou je wel een kleine condensator (bijv 22-50pF,) kunnen toevoegen, maar ik heb nog een beter idee:Nog beter: ik bedenk me dat ESD beveiliging op de batterij connector een goed idee is, omdat de gebruiker die kan aanraken en/of er statische zaps kunnen ontstaan bij het in/uit pluggen.
Suggestie: component Power_Protection:ESDA6V1-5SC6 (in kicad library)
Dit component heeft als bijvangst ook ongeveer 50pF aan condensator, dus deze kan zal zowel als EMC filter als als ESD beveiliging werken.
[x] Ik zou zowel Tsense als A_DIAG1 naar 1 van de pinnen van de ESD chip verbinden (kies afhankelijk van wat fijn is in layout).

[x] BAT+ en BAT- voorzien van een 0603 condensator footprint (10nF of evt DNP) naar GND.
24V power connector Dit moeten we nog uitkiezen, houd voldoende ruimte in je layout. Denk dat er ook nog een zekering of input filter bij komt
Het kan handig zijn om wat testpunten toe te voegen.
In het schema kun je een 'testpoint' symbool toevoegen op signalen die je tijdens verificatie metingen zou willen nameten. Kijk maar even waar het in de layout nog past, hoeft niet op elk signaal. De simpelste footprint die ik vaak gebruik is TestPoint:TestPoint_Pad_D1.0mm.
6x3A * 8.4V / (24V * 90% efficiency) ?J1. Je kunt deze footprint pakken: itter_Footprints:Fuse_2410_6125Metric. Component kan eentje uit de Bel Fuse 0679H0250 serie worden (waarde iets boven maximum stroom kiezen)GND verbonden moet worden (zeer uitzonderlijk):CSN. Deze als enige aan de board ground verbindenCSPL/GND/EP): allemaal samen aan BAT- verbinden.C92) tussen REG en BAT-
BAT- track korter maken.BAT- mag wel wat breder. Hier loopt dezelfde stroom als door BAT+!D31) zit al mooi dichtbij de connector, wel zou ik de tracks wat verder weg routen van andere signalen zodat de ESD pulse niet 'overspringt' of stoort. Misschien nog wel het makkelijkst om dit via de onderste laag te doen (connector is toch al through-hole op alle lagen anwezig)
Tip: de pinnen op de ESD-beveiligingschip zijn uitwisselbaar, als het handig uitkomt voor je layout kun j ook andere pinnen gebruiken (hoeft niet perse pin 4/6 te zijn).
Ik zou bijvoorbeeld zoiets doen (als voorbeeld):


Een voorbeeldje zoals ik dit stukje zou doen:
REGN condensator, ideal zou je deze in de layout voorrang geven op het blokje weerstanden (die kunnen ook wel een stukje naar links)
Dingen om even na te lopen na alle overige wijzigingen:
[x] voldoende GND vias
[x] silk screen: designators weglaten of netjes zetten
[x] silk screen: text/label bij de connectors en een PCB naam+versienummer
[x] 3D footprints toevoegen zodat we de PCB goed in een behuizing kunnen inpassen
Dit is meer een tip, huidige layout is goed genoeg. Ik verwacht dat de laadstroom hierdoor een paar % lager uitvalt dan je zou verwachten maar dat komt niet zo precies.
Misschien wel leerzaam want dit komt vaak voor: het meten van stromen dmv een serieweerstandje. Als je dit nauwkeurig wilt doen kun je het beste 'kelvin style' doen (die term vind je ook vaak in datasheets etc terug). Dit is iets dat je niet makkelijk in het schema kunt aangeven dus iets om altijd even extra op te letten bij de layout.
Als voorbeeld: de oplader chip meet de laadstroom door de spanningsval over RSR1 te meten. De stroom loopt in de richting van de blauwe pijlen, en ziet dus behalve RSR1 ook een beetje weerstand van het koper: in principe zul je dus op elke plek op de BAT+ track op die route een net andere spanning meten.
Het meest nauwkeurig is om een apart paar tracks (in groen aangegeven) te routen direct vanaf de weerstand naar de chip, zonder deze bijv. te delen met het voltage feedback netwerk: