Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for LTM4673 #2660

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cencarna
Copy link

@cencarna cencarna commented Dec 2, 2024

PR Description

This adds support for LTM4673 to the LTC2978 PMBus client driver which contains
support to similar μModule regulators. The LTM4673 is a quad output, dual 12A and
dual 5A, switching mode DC/DC step-down μModule regulator.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly (if there is the case)

Add LTM4673 to supported devices of LTC2978. Unlike other LTM46xx
devices, LTM4673 is a μModule regulator that outputs to four channels.

Signed-off-by: Cedric Encarnacion <[email protected]>
Add LTM4673 to compatible devices in LTC2978.

Signed-off-by: Cedric Encarnacion <[email protected]>
Add support for LTM4673. The LTM4673 is a quad output, dual 12A and dual
5A, switching mode DC/DC step-down μModule regulator integrated with
4-channel power system manager.

This adds only the chip id, the checks for the manufacturer special id,
and the relevant attributes for the device's pmbus_driver_info.
The device does not support clear peaks.

Signed-off-by: Cedric Encarnacion <[email protected]>
Copy link
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me. Please go ahead and send it upstream

@@ -227,6 +235,7 @@ Description
- LTM2987 is a 16-channel Power System Manager with two LTC2977 plus
- additional components on a single die. The chip is instantiated and reported
- as two separate chips on two different I2C bus addresses.
- LTM4673 is a dual 12A and dual 5A μModule regulator.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not likely a problem but I think you squash this change with the patch adding the driver.

@nunojsa
Copy link
Collaborator

nunojsa commented Jan 22, 2025

@cencarna, Is this sent?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants