> Oliver did not suggest that the patch be written this way. > > Now with the patch written out, I think a better name for this variable > would be media_initialized. That is a better description of what it > means (it doesn't mean that the driver or the device is initialized). > And then you could remove the comment, because it would be obvious. > > You don't need to do this. The info pointer is an alias for us->extra, > which is allocated by kzalloc(), which clears all the memory it > allocates to zero. > > Alan Stern Thanks for your patient feedback! I will rewrite the patch soon.