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 driver for LT717x #2658

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

Conversation

csarip
Copy link

@csarip csarip commented Nov 28, 2024

PR Description

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 Analog Devices LT7170, LT7170-1, LT7171, and LT7171-1 Single- or
Dual-Phase Step-Down Silent Switcher.

Signed-off-by: Cherrence Sarip <[email protected]>
LT7170 20A, 16V Single- or Dual-Phase Step-Down Silent Switcher with
PMBus Interface
LT7170-1 20A, 16V Single- or Dual-Phase Step-Down Silent Switcher with
PMBus Interface
LT7171 20A, 16V PolyPhase Step-Down Silent Switcher with
PMBus Interface
LT7171-1 20A, 16V PolyPhase Step-Down Silent Switcher with
PMBus Interface

Signed-off-by: Cherrence Sarip <[email protected]>
Add entries for the LT7170 driver.

Signed-off-by: Cherrence Sarip <[email protected]>
@csarip csarip marked this pull request as ready for review December 9, 2024 10:01
@csarip
Copy link
Author

csarip commented Dec 9, 2024

V2:

Changes:

  • Fixed minor formatting issues

@csarip
Copy link
Author

csarip commented Dec 9, 2024

This error has caused some checks to fail.

image

@csarip
Copy link
Author

csarip commented Dec 9, 2024

This LT717x driver includes unreleased parts with similar functionality to other devices in the family. I'll just add them after this PR is approved.

@kister-jimenez
Copy link
Contributor

Can you check if this can be combined to the LT7182s? Both have a lot of similarities.

@kister-jimenez
Copy link
Contributor

Can you check if this can be combined to the LT7182s? Both have a lot of similarities.

Got a reply from maintainers. There is no particular reason why LT7182s was not merged with LTC2978. So maybe consider merging instead to LTC2978.

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