* [PATCH] Staging: iio: tsl2x7x_core: Remove extra parentheses from DIV_ROUND_UP @ 2015-10-16 14:10 Shivani Bhardwaj 2015-10-16 14:27 ` [Outreachy kernel] " Julia Lawall 0 siblings, 1 reply; 3+ messages in thread From: Shivani Bhardwaj @ 2015-10-16 14:10 UTC (permalink / raw) To: outreachy-kernel Remove parentheses surrounding the first argument of the macro DIV_ROUND_UP as they are not required. Semantic patch used: @@ expression n,d; @@ - DIV_ROUND_UP((n),d) + DIV_ROUND_UP(n,d) Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com> --- drivers/staging/iio/light/tsl2x7x_core.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/iio/light/tsl2x7x_core.c b/drivers/staging/iio/light/tsl2x7x_core.c index 010e607..bf9627e 100644 --- a/drivers/staging/iio/light/tsl2x7x_core.c +++ b/drivers/staging/iio/light/tsl2x7x_core.c @@ -418,9 +418,9 @@ static int tsl2x7x_get_lux(struct iio_dev *indio_dev) if (p->ratio == 0) { lux = 0; } else { - ch0lux = DIV_ROUND_UP((ch0 * p->ch0), + ch0lux = DIV_ROUND_UP(ch0 * p->ch0, tsl2X7X_als_gainadj[chip->tsl2x7x_settings.als_gain]); - ch1lux = DIV_ROUND_UP((ch1 * p->ch1), + ch1lux = DIV_ROUND_UP(ch1 * p->ch1, tsl2X7X_als_gainadj[chip->tsl2x7x_settings.als_gain]); lux = ch0lux - ch1lux; } @@ -1057,7 +1057,7 @@ static ssize_t tsl2x7x_als_persistence_store(struct device *dev, z = y * TSL2X7X_MIN_ITIME; filter_delay = - DIV_ROUND_UP(((result.integer * 1000) + result.fract), z); + DIV_ROUND_UP((result.integer * 1000) + result.fract, z); chip->tsl2x7x_settings.persistence &= 0xF0; chip->tsl2x7x_settings.persistence |= (filter_delay & 0x0F); @@ -1103,7 +1103,7 @@ static ssize_t tsl2x7x_prox_persistence_store(struct device *dev, z = y * TSL2X7X_MIN_ITIME; filter_delay = - DIV_ROUND_UP(((result.integer * 1000) + result.fract), z); + DIV_ROUND_UP((result.integer * 1000) + result.fract, z); chip->tsl2x7x_settings.persistence &= 0x0F; chip->tsl2x7x_settings.persistence |= ((filter_delay << 4) & 0xF0); -- 2.1.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Outreachy kernel] [PATCH] Staging: iio: tsl2x7x_core: Remove extra parentheses from DIV_ROUND_UP 2015-10-16 14:10 [PATCH] Staging: iio: tsl2x7x_core: Remove extra parentheses from DIV_ROUND_UP Shivani Bhardwaj @ 2015-10-16 14:27 ` Julia Lawall 2015-10-16 15:06 ` Shivani Bhardwaj 0 siblings, 1 reply; 3+ messages in thread From: Julia Lawall @ 2015-10-16 14:27 UTC (permalink / raw) To: Shivani Bhardwaj; +Cc: outreachy-kernel On Fri, 16 Oct 2015, Shivani Bhardwaj wrote: > Remove parentheses surrounding the first argument of the macro > DIV_ROUND_UP as they are not required. > Semantic patch used: > > @@ > expression n,d; > @@ > > - DIV_ROUND_UP((n),d) > + DIV_ROUND_UP(n,d) This is just a comment; no need to change anything. In Coccinelle, there is a notion of isomorphism, where it will take the term that you write and convert it to other terms that are "pretty much" the same. One of these isomorphisms is to remove parentheses. Thus, if you run spatch --parse-cocci this_sp.cocci, you will see that it matches both the case where the parentheses are there and the case where they are not there, and transforms both of them according to your + code. In your case, this will amount to saying - DIV_ROUND_UP(n, d) + DIV_ROUND_UP(n, d) which looks completely useless. But it doesn't actually necessarily do nothing at all. Actually, it will remove DIV_ROUND_UP(n, d), and then use its own strategy for pretty printing DIV_ROUND_UP(n, d) again. Thus the output might have some newline that the original code didn't have. (This might be a good thing :). To avoid it removing code and putting the same code back, you could write your rule as: @@ @@ DIV_ROUND_UP( - ( n - ) ,d) Now it knows that the () are important to the rule, because they are annotated with - and the rest is not. It will only transform cases that actually have the parentheses in this case. julia > > Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com> > --- > drivers/staging/iio/light/tsl2x7x_core.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/iio/light/tsl2x7x_core.c b/drivers/staging/iio/light/tsl2x7x_core.c > index 010e607..bf9627e 100644 > --- a/drivers/staging/iio/light/tsl2x7x_core.c > +++ b/drivers/staging/iio/light/tsl2x7x_core.c > @@ -418,9 +418,9 @@ static int tsl2x7x_get_lux(struct iio_dev *indio_dev) > if (p->ratio == 0) { > lux = 0; > } else { > - ch0lux = DIV_ROUND_UP((ch0 * p->ch0), > + ch0lux = DIV_ROUND_UP(ch0 * p->ch0, > tsl2X7X_als_gainadj[chip->tsl2x7x_settings.als_gain]); > - ch1lux = DIV_ROUND_UP((ch1 * p->ch1), > + ch1lux = DIV_ROUND_UP(ch1 * p->ch1, > tsl2X7X_als_gainadj[chip->tsl2x7x_settings.als_gain]); > lux = ch0lux - ch1lux; > } > @@ -1057,7 +1057,7 @@ static ssize_t tsl2x7x_als_persistence_store(struct device *dev, > z = y * TSL2X7X_MIN_ITIME; > > filter_delay = > - DIV_ROUND_UP(((result.integer * 1000) + result.fract), z); > + DIV_ROUND_UP((result.integer * 1000) + result.fract, z); > > chip->tsl2x7x_settings.persistence &= 0xF0; > chip->tsl2x7x_settings.persistence |= (filter_delay & 0x0F); > @@ -1103,7 +1103,7 @@ static ssize_t tsl2x7x_prox_persistence_store(struct device *dev, > z = y * TSL2X7X_MIN_ITIME; > > filter_delay = > - DIV_ROUND_UP(((result.integer * 1000) + result.fract), z); > + DIV_ROUND_UP((result.integer * 1000) + result.fract, z); > > chip->tsl2x7x_settings.persistence &= 0x0F; > chip->tsl2x7x_settings.persistence |= ((filter_delay << 4) & 0xF0); > -- > 2.1.0 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20151016141011.GA14411%40ubuntu. > For more options, visit https://groups.google.com/d/optout. > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Outreachy kernel] [PATCH] Staging: iio: tsl2x7x_core: Remove extra parentheses from DIV_ROUND_UP 2015-10-16 14:27 ` [Outreachy kernel] " Julia Lawall @ 2015-10-16 15:06 ` Shivani Bhardwaj 0 siblings, 0 replies; 3+ messages in thread From: Shivani Bhardwaj @ 2015-10-16 15:06 UTC (permalink / raw) To: Julia Lawall; +Cc: outreachy-kernel Shivani On Fri, Oct 16, 2015 at 7:57 PM, Julia Lawall <julia.lawall@lip6.fr> wrote: > On Fri, 16 Oct 2015, Shivani Bhardwaj wrote: > >> Remove parentheses surrounding the first argument of the macro >> DIV_ROUND_UP as they are not required. >> Semantic patch used: >> >> @@ >> expression n,d; >> @@ >> >> - DIV_ROUND_UP((n),d) >> + DIV_ROUND_UP(n,d) > > This is just a comment; no need to change anything. > > In Coccinelle, there is a notion of isomorphism, where it will take the > term that you write and convert it to other terms that are "pretty much" > the same. One of these isomorphisms is to remove parentheses. Thus, if > you run spatch --parse-cocci this_sp.cocci, you will see that it matches > both the case where the parentheses are there and the case where they are > not there, and transforms both of them according to your + code. In your > case, this will amount to saying > > - DIV_ROUND_UP(n, d) > + DIV_ROUND_UP(n, d) > > which looks completely useless. But it doesn't actually necessarily do > nothing at all. Actually, it will remove DIV_ROUND_UP(n, d), and then use > its own strategy for pretty printing DIV_ROUND_UP(n, d) again. Thus the > output might have some newline that the original code didn't have. (This > might be a good thing :). > > To avoid it removing code and putting the same code back, you could write > your rule as: > > @@ > @@ > > DIV_ROUND_UP( > - ( > n > - ) > ,d) > > Now it knows that the () are important to the rule, because they are > annotated with - and the rest is not. It will only transform cases that > actually have the parentheses in this case. > > julia > > Thanks, Julia! :) I'm updating the patch. >> >> Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com> >> --- >> drivers/staging/iio/light/tsl2x7x_core.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/staging/iio/light/tsl2x7x_core.c b/drivers/staging/iio/light/tsl2x7x_core.c >> index 010e607..bf9627e 100644 >> --- a/drivers/staging/iio/light/tsl2x7x_core.c >> +++ b/drivers/staging/iio/light/tsl2x7x_core.c >> @@ -418,9 +418,9 @@ static int tsl2x7x_get_lux(struct iio_dev *indio_dev) >> if (p->ratio == 0) { >> lux = 0; >> } else { >> - ch0lux = DIV_ROUND_UP((ch0 * p->ch0), >> + ch0lux = DIV_ROUND_UP(ch0 * p->ch0, >> tsl2X7X_als_gainadj[chip->tsl2x7x_settings.als_gain]); >> - ch1lux = DIV_ROUND_UP((ch1 * p->ch1), >> + ch1lux = DIV_ROUND_UP(ch1 * p->ch1, >> tsl2X7X_als_gainadj[chip->tsl2x7x_settings.als_gain]); >> lux = ch0lux - ch1lux; >> } >> @@ -1057,7 +1057,7 @@ static ssize_t tsl2x7x_als_persistence_store(struct device *dev, >> z = y * TSL2X7X_MIN_ITIME; >> >> filter_delay = >> - DIV_ROUND_UP(((result.integer * 1000) + result.fract), z); >> + DIV_ROUND_UP((result.integer * 1000) + result.fract, z); >> >> chip->tsl2x7x_settings.persistence &= 0xF0; >> chip->tsl2x7x_settings.persistence |= (filter_delay & 0x0F); >> @@ -1103,7 +1103,7 @@ static ssize_t tsl2x7x_prox_persistence_store(struct device *dev, >> z = y * TSL2X7X_MIN_ITIME; >> >> filter_delay = >> - DIV_ROUND_UP(((result.integer * 1000) + result.fract), z); >> + DIV_ROUND_UP((result.integer * 1000) + result.fract, z); >> >> chip->tsl2x7x_settings.persistence &= 0x0F; >> chip->tsl2x7x_settings.persistence |= ((filter_delay << 4) & 0xF0); >> -- >> 2.1.0 >> >> -- >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. >> To post to this group, send email to outreachy-kernel@googlegroups.com. >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20151016141011.GA14411%40ubuntu. >> For more options, visit https://groups.google.com/d/optout. >> ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-10-16 15:06 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-10-16 14:10 [PATCH] Staging: iio: tsl2x7x_core: Remove extra parentheses from DIV_ROUND_UP Shivani Bhardwaj 2015-10-16 14:27 ` [Outreachy kernel] " Julia Lawall 2015-10-16 15:06 ` Shivani Bhardwaj
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.