Hi Lukasz,

> Hi Tobias,
> 
> > The CD check is currently inverted. dm_gpio_get_value() returns
> > one when a card is detected. All other values (zero when there
> > is no card, or negative values for the internal errors) indicate
> > failure.
> > 
> > Signed-off-by: Tobias Jakobi <tjak...@math.uni-bielefeld.de>
> > ---
> >  drivers/mmc/s5p_sdhci.c | 20 +++++++++++---------
> >  1 file changed, 11 insertions(+), 9 deletions(-)
> > 
> > diff --git a/drivers/mmc/s5p_sdhci.c b/drivers/mmc/s5p_sdhci.c
> > index 6be3609..bc04370 100644
> > --- a/drivers/mmc/s5p_sdhci.c
> > +++ b/drivers/mmc/s5p_sdhci.c
> > @@ -102,29 +102,31 @@ struct sdhci_host sdhci_host[SDHCI_MAX_HOSTS];
> >  
> >  static int do_sdhci_init(struct sdhci_host *host)
> >  {
> > -   int dev_id, flag;
> > -   int err = 0;
> > +   int dev_id, flag, ret;
> >  
> >     flag = host->bus_width == 8 ? PINMUX_FLAG_8BIT_MODE :
> > PINMUX_FLAG_NONE; dev_id = host->index + PERIPH_ID_SDMMC0;
> >  
> >     if (dm_gpio_is_valid(&host->pwr_gpio)) {
> >             dm_gpio_set_value(&host->pwr_gpio, 1);
> > -           err = exynos_pinmux_config(dev_id, flag);
> > -           if (err) {
> > +           ret = exynos_pinmux_config(dev_id, flag);
> > +           if (ret) {
> >                     debug("MMC not configured\n");
> > -                   return err;
> > +                   return ret;
> >             }
> >     }
> >  
> >     if (dm_gpio_is_valid(&host->cd_gpio)) {
> > -           if (dm_gpio_get_value(&host->cd_gpio))
> > +           ret = dm_gpio_get_value(&host->cd_gpio);
> > +           if (ret != 1) {
> > +                   debug("No card detected (%d)\n", ret);
> >                     return -ENODEV;
> > +           }
> >  
> > -           err = exynos_pinmux_config(dev_id, flag);
> > -           if (err) {
> > +           ret = exynos_pinmux_config(dev_id, flag);
> > +           if (ret) {
> >                     printf("external SD not configured\n");
> > -                   return err;
> > +                   return ret;
> >             }
> >     }
> >  
> 
> Reviewed-by: Lukasz Majewski <l.majew...@samsung.com>
> 

Sorry, I was too fast. I've read the whole thread and I can confirm
that your change would break Trats board.

I hope that we will come up with proper solution.

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to