On Mon, Sep 3, 2012 at 2:50 PM, Subhash Jadavani <subhashj@xxxxxxxxxxxxxx> wrote: > As such this is ok if you want this behaviour. But i see other issue which > is not related to your patch. > Let's say if the host supports 400KHz, 25 MHz, 50MHz, 100MHz and currently > card is operating in HS (High Speed) mode@50MHz. and now if someone tries to > change the frequency to 100MHz, then we should just don't let the clock > change to 100MHz before we put the card in appropriate bus speed mode (in SD > cards case, it would be SDR50 and eMMC case it would be HS200). Although > this is anyway not an issue created by your patch but this is just to let > you know that frequency can't be changed indepently without informing the > card beforehand. That's why the file is located under debugfs. There is an assumption that developer understands what he or she is doing by changing this parameter. -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
