Re: [PATCH] firmware: tegra: add BPMP debugfs support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 29.09.2017 17:58, Jonathan Hunter wrote:
+static int bpmp_populate_dir(struct tegra_bpmp *bpmp, struct seqbuf
*seqbuf,
+                 struct dentry *parent, uint32_t depth)

Do we need to use uint32_t here?

+{
+    int err;
+    uint32_t d, t;

And here?

It's part of the BPMP ABI that the data passed is 32 bit unsigned
integer. I wanted to emphasise that with the choice of integer type. Or
did you mean why I did not use u32?

Yes why not just u32?


No other reason than just a personal bias to use standard C99 integer types. The downstream driver was using a mix of both so wanted to standardize to just one.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux