[U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic

[U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic

[sbabic at denx.de]

> 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 !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

[U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic

[Vaittinen, Matti]

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 at 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 at 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

[U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic

[Vaittinen, Matti]

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 at 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 at 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

[U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic

[Stefano Babic]

Hi Maitti,

On 12/06/19 12:24, Vaittinen, Matti wrote:
> Hello Stefano,
> 
> On Mon, 2019-06-10 at 11:40 +0200, sbabic at 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



-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

[U-Boot] [PATCH v4 1/2] regulator: bd71837: copy the bd71837 pmic

[Vaittinen, Matti]

Hello Stefano,

On Mon, 2019-06-10 at 11:40 +0200, sbabic at 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