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

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) {

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");
if (node <= 0) {"marvell,armada-8k-sdhci");
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) {

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

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
participants (2)
-
Jaehoon Chung
-
Stefan Roese