Skip to content

fby35: hd: Adjust LTC4282 power and current#558

Closed
Yi-Shum wants to merge 3 commits into
facebook:mainfrom
Yi-Shum:fby3.5-hd-adjust_ltc4282_power_and_current
Closed

fby35: hd: Adjust LTC4282 power and current#558
Yi-Shum wants to merge 3 commits into
facebook:mainfrom
Yi-Shum:fby3.5-hd-adjust_ltc4282_power_and_current

Conversation

@Yi-Shum

@Yi-Shum Yi-Shum commented Sep 14, 2022

Copy link
Copy Markdown
Collaborator

Summary:

  • Adjust the power and current of the LTC4282.

Test Plan:

  • Build code: PASS
Summary:
- Adjust the power and current of the LTC4282 by a factor of 0.96.

Test Plan:
- Build code: PASS
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 14, 2022
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@GoldenBug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.


bool post_ltc4282_cur_read(uint8_t sensor_num, void *args, int *reading)
{
sensor_val *sval = (sensor_val *)reading;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check reading to see if it's NULL.

There's a common macro we've added for that which also logs errors.

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.

OK, will use the common macro to check if "reading" is NULL.

return true;
}

bool post_ltc4282_pwr_read(uint8_t sensor_num, void *args, int *reading)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check reading to see if it's NULL.

There's a common macro we've added for that which also logs errors.

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.

OK, will use the common macro to check if "reading" is NULL.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@Yi-Shum has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@Yi-Shum has updated the pull request. You must reimport the pull request before landing.

@Yi-Shum

Yi-Shum commented Sep 15, 2022

Copy link
Copy Markdown
Collaborator Author

Hi @GoldenBug,
I has modify it according to your comment.
Please review it again, thanks.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@GoldenBug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Sep 15, 2022
Summary:
- Version commit for Yv3.5 HalfDome BIC oby35-hd-2022.37.01.

Dependency: #558

Pull Request resolved: #568

Test Plan:
- Build code: Pass
- Get BIC version: Pass

Log:
root@bmc-oob:~# fw-util slot1 --version bic
SB Bridge-IC Version: oby35-hd-v2022.37.01

root@bmc-oob:~# bic-util slot1 0x18 0x1
00 80 11 05 02 BF 15 A0 00 00 00 00 00 00 00

Reviewed By: garnermic

Differential Revision: D39551638

Pulled By: GoldenBug

fbshipit-source-id: 15a146beaff0c0ac69524e2d64817a1447c339ca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

3 participants