Re: [PATCH 00/87] Fix some style warnings in hfa384x.h

On Wed, Sep 28, 2016 at 7:27 PM, Sergio Paracuellos wrote: > This patch series fix some warnings reported by checkpatch.pl script in > hfa384x.h: > WARNING: Block comments use * on subsequent lines > WARNING: Block comments use a trailing */ on a separate line >

Re: [PATCH] Staging: comedi: ni_daq_dio24.c: Fix block comments use * on subsequent lines.

On Sun, Sep 4, 2016 at 12:08 PM, Amit Ghadge wrote: > This is a patch to the ni_daq_dio24.c that fixes checkpatch warning: > WARNING: Block comments use * on subsequent lines > > Signed-off-by: Amit Ghadge > --- [...] Why are you sending so many copies

Re: [PATCH 1651/1651] staging: i4l: pcbit: Remove explicit NULL comparison

Hello shyam saini, On Tue, Aug 2, 2016 at 2:33 PM, shyam saini wrote: > Remove the explicit NULL comparison and rewrite in a compact form using > Coccinelle > > Signed-off-by: shyam saini Where are the other 1650 patches? Also, a patch series

Re: [PATCH] Staging: android: ion: ion.c: Compression of lines for

On Sun, Jul 31, 2016 at 6:44 PM, Nadim almas wrote: > This patch compresses two lines in to a single line in file > ion.c > if immediate return statement is found.It also removes variable > ret as it is no longer needed. > > ne using script Coccinelle. And coccinelle uses

Re: [PATCH v2] rtl8712: Fixed alignment to match open parenthesis

On Mon, Apr 25, 2016 at 4:00 AM, Parth Sane wrote: > Added missing signed off by line and fixed alignment to match > open parenthesis. Put "Added missing signed off by line" below the ---. This is not what we want to have in change logs.

Re: [PATCH] Staging: most: Remove check for __cplusplus in headers

On Sun, Mar 06, 2016 at 03:50:16PM +0530, PrasannaKumar Muralidharan wrote: > From: PrasannaKumar Muralidharan > > Signed-off-by: PrasannaKumar Muralidharan Commit message is missing. ___ devel

Re: [PATCH 3/8] fix staging:android style issue:Alignment should match open parenthesis

On Wed, Aug 26, 2015 at 11:52:12AM +0800, Peng Sun wrote: Signed-off-by: Peng Sun sironhide0n...@gmail.com Please always add a changelog message. Also, you need to fix the subject. The convention is to use something like: a: b: ... So in this case it should be something like: staging: android:

Re: [PATCH] staging: fbtft: Made into two lines

On Sun, Aug 23, 2015 at 07:00:23PM +0530, Aparna Karuthodi wrote: Oh! Sorry! I made the changes to correct the faults you figured out. Is it okay now? Resubmit it properly with PATCH v2 in the title and a good commit message. Signed-off-by: Aparna Karuthodi kdasapa...@gmail.com ---

Re: [PATCH] staging: fbtft: Made into two lines

On Sun, Aug 23, 2015 at 07:00:23PM +0530, Aparna Karuthodi wrote: Oh! Sorry! I made the changes to correct the faults you figured out. Is it okay now? Better. Also, submit it again with proper commit message and put PATCH v2 in the subject. Signed-off-by: Aparna Karuthodi

Re: [PATCH] staging: fbtft: Made into two lines

On Fri, Aug 21, 2015 at 06:38:39PM +0530, Aparna Karuthodi wrote: Made the comment into a new lineto remove a coding style error detected by checkpatch. The warning is given below: drivers/staging/fbtft/fb_pcd8544.c:37: WARNING: line over 80 characters Signed-off-by: Aparna Karuthodi

[PATCH v2] staging: ft1000: convert pack pragma to __packed

Convert a Microsoft compiler specific directive #pragma pack(1) to a GCC equivalent __packed. Also, by doing this we save ourselves from trouble if any other struct definitions are added after the #pragma because it will be applied to all of the definitions following it. Signed-off-by: Giedrius

[PATCH] staging: ft1000: convert pack pragma to __packed__ attribute

it. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/ft1000/ft1000-usb/ft1000_download.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_download.c b/drivers/staging/ft1000/ft1000-usb/ft1000_download.c

Re: [PATCH] mfd: rtsx: add support for rts522A

On Thu, 16 Apr 2015, micky_ch...@realsil.com.cn wrote: From: Micky Ching micky_ch...@realsil.com.cn rts522a(rts5227s) is derived from rts5227, and mainly same with rts5227. Add it to file mfd/rts5227.c to support this chip. Signed-off-by: Micky Ching micky_ch...@realsil.com.cn --- Maybe

Re: [PATCH v2 1/2 RESEND] staging: dgnc: remove dead code in dgnc_tty_write()

On Sat, 11 Apr 2015, Sudip Mukherjee wrote: On Fri, Apr 10, 2015 at 05:48:54PM +0300, Giedrius Statkevičius wrote: Remove the dead code protected by in_user in dgnc_tty_write() because it is set to 0 and never changed to 1 thus the code in ifs never gets executed. dgnc_tty_write

Re: [PATCH] staging: dgnc: check if kzalloc fails in dgnc_tty_init()

On Tue, 7 Apr 2015, Sudip Mukherjee wrote: On Tue, Apr 07, 2015 at 01:26:32PM +0300, Giedrius Statkevičius wrote: kzalloc() could fail so add a check and return -ENOMEM if it does that gets propogated to the pci layer Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com

[PATCH v2] staging: dgnc: check if kzalloc fails in dgnc_tty_init()

won't execute that if any of the allocations failed. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: Only returning -ENOMEM if an allocation failed isn't enough as it was spotted by Sudip so create a new label that frees all successfully allocated stuff and only

Re: [PATCH v3] staging: dgnc: check if kzalloc fails in dgnc_tty_init()

On Tue, 7 Apr 2015, Sudip Mukherjee wrote: On Tue, Apr 07, 2015 at 05:11:15PM +0300, Giedrius Statkevičius wrote: If one of the allocations of memory for storing a channel information struct fails then free all the successful allocations and return -ENOMEM that gets propogated to the pci

[PATCH v3] staging: dgnc: check if kzalloc fails in dgnc_tty_init()

