From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Date: Thu, 26 Feb 2015 16:53:54 +0000 Subject: [patch -next] drm/i915: fix a printk format Message-Id: <20150226165354.GA6820@mwanda> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org, kernel-janitors@vger.kernel.org, dri-devel@lists.freedesktop.org This printk leads to the following Smatch warning: drivers/gpu/drm/i915/i915_gem_gtt.c:336 alloc_pt_range() error: '%pa' expects argument of type 'phys_addr_t*', argument 5 has type 'struct i915_page_table_entry*' It looks like a simple typo to me where "%p" was intended instead of "%pa". Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c index e05488e..48eff65 100644 --- a/drivers/gpu/drm/i915/i915_gem_gtt.c +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c @@ -334,7 +334,7 @@ static int alloc_pt_range(struct i915_page_directory_entry *pd, uint16_t pde, si goto err_out; } WARN(pd->page_table[i], - "Leaking page directory entry %d (%pa)\n", + "Leaking page directory entry %d (%p)\n", i, pd->page_table[i]); pd->page_table[i] = pt; } From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: [patch -next] drm/i915: fix a printk format Date: Thu, 26 Feb 2015 19:53:54 +0300 Message-ID: <20150226165354.GA6820@mwanda> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org, kernel-janitors@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org VGhpcyBwcmludGsgbGVhZHMgdG8gdGhlIGZvbGxvd2luZyBTbWF0Y2ggd2FybmluZzoKCglkcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9ndHQuYzozMzYgYWxsb2NfcHRfcmFuZ2UoKQoJCWVy cm9yOiAnJXBhJyBleHBlY3RzIGFyZ3VtZW50IG9mIHR5cGUgJ3BoeXNfYWRkcl90KicsCgkJYXJn dW1lbnQgNSBoYXMgdHlwZSAnc3RydWN0IGk5MTVfcGFnZV90YWJsZV9lbnRyeSonCgpJdCBsb29r cyBsaWtlIGEgc2ltcGxlIHR5cG8gdG8gbWUgd2hlcmUgIiVwIiB3YXMgaW50ZW5kZWQgaW5zdGVh ZCBvZgoiJXBhIi4KClNpZ25lZC1vZmYtYnk6IERhbiBDYXJwZW50ZXIgPGRhbi5jYXJwZW50ZXJA b3JhY2xlLmNvbT4KCmRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9n dHQuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtX2d0dC5jCmluZGV4IGUwNTQ4OGUu LjQ4ZWZmNjUgMTAwNjQ0Ci0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtX2d0dC5j CisrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtX2d0dC5jCkBAIC0zMzQsNyArMzM0 LDcgQEAgc3RhdGljIGludCBhbGxvY19wdF9yYW5nZShzdHJ1Y3QgaTkxNV9wYWdlX2RpcmVjdG9y eV9lbnRyeSAqcGQsIHVpbnQxNl90IHBkZSwgc2kKIAkJCWdvdG8gZXJyX291dDsKIAkJfQogCQlX QVJOKHBkLT5wYWdlX3RhYmxlW2ldLAotCQkgICAgICJMZWFraW5nIHBhZ2UgZGlyZWN0b3J5IGVu dHJ5ICVkICglcGEpXG4iLAorCQkgICAgICJMZWFraW5nIHBhZ2UgZGlyZWN0b3J5IGVudHJ5ICVk ICglcClcbiIsCiAJCSAgICAgaSwgcGQtPnBhZ2VfdGFibGVbaV0pOwogCQlwZC0+cGFnZV90YWJs ZVtpXSA9IHB0OwogCX0KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Au b3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2 ZWwK