* [U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates @ 2017-01-25 7:51 ` Stefan Roese 2017-01-31 3:43 ` Jaehoon Chung 0 siblings, 1 reply; 4+ messages in thread From: Stefan Roese @ 2017-01-25 7:51 UTC (permalink / raw) To: u-boot 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 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates 2017-01-25 7:51 ` [U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates Stefan Roese @ 2017-01-31 3:43 ` Jaehoon Chung 2017-01-31 5:44 ` Stefan Roese 0 siblings, 1 reply; 4+ messages in thread From: Jaehoon Chung @ 2017-01-31 3:43 UTC (permalink / raw) To: u-boot 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) { > ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates 2017-01-31 3:43 ` Jaehoon Chung @ 2017-01-31 5:44 ` Stefan Roese 2017-01-31 6:13 ` Jaehoon Chung 0 siblings, 1 reply; 4+ messages in thread From: Stefan Roese @ 2017-01-31 5:44 UTC (permalink / raw) To: u-boot 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates 2017-01-31 5:44 ` Stefan Roese @ 2017-01-31 6:13 ` Jaehoon Chung 0 siblings, 0 replies; 4+ messages in thread From: Jaehoon Chung @ 2017-01-31 6:13 UTC (permalink / raw) To: u-boot 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 > > > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-01-31 6:13 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20170125075729epcas5p1d532e484a877314a7ba888dd106e3e44@epcas5p1.samsung.com>
2017-01-25 7:51 ` [U-Boot] [PATCH] phy: comphy_a3700: Change SD/MMC compatible DT node to match the updates Stefan Roese
2017-01-31 3:43 ` Jaehoon Chung
2017-01-31 5:44 ` Stefan Roese
2017-01-31 6:13 ` Jaehoon Chung
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox