From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753051AbeAQNXy (ORCPT + 1 other); Wed, 17 Jan 2018 08:23:54 -0500 Received: from 8bytes.org ([81.169.241.247]:34582 "EHLO theia.8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752832AbeAQNXw (ORCPT ); Wed, 17 Jan 2018 08:23:52 -0500 Date: Wed, 17 Jan 2018 14:23:49 +0100 From: Joerg Roedel To: Geert Uytterhoeven Cc: iommu@lists.linux-foundation.org, Tony Lindgren , linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] iommu/omap: Fix debugfs_create_*() usage Message-ID: <20180117132349.3q4gkabp3rmm2yim@8bytes.org> References: <1514906985-9712-1-git-send-email-geert+renesas@glider.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1514906985-9712-1-git-send-email-geert+renesas@glider.be> User-Agent: NeoMutt/20170421 (1.8.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Tue, Jan 02, 2018 at 04:29:45PM +0100, Geert Uytterhoeven wrote: > When exposing data access through debugfs, the correct > debugfs_create_*() functions must be used, depending on data type. > > Remove all casts from data pointers passed to debugfs_create_*() > functions, as such casts prevent the compiler from flagging bugs. > > omap_iommu.nr_tlb_entries is "int", hence casting to "u8 *" exposes only > a part of it. Fix this by using debugfs_create_u32() instead. > > Signed-off-by: Geert Uytterhoeven > --- > Compile-tested only. > --- > drivers/iommu/omap-iommu-debug.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Applied, thanks.