if we can't allocate enough memory means that this scenario isn't possible. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v3: Remove the wrong comment at dgnc_tty_init() that says the allocation could happen somewhere else before this and remove the check if (!brd

Re: [PATCH v2] staging: dgnc: check if kzalloc fails in dgnc_tty_init()

On Tue, 7 Apr 2015, Dan Carpenter wrote: On Tue, Apr 07, 2015 at 03:40:17PM +0300, Giedrius Statkevičius wrote: If one of the allocations of memory for storing a channel information struct fails then free all the successful allocations and return -ENOMEM that gets propogated to the pci

Re: [PATCH v3] staging: dgnc: check if kzalloc fails in dgnc_tty_init()

On Tue, 7 Apr 2015, Dan Carpenter wrote: You will need to update the subject to reflect the new patch. The original code did check for kzalloc() failure but it had lots of checks scattered around instead nicely at the point where the memory was allocated. There are a lot missing too. For

[PATCH v4 3/3] staging: dgnc: remove redundant !ch checks

Remove checks that are redundant since we don't have boards with partially initialized -channels[i]. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v4: splitted this from the one patch. drivers/staging/dgnc/dgnc_cls.c | 4 ++-- drivers/staging/dgnc/dgnc_neo.c | 2 +- 2

[PATCH v4 2/3] staging: dgnc: don't forget to check if -channels[i] is NULL in dgnc_tty_uninit()

Add a check if -channels[i] is NULL because a NULL pointer may be dereferenced in case one of the allocations failed Reported-by: Dan Carpenter dan.carpen...@oracle.com Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v4: new patch that fixes a bug reported by Dan

[PATCH v4 1/3] staging: dgnc: clean up allocation of -channels[i]

and check in the same loop because -channels[i] isn't allocated anywhere else. Finally, remove a unnecessary check if -channels[i] is NULL in the next loop because it can't be. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v4: Make this patch only for dgnc_tty.c and only

Re: [PATCH 1/2] staging: dgnc: remove dead code in dgnc_tty_write()

to 1 thus the code in ifs never gets executed. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: Just remove the dead code protected by in_user and join the first and third patches. drivers/staging/dgnc/dgnc_tty.c | 45

Re: [PATCH] staging: sm7xxfb: Fix sparse warning

On Sat, 4 Apr 2015, Nickolaus Woodruff wrote: This patch fixes the following sparse warning in sm7xx.h: drivers/staging/sm7xxfb/sm7xx.h:122:17: warning: symbol 'vgamode' was not declared. Should it be static? Signed-off-by: Nickolaus Woodruff nickolauswoodr...@gmail.com ---

Re: [PATCH] staging: fbtft: 80 characters per line in fb_ili9163.c

On Sat, 4 Apr 2015, Andreas Theodosiou wrote: This is a patch to the fb_ili9163.c file that inserts a line break in line #92 to make the line fit into the 80 character limit. Signed-off-by: Andreas Theodosiou andreas...@gmail.com --- drivers/staging/fbtft/fb_ili9163.c | 3 ++- 1 file

Re: [PATCH] sm750fb: Fix Comment and if else braces

Hi Amitoj Kaur Chawla, On 2015.03.19 20:05, Amitoj Kaur Chawla wrote: The edits have been made to add a space before the comment and the errors in braces in the if-else statements. Previously in commit 2d34f53f1065878cd881ac61a183f8e836583d51, removed the C99 comments and errors in if-else

Re: [PATCH] sm750fb: Fix C99 Comments and if else braces

Hi Amitoj Kaur Chawla, On 2015.03.19 19:39, Amitoj Kaur Chawla wrote: The edits have been made to remove C99 Comments and properly indent the if-else statements in the file while taking care of the braces according to Linux coding style. Signed-off-by: Amitoj Kaur Chawla

Re: [PATCH v2] dgnc: Don't save boards in memory that have failed to initialize

Hi Greg, On 2015.03.12 12:08, Greg KH wrote: On Mon, Mar 09, 2015 at 06:29:38PM +0200, Giedrius Statkevičius wrote: Remove BOARD_FAILED and don't save dgnc_boards which failed to initialize. Assign the result of kzalloc() to brd in dgnc_found_board() and only put it in the dgnc_Board

Re: [PATCH] staging: sm750fb: braces, indents, spaces fix

Hi Ragavendra, On 2015.03.14 03:04, Ragavendra Nagraj wrote: This patch removes the braces for the single line if statement. It fixes the indent positions correctly. It fixes the spaces appropriately making the code give no warnings by the checpath.pl. Please accept. The Please accept. part

[PATCH v2 4/7] dgnc: use linux/types.h instead of dgnc_types.h

Dgnc_types.h unnecesarily defines TRUE as 1 and FALSE as 0 because we already have a widely used linux/types.h so convert all TRUE to true, FALSE to false and edit the dgnc_board struct to make sure it uses bool. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- I'm still

[PATCH v2 5/7] dgnc: remove old 2.4-2.6 compat kernel defines

dgnc_kcompat.h contains some old legacy defines in case the kernel doesn't have __user defined but for current kernel versions these defines don't make sense and are useless so remove them. Move the TTY_FLIPBUF_SIZE define to digi.h because it's used in the code. Signed-off-by: Giedrius

[PATCH v2 2/7] dgnc: clean up digi.h

Remove a lot of unused structs and defines from digi.h. We still have to be careful with TIOCM_LE and TIOCMSET/TIOCMBIC because termios.h and ioctls.h respectfully redefine them. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: Updated commit msg and remove some more

[PATCH v2 1/7] dgnc: remove unused dgnc_ioctl_name() command

dgnc_ioctl_name() is never used anywhere so remove it Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: Forgot to include this patch in the first version. Without this some of the other patches in this set will fail. drivers/staging/dgnc/dgnc_utils.c | 52

[PATCH v2 7/7] dgnc: Clean up dgnc_sysfs.h

Remove redundant blank lines, move absolute include after relative include. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: no change drivers/staging/dgnc/dgnc_sysfs.h | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/dgnc

[PATCH v2 3/7] dgnc: get rid of dpacompat.h, move remaining stuff to digi.h

Dpacompat.h contained a lot of unused #defines and only few things are used from it so since we've trimmed down digi.h, now we can delete dpacompat.h and move remaining stuff into digi.h. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: No change drivers/staging/dgnc

Re: [PATCH v2] dgnc: Don't save boards in memory that have failed to initialize

On 2015.03.12 12:08, Greg KH wrote: On Mon, Mar 09, 2015 at 06:29:38PM +0200, Giedrius Statkevičius wrote: Remove BOARD_FAILED and don't save dgnc_boards which failed to initialize. Assign the result of kzalloc() to brd in dgnc_found_board() and only put it in the dgnc_Board

[PATCH 2/6] dgnc: get rid of dpacompat.h, move remaining stuff to digi.h

Dpacompat.h contained a lot of unused #defines and only few things are used from it so since we've trimmed down digi.h, now we can delete dpacompat.h and move remaining stuff into digi.h. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_driver.c

[PATCH 1/6] dgnc: clean up digi.h

/ioctls.h defines them too. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/digi.h | 235 1 file changed, 235 deletions(-) diff --git a/drivers/staging/dgnc/digi.h b/drivers/staging/dgnc/digi.h index 554fbeb

[PATCH 5/6] dgnc: remove unused stuff from dgnc_cls.h

Remove unused defines from dgnc_cls.h Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_cls.h | 13 - 1 file changed, 13 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_cls.h b/drivers/staging/dgnc/dgnc_cls.h index 85042bd

[PATCH 4/6] dgnc: remove old 2.4-2.6 compat kernel defines

dgnc_kcompat.h contains some old legacy defines in case the kernel doesn't have __user defined but for current kernel versions these defines don't make sense and are useless so remove them. Move the TTY_FLIPBUF_SIZE define to digi.h because it's used in the code. Signed-off-by: Giedrius

[PATCH 6/6] dgnc: Clean up dgnc_sysfs.h

Remove redundant blank lines, move absolute include after relative include. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_sysfs.h | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_sysfs.h b

[PATCH 3/6] dgnc: convert TRUE/FALSE #defines to 1/0

TRUE/FALSE defined in dgnc_types.h are only used in dgnc_tty.c thus it would be better to just use 1/0 to avoid unnecessary extra files and includes. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_cls.h| 3 --- drivers/staging/dgnc

Re: [PATCH 3/6] dgnc: convert TRUE/FALSE #defines to 1/0

On 2015.03.12 20:59, Dan Carpenter wrote: On Thu, Mar 12, 2015 at 08:14:25PM +0200, Giedrius Statkevičius wrote: TRUE/FALSE defined in dgnc_types.h are only used in dgnc_tty.c thus it would be better to just use 1/0 to avoid unnecessary extra files and includes. Use true/false. I don't

Re: [PATCH 2/3] Staging: dgnc: Use goto for error handling

On 2015.03.11 16:04, Quentin Lambert wrote: This patch introduces goto statments for error handling and in cases where a lock needs to be released. A simplified version of the semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr) @candidates exists@ identifier

