[ath9k-devel] [PATCH V3 2/2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-12-14 Thread Viresh Kumar
bool at all. That required updates to all user sites as well in a single commit. regmap core was also using debugfs_{read|write}_file_bool(), directly and variable types were updated for that to be bool as well. Acked-by: Mark Brown <broo...@kernel.org> Signed-off-by: Viresh Kumar <

[ath9k-devel] [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-12-14 Thread Viresh Kumar
bool at all. That required updates to all user sites as well in a single commit. regmap core was also using debugfs_{read|write}_file_bool(), directly and variable types were updated for that to be bool as well. Signed-off-by: Viresh Kumar <viresh.ku...@linaro.org> --- V1->V2: - V1 onl

Re: [ath9k-devel] [PATCH V3 2/2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-12-14 Thread Viresh Kumar
would be better to update this API to make it accept 'bool *' instead, as that will make it more consistent and often more convenient. Over that bool takes just a byte. That required updates to all user sites as well in a single commit. regmap core was also using debugfs_{read|write}_file_bool(), directly

Re: [ath9k-devel] [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-12-14 Thread Viresh Kumar
On 14-09-15, 17:39, Arnd Bergmann wrote: > On Monday 14 September 2015 09:21:54 Viresh Kumar wrote: > > diff --git a/drivers/acpi/ec_sys.c b/drivers/acpi/ec_sys.c > > index b4c216bab22b..bea8e425a8de 100644 > > --- a/drivers/acpi/ec_sys.c > > +++ b/drivers/acpi/ec

Re: [ath9k-devel] [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-12-14 Thread Viresh Kumar
On 14-09-15, 09:03, Greg KH wrote: > What ever ones you think it is relevant for :) "Relevant" is a relevant term :) So, the patch which defined the type bool as _Bool was added in v2.6.19 :) 6e2182874324 ("[PATCH] Generic boolean") So, I will try at least for v3.10+ as they are used by a lot

Re: [ath9k-devel] [PATCH V3 2/2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-12-14 Thread Viresh Kumar
On 15-09-15, 10:04, Steven Rostedt wrote: > On Tue, 15 Sep 2015 14:04:59 +0530 > Viresh Kumar <viresh.ku...@linaro.org> wrote: > > > diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c > > index 2614a839c60d..f11e17ad7834 100644 > > --- a/drivers/acpi