From: Andy Shevchenko <andy.shevchenko@gmail.com> To: "Heiko Stübner" <heiko@sntech.de> Cc: "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Mike Turquette <mturquette@linaro.org>, Seungwon Jeon <tgih.jun@samsung.com>, Jaehoon Chung <jh80.chung@samsung.com>, Chris Ball <cjb@laptop.org>, "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>, Grant Likely <grant.likely@linaro.org>, Rob Herring <rob.herring@calxeda.com>, Linus Walleij <linus.walleij@linaro.org>, Devicetree Discuss <devicetree-discuss@lists.ozlabs.org>, Russell King <linux@arm.linux.org.uk>, Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Subject: Re: [PATCH v3 1/7] clk: divider: add flag to limit possible dividers to even numbers Date: Tue, 11 Jun 2013 15:37:27 +0300 [thread overview] Message-ID: <CAHp75VeQOSns1nL7p+rU1TS2stPuWibFtN_K9mFvzuGW+3WqOA@mail.gmail.com> (raw) In-Reply-To: <201306111406.14850.heiko@sntech.de> On Tue, Jun 11, 2013 at 3:06 PM, Heiko Stübner <heiko@sntech.de> wrote: > Am Dienstag, 11. Juni 2013, 13:51:56 schrieb Andy Shevchenko: >> On Tue, Jun 11, 2013 at 2:29 PM, Heiko Stübner <heiko@sntech.de> wrote: [] >> > @@ -141,6 +149,8 @@ static bool _is_valid_div(struct clk_divider >> > *divider, unsigned int div) >> > >> > return is_power_of_2(div); >> > >> > if (divider->table) >> > >> > return _is_valid_table_div(divider->table, div); >> > >> > + if (divider->flags & CLK_DIVIDER_EVEN && div != 1 && (div % 2) != >> > 0) + return false; >> > >> > return true; >> > >> > } >> >> What if rewrite like >> >> if (divider->flags & CLK_DIVIDER_EVEN == 0) >> return true; >> >> return div < 2 || div % 2 == 0; > > hmm, the current structure is of the form of testing for each feature and > doing a applicable action if the flag is set. So it also is extensible for > future flags and checking for the absence of an attribute while the rest of > the conditionals check for the presence also might make the code harder to > read. > > So for me the current variant somehow looks more intuitive. This variant I think fits: if (!(divider->flags & CLK_DIVIDER_EVEN)) return div < 2 || div % 2 == 0; You check for feature and do accordingly. -- With Best Regards, Andy Shevchenko
WARNING: multiple messages have this Message-ID (diff)
From: andy.shevchenko@gmail.com (Andy Shevchenko) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 1/7] clk: divider: add flag to limit possible dividers to even numbers Date: Tue, 11 Jun 2013 15:37:27 +0300 [thread overview] Message-ID: <CAHp75VeQOSns1nL7p+rU1TS2stPuWibFtN_K9mFvzuGW+3WqOA@mail.gmail.com> (raw) In-Reply-To: <201306111406.14850.heiko@sntech.de> On Tue, Jun 11, 2013 at 3:06 PM, Heiko St?bner <heiko@sntech.de> wrote: > Am Dienstag, 11. Juni 2013, 13:51:56 schrieb Andy Shevchenko: >> On Tue, Jun 11, 2013 at 2:29 PM, Heiko St?bner <heiko@sntech.de> wrote: [] >> > @@ -141,6 +149,8 @@ static bool _is_valid_div(struct clk_divider >> > *divider, unsigned int div) >> > >> > return is_power_of_2(div); >> > >> > if (divider->table) >> > >> > return _is_valid_table_div(divider->table, div); >> > >> > + if (divider->flags & CLK_DIVIDER_EVEN && div != 1 && (div % 2) != >> > 0) + return false; >> > >> > return true; >> > >> > } >> >> What if rewrite like >> >> if (divider->flags & CLK_DIVIDER_EVEN == 0) >> return true; >> >> return div < 2 || div % 2 == 0; > > hmm, the current structure is of the form of testing for each feature and > doing a applicable action if the flag is set. So it also is extensible for > future flags and checking for the absence of an attribute while the rest of > the conditionals check for the presence also might make the code harder to > read. > > So for me the current variant somehow looks more intuitive. This variant I think fits: if (!(divider->flags & CLK_DIVIDER_EVEN)) return div < 2 || div % 2 == 0; You check for feature and do accordingly. -- With Best Regards, Andy Shevchenko
next prev parent reply other threads:[~2013-06-11 12:37 UTC|newest] Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-06-11 11:28 [PATCH v3 0/7] arm: add basic support for Rockchip Cortex-A9 SoCs Heiko Stübner 2013-06-11 11:28 ` Heiko Stübner 2013-06-11 11:28 ` Heiko Stübner 2013-06-11 11:29 ` [PATCH v3 1/7] clk: divider: add flag to limit possible dividers to even numbers Heiko Stübner 2013-06-11 11:29 ` Heiko Stübner 2013-06-11 11:29 ` Heiko Stübner 2013-06-11 11:51 ` Andy Shevchenko 2013-06-11 11:51 ` Andy Shevchenko 2013-06-11 11:51 ` Andy Shevchenko 2013-06-11 12:06 ` Heiko Stübner 2013-06-11 12:06 ` Heiko Stübner 2013-06-11 12:06 ` Heiko Stübner 2013-06-11 12:37 ` Andy Shevchenko [this message] 2013-06-11 12:37 ` Andy Shevchenko 2013-06-11 12:37 ` Andy Shevchenko 2013-06-11 12:39 ` Andy Shevchenko 2013-06-11 12:39 ` Andy Shevchenko 2013-06-11 12:39 ` Andy Shevchenko 2013-06-11 18:57 ` Mike Turquette 2013-06-11 18:57 ` Mike Turquette 2013-06-11 19:23 ` Heiko Stübner 2013-06-11 19:23 ` Heiko Stübner 2013-06-11 19:23 ` Heiko Stübner 2013-06-11 11:29 ` [PATCH v3 2/7] mmc: dw_mmc-pltfm: remove static from dw_mci_pltfm_remove Heiko Stübner 2013-06-11 11:29 ` Heiko Stübner 2013-06-11 11:29 ` Heiko Stübner 2013-06-11 11:30 ` [PATCH v3 3/7] mmc: dw_mmc-pltfm: move probe and remove below dt match table Heiko Stübner 2013-06-11 11:30 ` Heiko Stübner 2013-06-11 11:30 ` Heiko Stübner 2013-06-12 1:16 ` Seungwon Jeon 2013-06-12 1:16 ` Seungwon Jeon 2013-06-11 11:30 ` [PATCH v3 4/7] mmc: dw_mmc-pltfm: add Rockchip variant Heiko Stübner 2013-06-11 11:30 ` Heiko Stübner 2013-06-11 11:30 ` Heiko Stübner 2013-06-12 1:22 ` Seungwon Jeon 2013-06-12 1:22 ` Seungwon Jeon 2013-06-11 11:31 ` [PATCH v3 5/7] clk: add basic Rockchip rk3066a clock support Heiko Stübner 2013-06-11 11:31 ` Heiko Stübner 2013-06-11 11:31 ` Heiko Stübner 2013-06-11 20:06 ` Mike Turquette 2013-06-11 20:06 ` Mike Turquette 2013-06-12 22:45 ` Heiko Stübner 2013-06-12 22:45 ` Heiko Stübner 2013-06-12 22:45 ` Heiko Stübner 2013-06-12 23:02 ` Olof Johansson 2013-06-12 23:02 ` Olof Johansson 2013-06-12 23:02 ` Olof Johansson 2013-06-12 23:40 ` Heiko Stübner 2013-06-12 23:40 ` Heiko Stübner 2013-06-12 23:40 ` Heiko Stübner 2013-06-11 11:32 ` [PATCH v3 6/7] arm: add debug uarts for rockchip rk29xx and rk3xxx series Heiko Stübner 2013-06-11 11:32 ` Heiko Stübner 2013-06-11 11:32 ` Heiko Stübner 2013-06-11 11:32 ` [PATCH v3 7/7] arm: add basic support for Rockchip RK3066a boards Heiko Stübner 2013-06-11 11:32 ` Heiko Stübner 2013-06-11 11:32 ` Heiko Stübner
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAHp75VeQOSns1nL7p+rU1TS2stPuWibFtN_K9mFvzuGW+3WqOA@mail.gmail.com \ --to=andy.shevchenko@gmail.com \ --cc=arnd@arndb.de \ --cc=cjb@laptop.org \ --cc=devicetree-discuss@lists.ozlabs.org \ --cc=grant.likely@linaro.org \ --cc=heiko@sntech.de \ --cc=jh80.chung@samsung.com \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=mturquette@linaro.org \ --cc=olof@lixom.net \ --cc=rob.herring@calxeda.com \ --cc=tgih.jun@samsung.com \ --cc=thomas.petazzoni@free-electrons.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.