On Thu, Mar 11, 2021 at 05:04:06PM +0300, Dmitry Osipenko wrote: > 10.03.2021 06:36, Nicolin Chen пишет: > ... > > +static int tegra_smmu_mappings_show(struct seq_file *s, void *data) > > +{ > > + struct tegra_smmu_group_debug *group_debug = s->private; > > + const struct tegra_smmu_swgroup *group; > > + struct tegra_smmu_as *as; > > + struct tegra_smmu *smmu; > > + int pd_index, pt_index; > > > +DEFINE_SHOW_ATTRIBUTE(tegra_smmu_mappings); > > + > > static void tegra_smmu_debugfs_init(struct tegra_smmu *smmu) > > { > > + const struct tegra_smmu_soc *soc = smmu->soc; > > + struct tegra_smmu_group_debug *group_debug; > > + struct device *dev = smmu->dev; > > + struct dentry *d; > > + int i; > > You should use unsigned types for everything that isn't signed. Okay. Will fix. Thanks!