Skip to content

Set SRAM LDO off on power down - additional power gating on D0->D3 path.#500

Merged
lgirdwood merged 1 commit intothesofproject:masterfrom
lbetlej:ldo_off_on_power_down
Oct 25, 2018
Merged

Set SRAM LDO off on power down - additional power gating on D0->D3 path.#500
lgirdwood merged 1 commit intothesofproject:masterfrom
lbetlej:ldo_off_on_power_down

Conversation

@lbetlej
Copy link
Copy Markdown
Collaborator

@lbetlej lbetlej commented Oct 19, 2018

Added assembly macros to control state of LDO for LP and HP SRAM.
Macros are used during power down procedure to properly handle LDO state.

Signed-off-by: Lech Betlej lech.betlej@linux.intel.com

Copy link
Copy Markdown
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

Minor change to make it obvious for anyone trying to include this in C sources.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

best to say #error "header can only be used by assembly sources"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

changes applied to the PR

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

PR updated .

@lbetlej lbetlej force-pushed the ldo_off_on_power_down branch 3 times, most recently from 05c4fdc to cd8c78d Compare October 22, 2018 17:42
@lbetlej lbetlej changed the title Set SRAM LDO off on power down - addtional power gating on D0->D3 path. Set SRAM LDO off on power down - additional power gating on D0->D3 path. Oct 22, 2018
@lbetlej lbetlej force-pushed the ldo_off_on_power_down branch from cd8c78d to dfcf50c Compare October 24, 2018 08:21
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

github is showing a warning about no newline at end of file here.

Comment thread src/platform/apollolake/power_down.S Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

indentation looks off here ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Changes applied.

Added assembly macros to control state of LDO for LP and HP SRAM.
Macros are used during power down procedure to properly handle LDO state.
LDO handling sequence in power down procedure is specific for cAVS 1.5.

Signed-off-by: Lech Betlej <lech.betlej@linux.intel.com>
@lbetlej lbetlej force-pushed the ldo_off_on_power_down branch from dfcf50c to d5c3828 Compare October 24, 2018 13:06
@lgirdwood lgirdwood merged commit 8db46b4 into thesofproject:master Oct 25, 2018
@lbetlej lbetlej deleted the ldo_off_on_power_down branch October 15, 2019 11:19
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