Re: [PATCH v6 0/2] phy: intel,lgm-emmc-phy: Add support for eMMC PHY on Intel LGM SoC

[Ramuthevar, Vadivel MuruganX]

Hi Kishon,

 Kindly , could you please review the Intel's LGM SoC eMMC-PHY driver 
series of patches if you have time.
 Please consider and do the needful.
 Rob given Reviewed-by tag to eMMC-PHY dt_schema YAML file.
 Andy given Reviewed-by tag to eMMC-PHY driver file.

Regards
vadivel
On 27/11/2019 9:28 AM, Ramuthevar, Vadivel MuruganX wrote:
> Hi Kishon,
>
> Â Kindly , can you please review the eMMC-PHY driver series of patches 
> if you have time.
>
> ---
> With Best Regards
> Vadivel Murugan
> On 21/10/2019 5:54 PM, Ramuthevar,Vadivel MuruganX wrote:
> > Â Rebased to kernel verson 5.4
> >
> > dt-bindings: phy: intel-emmc-phy: Add YAML schema for LGM eMMC PHY
> > changes in v6:
> > ÂÂÂ - cobined comaptible strings
> > ÂÂÂ - added as contiguous and can be a single entry for reg properties
> > changes in v5:
> > ÂÂÂ - earlier Review-by tag given by Rob
> > ÂÂÂ - rework done with syscon parent node.
> >
> > Â changes in v4:
> > ÂÂÂ - As per Rob's review: validate 5.2 and 5.3
> > ÂÂÂ - drop unrelated items.
> >
> > Â changes in v3:
> > ÂÂÂ - resolve 'make dt_binding_check' warnings
> >
> > Â changes in v2:
> > ÂÂÂ As per Rob Herring review comments, the following updates
> > ÂÂ - change GPL-2.0 -> (GPL-2.0-only OR BSD-2-Clause)
> > ÂÂ - filename is the compatible string plus .yaml
> > ÂÂ - LGM: Lightning Mountain
> > ÂÂ - update maintainer
> > ÂÂ - add intel,syscon under property list
> > ÂÂ - keep one example instead of two
> >
> > phy: intel-lgm-emmc: Add support for eMMC PHY
> >  No changes from patchV5
> > Â Ramuthevar Vadivel Murugan (2):
> > ÂÂ dt-bindings: phy: intel-emmc-phy: Add YAML schema for LGM eMMC PHY
> >
> > ÂÂ phy: intel-lgm-emmc: Add support for eMMC PHY
> >
> > Â .../bindings/phy/intel,lgm-emmc-phy.yamlÂÂÂÂÂÂÂÂÂÂ |Â 63 +++++
> > Â drivers/phy/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 1 +
> > Â drivers/phy/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 1 +
> > Â drivers/phy/intel/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 9 +
> > Â drivers/phy/intel/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 2 +
> > Â drivers/phy/intel/phy-intel-emmc.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ | 283 
> > +++++++++++++++++++++
> > Â 6 files changed, 359 insertions(+)
> > Â create mode 100644 
> > Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml
> > Â create mode 100644 drivers/phy/intel/Kconfig
> > Â create mode 100644 drivers/phy/intel/Makefile
> > Â create mode 100644 drivers/phy/intel/phy-intel-emmc.c