Skip to content

Conversation

@ranranff
Copy link

@ranranff ranranff commented Sep 6, 2024

Add initial driver for the AXP2101 PMIC
Support:

  • DCDC1-5
  • LDOA1-A4, LDOB1-B2, LDOC1, LDOD1-D2

Change-Id: I3f93900423ec592a44d7e80a4353672c63cfa3a4

@ranranff
Copy link
Author

ranranff commented Sep 6, 2024

Sorry, the previous one has been closed by a mistake. #77804
I have removed the duplicated properties.

Copy link
Member

Choose a reason for hiding this comment

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

Please align the name to the datasheet notation.
Here, ldoa1-> aldo1.
The others so on.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for review. Fixed.

Copy link
Member

@soburi soburi Oct 19, 2024

Choose a reason for hiding this comment

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

It is also needed to fix dts and commit messages.

Copy link
Member

Choose a reason for hiding this comment

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

Please create a header file in include/zephyr/drivers/mfd/
and move these definitions in the same as done in AXP192.

@soburi
Copy link
Member

soburi commented Oct 14, 2024

@ranranff

I created the LED control part. Please refer to it if you are interested.

#79765

@soburi
Copy link
Member

soburi commented Oct 17, 2024

@ranranff
Merge branch 'zephyrproject-rtos:main' into main

Please use rebase to apply the newest main contents.
The main branch can't have a merge commit.

@ranranff ranranff force-pushed the main branch 2 times, most recently from 11f3ee0 to eedb7a4 Compare October 17, 2024 12:08
Add initial driver for the AXP2101 PMIC
Support:
  - DCDC1-5
  - LDOA1-A4, LDOB1-B2, LDOC1, LDOD1-D2

Signed-off-by: Deng Baoan <[email protected]>
@ranranff ranranff force-pushed the main branch 2 times, most recently from 11f3ee0 to 7aebc00 Compare October 17, 2024 13:36
LOG_INSTANCE_PTR_DECLARE(log);
};

/*! ----------------------------------------------------------------------------------------*/
Copy link
Member

Choose a reason for hiding this comment

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

reg:
required: true

vbus-enable:
Copy link
Member

Choose a reason for hiding this comment

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

Is it currently used? If no, remove it.

compatible: "x-powers,axp2101"

include: i2c-device.yaml

Copy link
Member

Choose a reason for hiding this comment

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

Please define bus. Such like,
bus: axp2101

LOG_INST_DBG(config->log, "[0x%02x]=0x%02x mask=0x%02x", config->desc->enable_reg,
config->desc->enable_val, config->desc->enable_mask);

/* special case for LDOIO0, which is multiplexed with GPIO0 */
Copy link
Member

Choose a reason for hiding this comment

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

Is this comment correct?

@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Jan 30, 2025
@github-actions github-actions bot closed this Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants