On Tue, Jan 18, 2011 at 01:28:58PM -0800, Daniel Walker wrote: > On Tue, 2011-01-18 at 21:16 +0000, Russell King - ARM Linux wrote: > > On Tue, Jan 18, 2011 at 01:00:20PM -0800, Daniel Walker wrote: > > > Just greping the driver I found at least one writel() .. I'm not really > > > concerned with that tho. I use this driver, I need this driver to work. > > > You broke the driver, and your NAK'ing the fix .. > > > > If drivers use documented internal APIs, they _will_ break, and that's > > tough luck. The advertised API (DMA-mapping) is what we guarantee to > > support and fixup drivers for. > > I don't think that's a good policy. There could be reasons why a driver > might use an internal API .. Breaking the build is worse, to me, than > using an internal API. If things use the wrong APIs they get broken, plain and simple. It happens, and we cope with it and fix it properly when it happens, rather than subsituting the wrong way for another wrong way. > Did you give us a fix for this issue? That's what I was asking in the > last email cause it didn't appear like you touched that area. I gave you an example of how to fix the problem you reported - what more do you want? -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html