Re: [U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic
Hello Stefano,
On Mon, 2019-06-10 at 11:40 +0200, sbabic@denx.de wrote:
https://source.codeaurora.org/external/imx/uboot-imx cherry picked, styled and merged commits:
- MLK-18387 pmic: Add pmic driver for BD71837: e9a3bec2e95a
- MLK-18590 pmic: bd71837: Change to use new fdt API: acdc5c297a96
Signed-off-by: Ye Li ye.li@nxp.com Signed-off-by: Matti Vaittinen matti.vaittinen@fi.rohmeurope.com Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot-imx, master, thanks !
Thanks for merging the patch :] Do you think the 2/2 could also be taken in?
Best regards Matti Vaittinen
Hi Maitti,
On 12/06/19 12:24, Vaittinen, Matti wrote:
Hello Stefano,
On Mon, 2019-06-10 at 11:40 +0200, sbabic@denx.de wrote:
https://source.codeaurora.org/external/imx/uboot-imx cherry picked, styled and merged commits:
- MLK-18387 pmic: Add pmic driver for BD71837: e9a3bec2e95a
- MLK-18590 pmic: bd71837: Change to use new fdt API: acdc5c297a96
Signed-off-by: Ye Li ye.li@nxp.com Signed-off-by: Matti Vaittinen matti.vaittinen@fi.rohmeurope.com Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot-imx, master, thanks !
Thanks for merging the patch :] Do you think the 2/2 could also be taken in?
The thing is that I did not know why patches were assigned to me. 1/2 was just defines and I decided to merge it instead of postpone it. 2/2 is the driver, and should flow into power tree (maintainer is Jaehoon), even if this is thought for i.MX8. It is enough to me to get an ACK by Jaehoon to merge into u-boot-imx, if Jaehonn agrees.
Best regards, Stefano Babic
Hello Jaehoon and Stefano,
On Wed, 2019-06-12 at 12:58 +0200, Stefano Babic wrote:
Hi Maitti,
On 12/06/19 12:24, Vaittinen, Matti wrote:
Hello Stefano,
On Mon, 2019-06-10 at 11:40 +0200, sbabic@denx.de wrote:
https://source.codeaurora.org/external/imx/uboot-imx cherry picked, styled and merged commits:
- MLK-18387 pmic: Add pmic driver for BD71837: e9a3bec2e95a
- MLK-18590 pmic: bd71837: Change to use new fdt API:
acdc5c297a96 Signed-off-by: Ye Li ye.li@nxp.com Signed-off-by: Matti Vaittinen < matti.vaittinen@fi.rohmeurope.com> Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot-imx, master, thanks !
Thanks for merging the patch :] Do you think the 2/2 could also be taken in?
The thing is that I did not know why patches were assigned to me. 1/2 was just defines and I decided to merge it instead of postpone it. 2/2 is the driver, and should flow into power tree (maintainer is Jaehoon), even if this is thought for i.MX8. It is enough to me to get an ACK by Jaehoon to merge into u-boot-imx, if Jaehonn agrees.
Allright. The BD71837 and BD71847 are indeed intended to be used for powering the i.MX8. I don't expect them to be used too much for other SoCs - so that might be the reason the patches were assignrd to you in first place. I know few people who are using the BD71837 and would be happy seeing the driver in u-Boot ;) So Jaehoon, how do you see this? Is the driver Ok to go in Stefano's tree? Should I take some action like resending the 2/2?
Best Regards Matti Vaittinen
Hello Jaehoon,
On Thu, 2019-06-13 at 11:33 +0300, Matti Vaittinen wrote:
Hello Jaehoon and Stefano,
On Wed, 2019-06-12 at 12:58 +0200, Stefano Babic wrote:
Hi Maitti,
On 12/06/19 12:24, Vaittinen, Matti wrote:
Hello Stefano,
On Mon, 2019-06-10 at 11:40 +0200, sbabic@denx.de wrote:
https://source.codeaurora.org/external/imx/uboot-imx cherry picked, styled and merged commits:
- MLK-18387 pmic: Add pmic driver for BD71837: e9a3bec2e95a
- MLK-18590 pmic: bd71837: Change to use new fdt API:
acdc5c297a96 Signed-off-by: Ye Li ye.li@nxp.com Signed-off-by: Matti Vaittinen < matti.vaittinen@fi.rohmeurope.com> Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot-imx, master, thanks !
Thanks for merging the patch :] Do you think the 2/2 could also be taken in?
The thing is that I did not know why patches were assigned to me. 1/2 was just defines and I decided to merge it instead of postpone it. 2/2 is the driver, and should flow into power tree (maintainer is Jaehoon), even if this is thought for i.MX8. It is enough to me to get an ACK by Jaehoon to merge into u-boot-imx, if Jaehonn agrees.
Allright. The BD71837 and BD71847 are indeed intended to be used for powering the i.MX8. I don't expect them to be used too much for other SoCs - so that might be the reason the patches were assignrd to you in first place. I know few people who are using the BD71837 and would be happy seeing the driver in u-Boot ;) So Jaehoon, how do you see this? Is the driver Ok to go in Stefano's tree? Should I take some action like resending the 2/2?
Just a gentle ping :) Should I resend the 2/2?
Br, Matti Vaittinen
participants (2)
-
Stefano Babic -
Vaittinen, Matti