Messages in this thread | | | Date | Thu, 13 Oct 2011 21:44:10 +0100 | From | Mark Brown <> | Subject | Re: [PATCH] staging:iio:proof of concept in kernel interface. |
| |
On Thu, Oct 13, 2011 at 03:46:04PM +0100, Jonathan Cameron wrote:
> > I guess an actual implementation would have wrappers for doing the > > indirections rather than having users peer into the ops table directly?
> Yup, for some reason the cover letter seems to have detached from this. > It suggested exactly that. There may be weird cases where peering this > deep into the ops makes sense, but not for something like this one.
Oh, right. As a general rule I don't read cover letters for single patches until after I've read the patch, generally they're either completely content free (if only by virtue of repeating the changelog) or there's a problem with the changelog in the actual patch not explaining what's going on.
| |