On Fri, Oct 7, 2011 at 12:43 AM, Greg KH <g...@kroah.com> wrote: > On Fri, Oct 07, 2011 at 10:33:06AM +0500, G, Manjunath Kondaiah wrote: >> >> Add new error value so that drivers can request deferred probe >> from drivercore. >> >> Signed-off-by: G, Manjunath Kondaiah <manj...@ti.com> >> Reported-by: Grant Likely <grant.lik...@secretlab.ca> >> --- >> Cc: linux-o...@vger.kernel.org >> Cc: linux-mmc@vger.kernel.org >> Cc: linux-ker...@vger.kernel.org >> Cc: Grant Likely <grant.lik...@secretlab.ca> >> Cc: Greg Kroah-Hartman <g...@kroah.com> >> Cc: Dilan Lee <di...@nvidia.com> >> Cc: Mark Brown <broo...@opensource.wolfsonmicro.com> >> Cc: Manjunath GKondaiah <manjunath.gkonda...@linaro.org> >> Cc: Arnd Bergmann <a...@arndb.de> >> >> include/linux/errno.h | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/include/linux/errno.h b/include/linux/errno.h >> index 4668583..83d8fcf 100644 >> --- a/include/linux/errno.h >> +++ b/include/linux/errno.h >> @@ -16,6 +16,7 @@ >> #define ERESTARTNOHAND 514 /* restart if no handler.. */ >> #define ENOIOCTLCMD 515 /* No ioctl command */ >> #define ERESTART_RESTARTBLOCK 516 /* restart by calling sys_restart_syscall >> */ >> +#define EPROBE_DEFER 517 /* restart probe again after some time */ > > Can we really do this?
According to Arnd, yes this is okay. > Isn't this some user/kernel api here? > > What's wrong with just "overloading" on top of an existing error code? > Surely one of the other 516 types could be used here, right? overloading makes it really hard to find the users at a later date. g. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html