[PATCHV3 2/4] OMAP: I2C: Remove the reset in the init path
khilman at ti.com
Thu Jul 21 11:30:29 EDT 2011
Shubhrajyoti D <shubhrajyoti at ti.com> writes:
> The reset in the driver at init is not needed
> anymore as the hwmod framework takes care of
> reseting it.
The addition of the new places for reset (after timeout, after error) is
not immediately clear, and not described here at all.
After digging myself (since it wasn't described in the changelog), I see
that since the reset was removed from omap_i2c_init(), which was called
not only during probe, but also after time out and error handling, new
reset calls were added back in those locations.
This is the kind of thing that needs to be thoroughly described in the
changelog so reviewers who are not necessarily experts in this driver
will easily and *quickly* understand what is going on without having to
dig into all the details themselves.
Summary: Descriptive, detailed changelogs are invaluable to
More information about the linux-arm-kernel