Re: [PATCH 1/1] Drivers: Staging: ft1000: checkpatch warning fixes

On 2015.03.11 23:03, Janakarajan Natarajan wrote: Addition of blank line after declaration in ft1000_hw.c Minor changes to remove {} from single line if and remove extra parenthesis. Fixes checkpatch warning for asm/bitops.h and asm/io.h usage. Greg may complain that you've done too much in 1

Re: [PATCHv4 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

Hi Daniele, On 2015.03.12 00:05, Daniele Alessandrelli wrote: Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli

[PATCH 2/3] dgnc: remove DGNC_VERIFY_BOARD macro

to initialize and make sysfs files after initializing the board in dgnc_driver.c or IOW they are already set for successfully initialized boards before their sysfs files created (412 and 593 lines in dgnc_driver.c). Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers

[PATCH 1/3] dgnc: Remove unneeded dgnc_state array of strings

Dgnc_state array of strings is never used anywhere and it seems pretty useless anyway since the board state enum names speak for themselves. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_driver.c | 8 drivers/staging/dgnc

[PATCH 3/3] dgnc: clean up comments at start of files

are responsible for these drivers - see TODO or MAINTAINERS for that. We can preserve the original copyright at the top of the most files because it shows who originally made them. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_cls.c | 16

[PATCH] dgnc: Don't save boards in memory that have failed to initialize

where state was set to BOARD_FAILED which was even redundant before this patch. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_driver.c | 19 ++- drivers/staging/dgnc/dgnc_driver.h | 3 +-- drivers/staging/dgnc/dgnc_mgmt.c | 5

Re: [PATCH] dgnc: Don't save boards in memory that have failed to initialize

On 2015.03.09 17:27, Giedrius Statkevičius wrote: Remove BOARD_FAILED and don't save dgnc_boards which failed to initialize. Assign the result of kzalloc() to brd in dgnc_found_board() and only put it in the dgnc_Board[] if it successfully initializes. Also, remove BOARD_FAILED enum and all

[PATCH v2] dgnc: Don't save boards in memory that have failed to initialize

where state was set to BOARD_FAILED which was even redundant before this patch. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- v2: Remove brd = dgnc_Board[dgnc_NumBoards]; line which I forgot to do in the first version drivers/staging/dgnc/dgnc_driver.c | 20

[PATCH] dgnc: move more repeating code under 'failed' label

-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_driver.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/drivers/staging/dgnc/dgnc_driver.c index fa1ee79..73ee667 100644 --- a/drivers/staging

[PATCH] dgnc: Make all lines under 80 characters in dgnc_cls.h

Some of the lines are over 80 characters so fix that by moving the comments before the struct definition and before #define's. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_cls.h | 40 1 file changed

[PATCH] dgnc: Make all lines under 80 characters in dgnc_driver.c

Some of the lines are over 80 characters in dgnc_driver.c so fix them by moving the comments closer to the code, tidying the comments to make them smaller, and remove a redundant space after +. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc

[PATCH] dgnc: Remove redundant blank lines in dgnc_cls.c

There are a lot double of blank lines in dgnc_cls.c thus remove them to make the file follow the CodingStyle. Also, remove one blank line at the end of dgnc_cls.c. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_cls.c | 36

[PATCH 4/4] dgnc: Make all lines under 80 characters in dgnc_driver.h

Fix comments to make lines less than 80 characters by moving them, breaking the lines or making them neater. Signed-off-by: Giedrius Statkevičius giedrius.statkevic...@gmail.com --- drivers/staging/dgnc/dgnc_driver.h | 55 +- 1 file changed, 36 insertions

Re: [PATCH 4/4] dgnc: Make all lines under 80 characters in dgnc_driver.h

This one is bad. I've accidentally sent it, sorry. Don't pull/review this. On 2015.03.09 01:49, Giedrius Statkevičius wrote: Fix comments to make lines less than 80 characters by moving them, breaking the lines or making them neater. [...] -- Thanks, Giedrius

Re: [PATCH 1/4] staging: sm750fb: wrong type for print

On 2015.03.08 14:31, Sudip Mukherjee wrote: mention correct format specifier while printing Signed-off-by: Sudip Mukherjee su...@vectorindia.org --- this patch will give checkpatch warnings about use of printk. this patch was mainly to fix the build warnings. printk will be converted to

Re: [PATCH 3/4] staging: sm750fb: correctly define SM750LE_REVISION_ID

On 2015.03.08 14:31, Sudip Mukherjee wrote: check if it is already defined before defining SM750LE_REVISION_ID again and at the same time mention correct data type. Signed-off-by: Sudip Mukherjee su...@vectorindia.org --- drivers/staging/sm750fb/ddk750_chip.h | 4 +++- 1 file changed, 3

Re: [PATCH 4/4] staging: sm750fb: fix undeclared function

On 2015.03.08 14:31, Sudip Mukherjee wrote: kbuild test robot reported that for microblaze-allyesconfig chan_to_field() and lynxfb_ops_set_par() were not defined. These two functions were defined under CONFIG_PM, so for any archtecture if CONFIG_PM is not defined we will have this error.

Re: [PATCH] iee80211:Remove no longer required strutucre defintion, _STA_QOS in rtl819x_Qos.h

On 2015.03.06 23:20, Nicholas Krause wrote: Removes the no longer required structure definition,_STA_QOS due to no functions using any structures of this type anymore. Signed-off-by: Nicholas Krause xerofo...@gmail.com --- drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h | 48

Re: [PATCH] staging: vt6655: fix space prohibted before that ',' Signed-off-by: Ahmadposten ahmad.hassan...@gmail.com

On 2015.01.19 01:38, Ahmadposten wrote: --- drivers/staging/vt6655/device_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c index f5cc78e..34e3a76 100644 ---

Re: [PATCH]imx-drm coding style fixes

On 2014.09.19 01:57, Rene Kolarik wrote: Hello, I've fixed coding style issues in drivers/staging/imx-drm directory. This is a part of Eudyptula Challenge. Please submit your patches inline (look at /Documentation/SubmittingPatches section 7). Also, think about if it's worth seperating this