On 06/17/2014 10:14 AM, Tejun Heo wrote:

On Tue, Jun 17, 2014 at 12:13:20PM -0400, Tejun Heo wrote:

quoted

On Tue, Jun 17, 2014 at 10:10:23AM -0600, Stephen Warren wrote:

quoted

quoted

sata_writel() and sata_read() static inlines don't seem to add any value.

Can they be removed?

Such functions are quite idiomatic in drivers, and serve to simplify all
the call-sites by removing the need to write out the addition of the
base address everywhere.

I think it obfuscates more than helps.  If you absoluately have to
keep it, please at least name it so that it's clear that it's
something specific to this driver.

Please also drop inline.  It isn't a difficult optimization to perform
for the compiler.

There are probably hundreds of drivers marking those functions inline.
If they aren't marked inline, it'll just stick out as unusual when
people read them all, and then they'll send patches to fix it.