RE: [PATCH v2] watchdog: constify watchdog_info structures

2017-01-03 Thread Adam Thomson
On 26 December 2016 17:05, Bhumika Goyal wrote: > Declare watchdog_info structures as const as they are only stored in the > info field of watchdog_device structures. This field is of type const > struct watchdog_info *, so watchdog_info structures having this property > can be declared const too.

Re: [PATCH v2] watchdog: constify watchdog_info structures

2016-12-28 Thread Guenter Roeck
On 12/26/2016 09:05 AM, Bhumika Goyal wrote: Declare watchdog_info structures as const as they are only stored in the info field of watchdog_device structures. This field is of type const struct watchdog_info *, so watchdog_info structures having this property can be declared const too. Done usin

Re: [PATCH v2] watchdog: constify watchdog_info structures

2016-12-26 Thread Baruch Siach
Hi Bhumika Goyal, On Mon, Dec 26, 2016 at 10:35:11PM +0530, Bhumika Goyal wrote: > Declare watchdog_info structures as const as they are only stored in the > info field of watchdog_device structures. This field is of type const > struct watchdog_info *, so watchdog_info structures having this prop

Re: [PATCH v2] watchdog: constify watchdog_info structures

2016-12-26 Thread Florian Fainelli
Le 12/26/16 à 09:05, Bhumika Goyal a écrit : > Signed-off-by: Bhumika Goyal > --- > Changes in v2: > * Drop "drivers:" from the subject line. > * Remove the file size details. > > drivers/watchdog/bcm7038_wdt.c | 2 +- > diff --git a/drivers/watchdog/bcm7038_wdt.c b/drivers/watchdog/bcm703

[PATCH v2] watchdog: constify watchdog_info structures

2016-12-26 Thread Bhumika Goyal
Declare watchdog_info structures as const as they are only stored in the info field of watchdog_device structures. This field is of type const struct watchdog_info *, so watchdog_info structures having this property can be declared const too. Done using Coccinelle: @r1 disable optional_qualifier@