Skip to content

Conversation

@ivangilmercano
Copy link
Contributor

Pull Request Description

The ADP1055 is a flexible, feature-rich digital secondary side controller that targets ac-to-dc and isolated dc-to-dc secondary side applications. The ADP1055 is optimized for minimal component count, maximum flexibility, and minimum design time. Features include differential remote voltage sense, primary and secondary side current sense, pulse-width modulation (PWM) generation, frequency synchronization, redundant OVP, and current sharing.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

@buha
Copy link
Contributor

buha commented Jan 27, 2025

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

dbogdan
dbogdan previously approved these changes Feb 3, 2025
@dbogdan
Copy link
Contributor

dbogdan commented Feb 3, 2025

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@kister-jimenez
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ivangilmercano ivangilmercano force-pushed the temp/dev/adp1055 branch 2 times, most recently from c4929b3 to 62cb38f Compare February 5, 2025 06:12
@CiprianRegus
Copy link
Contributor

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ivangilmercano ivangilmercano force-pushed the temp/dev/adp1055 branch 2 times, most recently from b026199 to 92078c5 Compare February 7, 2025 02:41
@kister-jimenez
Copy link
Collaborator

Can you change your commit summary and body to follow this formatting?:
https://github.com/analogdevicesinc/no-OS/pull/2406/commits

@kister-jimenez
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

"Disabled", "Enabled"
};

static const char *const adp1055_freq_avail[] = {
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these all the available frequencies?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, I just listed some of the available frequencies for adp1055. I noticed that in adp1050, not all the frequencies are listed so I figured that can be done in this case as well.

@kister-jimenez
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ivangilmercano ivangilmercano force-pushed the temp/dev/adp1055 branch 4 times, most recently from 46e8edf to 855958b Compare February 23, 2025 01:59
@cencarna
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@buha buha merged commit 0788ebb into analogdevicesinc:main Feb 24, 2025
14 checks passed
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.

8 participants