[PATCH 2/3] mmc: meson-gx: minor improvements in meson_mmc_set_ios

Kevin Hilman khilman at baylibre.com
Fri Jan 27 09:43:45 PST 2017


Heiner Kallweit <hkallweit1 at gmail.com> writes:

> val isn't used in the switch clause and afterwards there's an
> identical statement. So remove it.
>
> In case of an unexpected bus width the error message indicates
> the intention to set the bus width to 4 and to go on.
> So remove the return statement. This return statement also
> conflicts with "setting to 4" because nothing would be set
> actually before returning.
>
> Signed-off-by: Heiner Kallweit <hkallweit1 at gmail.com>

Acked-by: Kevin Hilman <khilman at baylibre.com>

Thanks for the fix,

Kevin



More information about the linux-amlogic mailing list