On Mon, Apr 11, 2011 at 2:18 PM, John Calixto <[email protected]> wrote: > Hi Andrei, > > On Tue, 5 Apr 2011, Andrei Warkentin wrote: >> diff --git a/include/linux/mmc/core.h b/include/linux/mmc/core.h >> index 07f27af..a7d7da7 100644 >> --- a/include/linux/mmc/core.h >> +++ b/include/linux/mmc/core.h >> @@ -92,7 +92,7 @@ struct mmc_command { >> * actively failing requests >> */ >> >> - unsigned int erase_timeout; /* in milliseconds */ >> + unsigned int cmd_timeout; /* in milliseconds */ >> >> struct mmc_data *data; /* data segment associated >> with cmd */ >> struct mmc_request *mrq; /* associated request */ > > Could you rename this field to "cmd_timeout_ms"? When I was testing the > addition of cmd_timeout to my ioctl patch, I noticed that data timeout > was being specified in ns and I had to scale cmd_timeout appropriately. >
Yes! Absolutely. Just sent the updated patch set. A -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
