maxim-max17048: Update to atopile 0.11.6 and simplify usage example

[nickkrstevski]

Summary

• Update requires-atopile from ^0.10.8 to ^0.11.6 to support latest atopile version
• Simplify usage.ato to remove modules without footprints for cleaner verification
• Update README.md to match the simplified usage example

Test plan

• ato build passes without errors
• ato build --frozen generates correct PCB layout
• ato package verify passes all checks

🤖 Generated with Claude Code

[claude]

Claude finished @nickkrstevski's task —— View job

---

PR Review for maxim-max17048 Package Update

Issues Found: ⚠️ 1 minor typo, otherwise package appears well-structured

Review Checklist:

• Verify PR title format (package name prefix)
• Read package building rules document
• Examine changed files for compliance
• Check usage.ato matches README.md exactly
• Run ato build test (Build commands require approval, but code structure analysis completed)
• Run ato build --frozen test (Would need approval to execute)
• Run ato package verify test (Would need approval to execute)
• Review code quality and functionality
• Check datasheet compliance
• Verify PR focus (single package only)

Issues Found

1. Variable Name Typo (Line 58)

File: maxim-max17048.ato:58
Issue: Variable declared as alter_pullup but should be alert_pullup

alter_pullup = new Resistor  # <- should be "alert_pullup"

This is used for the alert pin pull-up resistor, so the name should be consistent.

Positive Findings

✅ Package Structure & Compliance

• PR Format: Correctly titled with package name prefix "maxim-max17048"
• Single Focus: Only maxim-max17048 files modified, no unrelated changes
• File Structure: Follows required structure (ato.yaml, .ato file, README, usage.ato, parts/)
• Version Update: Correctly updated requires-atopile from ^0.10.8 to ^0.11.6
• Usage/README Match: usage.ato content exactly matches README.md Usage section ✅

✅ Code Quality

• Interfaces: Properly exposes power, I2C, battery_interface, qstrt, and alert pins
• I2C Configuration: Correct address (0x36) with proper pull-up resistors
• Power Supply: Valid voltage range (2.5V to 4.5V) with assertions
• Decoupling: Appropriate bypass capacitors (1µF) for both power and battery
• Pin Connections: All package pins properly connected
• Experimental Features: Correctly uses BRIDGE_CONNECT and FOR_LOOP pragmas

✅ Functionality (Datasheet Compliance)

• MAX17048 Features: Correctly implements fuel gauge functionality
• I2C Address: 0x36 is the correct fixed address for MAX17048
• Power Rails: VDD and battery monitoring properly separated
• Configuration Pins: QSTRT (quick start) and CTG (current gain) properly handled
• Alert Pin: Active-low alert with pull-up resistor implemented correctly

✅ Usage Example

• Minimal Example: Shows essential connections (power, I2C, battery)
• Clear Documentation: Good docstrings explaining pin purposes
• Practical: Demonstrates real-world usage pattern

Recommendations

1. Fix Typo: Change alter_pullup to alert_pullup in lines 58-61 for consistency
2. Build Testing: The build commands (ato build, ato build --frozen, ato package verify) should be run to confirm no runtime issues, but package structure suggests they should pass

Summary

This is a well-executed package update that follows atopile best practices. The main change from ^0.10.8 to ^0.11.6 appears to be handled correctly, with simplified usage that removes modules without footprints for cleaner verification. The only issue is a minor variable naming typo that should be fixed for code clarity.

Recommendation: Approve after fixing the typo

---