[U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates

[U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates

[Stefan Roese]

Now that the SD/SDIO/MMC DT properties are updated in the Marvell
A3700 and A7/8k DT files, we need to match the checks for compatible
node in the PHY driver as well.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Kostya Porotchkin <kostap@marvell.com>
Cc: Nadav Haklai <nadavh@marvell.com>
---
 drivers/phy/marvell/comphy_a3700.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/phy/marvell/comphy_a3700.c b/drivers/phy/marvell/comphy_a3700.c
index faa62f90ae..5afd23c052 100644
--- a/drivers/phy/marvell/comphy_a3700.c
+++ b/drivers/phy/marvell/comphy_a3700.c
@@ -884,11 +884,10 @@ void comphy_dedicated_phys_init(void)
 	}
 
 	node = fdt_node_offset_by_compatible(blob, -1,
-					     "marvell,armada-3700-sdio");
+					     "marvell,armada-8k-sdhci");
 	if (node <= 0) {
-		debug("No SDIO node in DT, looking for MMC one\n");
-		node = fdt_node_offset_by_compatible(blob, -1,
-						     "marvell,xenon-sdhci");
+		node = fdt_node_offset_by_compatible(
+			blob, -1, "marvell,armada-3700-sdhci");
 	}
 
 	if (node > 0) {
-- 
2.11.0

[U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates

[Jaehoon Chung]

Hi Stefan,

On 01/25/2017 04:51 PM, Stefan Roese wrote:
> Now that the SD/SDIO/MMC DT properties are updated in the Marvell
> A3700 and A7/8k DT files, we need to match the checks for compatible
> node in the PHY driver as well.

This patch was delegated to me. But i think right that it might be delegated to you.
I will change to you. If there is a problem, let me know, plz.

Best Regards,
Jaehoon Chung

> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Kostya Porotchkin <kostap@marvell.com>
> Cc: Nadav Haklai <nadavh@marvell.com>
> ---
>  drivers/phy/marvell/comphy_a3700.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/phy/marvell/comphy_a3700.c b/drivers/phy/marvell/comphy_a3700.c
> index faa62f90ae..5afd23c052 100644
> --- a/drivers/phy/marvell/comphy_a3700.c
> +++ b/drivers/phy/marvell/comphy_a3700.c
> @@ -884,11 +884,10 @@ void comphy_dedicated_phys_init(void)
>  	}
>  
>  	node = fdt_node_offset_by_compatible(blob, -1,
> -					     "marvell,armada-3700-sdio");
> +					     "marvell,armada-8k-sdhci");
>  	if (node <= 0) {
> -		debug("No SDIO node in DT, looking for MMC one\n");
> -		node = fdt_node_offset_by_compatible(blob, -1,
> -						     "marvell,xenon-sdhci");
> +		node = fdt_node_offset_by_compatible(
> +			blob, -1, "marvell,armada-3700-sdhci");
>  	}
>  
>  	if (node > 0) {
> 

[U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates

[Stefan Roese]

Hi Jaehoon,

On 31.01.2017 04:43, Jaehoon Chung wrote:
> On 01/25/2017 04:51 PM, Stefan Roese wrote:
>> Now that the SD/SDIO/MMC DT properties are updated in the Marvell
>> A3700 and A7/8k DT files, we need to match the checks for compatible
>> node in the PHY driver as well.
>
> This patch was delegated to me. But i think right that it might be delegated to you.
> I will change to you.

Thats just fine.

> If there is a problem, let me know, plz.

No problem.

Thanks,
Stefan

[U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates

[Jaehoon Chung]

Hi Stefan,

On 01/31/2017 02:44 PM, Stefan Roese wrote:
> Hi Jaehoon,
> 
> On 31.01.2017 04:43, Jaehoon Chung wrote:
>> On 01/25/2017 04:51 PM, Stefan Roese wrote:
>>> Now that the SD/SDIO/MMC DT properties are updated in the Marvell
>>> A3700 and A7/8k DT files, we need to match the checks for compatible
>>> node in the PHY driver as well.
>>
>> This patch was delegated to me. But i think right that it might be delegated to you.
>> I will change to you.
> 
> Thats just fine.
> 
>> If there is a problem, let me know, plz.
> 
> No problem.

Thanks! Anyway, I have checked this patch, it looks good to me.

Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>

Best Regards,
Jaehoon Chung

> 
> Thanks,
> Stefan
> 
> 
>