Skip to content

config.sonic: Build driver for sensor INA238#585

Open
RayWang-micas wants to merge 1 commit into
sonic-net:masterfrom
RayWang-micas:master
Open

config.sonic: Build driver for sensor INA238#585
RayWang-micas wants to merge 1 commit into
sonic-net:masterfrom
RayWang-micas:master

Conversation

@RayWang-micas
Copy link
Copy Markdown

Include the ina238 driver

@RayWang-micas RayWang-micas requested a review from a team as a code owner June 1, 2026 08:41
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Jun 1, 2026

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: RayWang-micas / name: RayWang-micas (9ca1985)

@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@RayWang-micas RayWang-micas changed the title Update config.sonic Enable ina238 Jun 1, 2026
Copy link
Copy Markdown
Contributor

@paulmenzel paulmenzel left a comment

Choose a reason for hiding this comment

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

Please make the commit subject/title more specific. Maybe:

config.sonic: Build driver for sensor INA238

Then amend the commit message, and document on what system you need this, and paste the Linux log lines of the driver, that it is actually used and working.

Do the same for the merge/pull request description.

@paulmenzel
Copy link
Copy Markdown
Contributor

And of course welcome to SONiC, and thank you for your contribution. It’s much appreciated.

@RayWang-micas RayWang-micas changed the title Enable ina238 config.sonic: Build driver for sensor INA238 Jun 1, 2026
Include the ina238 driver

Signed-off-by: RayWang-micas <ray@micasnetworks.com>
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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.

3 participants