From: Jerome Brunet <jbrunet@baylibre.com>
To: Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
Neil Armstrong <narmstrong@baylibre.com>,
Kevin Hilman <khilman@baylibre.com>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@codeaurora.org>
Cc: Jerome Brunet <jbrunet@baylibre.com>,
linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, carlo@caione.org
Subject: [PATCH v2 0/2] clk: meson: MPLL fixes for Meson8b
Date: Fri, 7 Apr 2017 17:34:31 +0200 [thread overview]
Message-ID: <20170407153433.18640-1-jbrunet@baylibre.com> (raw)
MPLL clocks have been recently added to the Meson8b clock driver: [0]
On meson8b boards this unfortunately causes a division by zero error which
is fixed by patch #1 in this series.
While investigating this I found that there also seems to be a 32bit
overflow in the calculation in rate_from_params(), which is fixed by
patch #2 in this series.
Changes since v1: [1]
- Return an error code when the mpll parameters are out of the specified
range.
- As agreed with martin, use DIV_ROUND_UP_ULL instead of mul_u64_u32_div.
[0] http://lists.infradead.org/pipermail/linux-amlogic/2017-March/002757.html
[1] https://lkml.kernel.org/r/20170401130225.8811-1-martin.blumenstingl@googlemail.com
Martin Blumenstingl (2):
clk: meson: mpll: fix division by zero in rate_from_params
clk: meson: mpll: use 64bit math in rate_from_params
drivers/clk/meson/clk-mpll.c | 26 +++++++++++++++-----------
1 file changed, 15 insertions(+), 11 deletions(-)
--
2.9.3
WARNING: multiple messages have this Message-ID (diff)
From: jbrunet@baylibre.com (Jerome Brunet)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/2] clk: meson: MPLL fixes for Meson8b
Date: Fri, 7 Apr 2017 17:34:31 +0200 [thread overview]
Message-ID: <20170407153433.18640-1-jbrunet@baylibre.com> (raw)
MPLL clocks have been recently added to the Meson8b clock driver: [0]
On meson8b boards this unfortunately causes a division by zero error which
is fixed by patch #1 in this series.
While investigating this I found that there also seems to be a 32bit
overflow in the calculation in rate_from_params(), which is fixed by
patch #2 in this series.
Changes since v1: [1]
- Return an error code when the mpll parameters are out of the specified
range.
- As agreed with martin, use DIV_ROUND_UP_ULL instead of mul_u64_u32_div.
[0] http://lists.infradead.org/pipermail/linux-amlogic/2017-March/002757.html
[1] https://lkml.kernel.org/r/20170401130225.8811-1-martin.blumenstingl at googlemail.com
Martin Blumenstingl (2):
clk: meson: mpll: fix division by zero in rate_from_params
clk: meson: mpll: use 64bit math in rate_from_params
drivers/clk/meson/clk-mpll.c | 26 +++++++++++++++-----------
1 file changed, 15 insertions(+), 11 deletions(-)
--
2.9.3
WARNING: multiple messages have this Message-ID (diff)
From: jbrunet@baylibre.com (Jerome Brunet)
To: linus-amlogic@lists.infradead.org
Subject: [PATCH v2 0/2] clk: meson: MPLL fixes for Meson8b
Date: Fri, 7 Apr 2017 17:34:31 +0200 [thread overview]
Message-ID: <20170407153433.18640-1-jbrunet@baylibre.com> (raw)
MPLL clocks have been recently added to the Meson8b clock driver: [0]
On meson8b boards this unfortunately causes a division by zero error which
is fixed by patch #1 in this series.
While investigating this I found that there also seems to be a 32bit
overflow in the calculation in rate_from_params(), which is fixed by
patch #2 in this series.
Changes since v1: [1]
- Return an error code when the mpll parameters are out of the specified
range.
- As agreed with martin, use DIV_ROUND_UP_ULL instead of mul_u64_u32_div.
[0] http://lists.infradead.org/pipermail/linux-amlogic/2017-March/002757.html
[1] https://lkml.kernel.org/r/20170401130225.8811-1-martin.blumenstingl at googlemail.com
Martin Blumenstingl (2):
clk: meson: mpll: fix division by zero in rate_from_params
clk: meson: mpll: use 64bit math in rate_from_params
drivers/clk/meson/clk-mpll.c | 26 +++++++++++++++-----------
1 file changed, 15 insertions(+), 11 deletions(-)
--
2.9.3
next reply other threads:[~2017-04-07 15:34 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-07 15:34 Jerome Brunet [this message]
2017-04-07 15:34 ` [PATCH v2 0/2] clk: meson: MPLL fixes for Meson8b Jerome Brunet
2017-04-07 15:34 ` Jerome Brunet
2017-04-07 15:34 ` [PATCH v2 1/2] clk: meson: mpll: fix division by zero in rate_from_params Jerome Brunet
2017-04-07 15:34 ` Jerome Brunet
2017-04-07 15:34 ` Jerome Brunet
2017-04-07 15:39 ` Neil Armstrong
2017-04-07 15:39 ` Neil Armstrong
2017-04-07 15:39 ` Neil Armstrong
2017-04-07 15:34 ` [PATCH v2 2/2] clk: meson: mpll: use 64bit math " Jerome Brunet
2017-04-07 15:34 ` Jerome Brunet
2017-04-07 15:34 ` Jerome Brunet
2017-04-07 15:39 ` Neil Armstrong
2017-04-07 15:39 ` Neil Armstrong
2017-04-07 15:39 ` Neil Armstrong
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=20170407153433.18640-1-jbrunet@baylibre.com \
--to=jbrunet@baylibre.com \
--cc=carlo@caione.org \
--cc=khilman@baylibre.com \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=mturquette@baylibre.com \
--cc=narmstrong@baylibre.com \
--cc=sboyd@codeaurora.org \
/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: link
Be 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.