Re: [PATCH] PCI: imx6: Add support for MX6SX LDO PCIE domain regulator

[Christoph Fritz <chf.fritz@googlemail.com>]

Hi Fabio

On Tue, 2016-06-07 at 21:07 -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
> 
> MX6SX has an internal LDO regulator for the PCIE domain, which needs
> to be turned on for PCIE functionality.
> 
> Add support for it.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
> ---
>  .../devicetree/bindings/pci/fsl,imx6q-pcie.txt     |  2 ++
>  drivers/pci/host/pci-imx6.c                        | 29 +++++++++++++++++++++-

There was a discussion doing the handling of the regulator inside the PM
backend:

> On Mon, 2016-02-15 at 07:24 +0000, Richard Zhu wrote:
> > As Lucas discussed with me before,  the GPC regulator operations should[n't] be touched in
> > Imx pcie driver at all.  These bits operations should be encapsulate into the PM system,
> >  for example, the regulator driver.

On Thu, 2016-02-18 at 12:59 +0100, Christoph Fritz wrote:
> Ok, for an initial version, I'll just leave the regulator in my
> devicetree enabled. For further enhancements I suppose
> arch/arm/mach-imx/gpc.c needs to get touched?
> 
@Richard and @Lucas: Any further hints on this?

Thanks
 -- Christoph