From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933697AbdEORsB (ORCPT ); Mon, 15 May 2017 13:48:01 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:51638 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755195AbdEORsA (ORCPT ); Mon, 15 May 2017 13:48:00 -0400 Date: Mon, 15 May 2017 19:47:39 +0200 (CEST) From: Stefan Wahren To: Lee Jones , Eric Anholt , Michael Turquette , Stephen Boyd , Florian Fainelli Cc: Phil Elwell , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org Message-ID: <1373231207.445862.1494870459462@email.1und1.de> In-Reply-To: <20170515173504.11337-1-eric@anholt.net> References: <20170515173504.11337-1-eric@anholt.net> Subject: Re: [PATCH] clk: bcm2835: Correct the prediv logic MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Priority: 3 Importance: Medium X-Mailer: Open-Xchange Mailer v7.8.1-Rev34 X-Originating-Client: open-xchange-appsuite X-Provags-ID: V03:K0:KiRBu46VGDIVpPznhZr9B3iUfc0NQUZdM0qtVKMmKgl0LBUfvBx +3WnZY1An+vVK72eCKaIC0pyAhawwDCBxgy4ethCgccL2qrJDwu827c96JFb2ojxXp7C9Da H+glIGsxChQjshyUctul9s/+pDo+BzLKGsGVqJCQT7fZD+hzUBanbKJx8Yb407lb2LeayRL 35YBC3rpl+e7PIta1HV8Q== X-UI-Out-Filterresults: notjunk:1;V01:K0:a3sGR1py8Us=:B90O0gfGJRTcrZI68EMrR2 XiQe1wotpxfinDgLwdpObdZTaRZ2dvYwbcnrDlQ5hckKvg49E+MhRCJIpQmRw9L4cdLPQxFfv UudCnQ/8bm/Vv5IoiAGlhZFrMuljQLcvW3HFRfGt32jqa+QBxrNzfYd4MVGi9X+GuhSQ9Qs83 UUePxY5c+UwMKFjN2TCm4cllzOY4teK7ruvD5UAhOl5PhnK16z5X/i2q5cPnG3TFegkkssJt2 9P/jhmg2s7Uj2yL3OV+xKV1AgtJ0N88SvCDCU01X4qhcQE71VSDNQQgaDszu2aYJB0y1aWJrW zWkc5O5Nutuy+7AAQTHBakKMJQvg7IbFXKcabVszQ7aeIMgpDG3y9+eQDAIKUtv4sejDdIiwN 7CCRboUrlm3xzg53V7Gx9+WNErcoHMi+tVCdlEAO58EofgRCFqH30rM60xFf42W2amG8zbqni aLFKKiAM7w59lqAYecbympNhBGOURzMbvsOOwO4OS535hFRRM0iky2mLdv/07A7rqYoTi0es5 1bD7vXGB7ddLfLIPX0dRUtG7lQsbILlh34f13irZbe95fwNnIxUJWkb/+a5zF/KhBgeo28gpz T5iRCn5V/p9JOwV5y9eRxPekosv6tH2NYp00NEPG7ZjXjFAZJ9CaoL/nmZXKVpA8e0QCMq2lB cDCUCZn3nC1MBDCam9EQGibW1ifyHnGG+6i1cGTivH/ttlSp4mRyqZapC9iMqzu8Rgkhfg+4Y II0amT8+d826Evf4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Eric, > Eric Anholt hat am 15. Mai 2017 um 19:35 geschrieben: > > > From: Phil Elwell > > If a clock has the prediv flag set, both the integer and fractional > parts must be scaled when calculating the resulting frequency. > > Signed-off-by: Phil Elwell > Signed-off-by: Eric Anholt > --- > > While this is a bugfix, I haven't put a "Fixes:" line in here to get > it automatically backported to stable. sorry, i can't follow. How should this happen without "Fixes: "? > We had trouble with the > out-of-tree DSI panel driver, at least: Our old set_rate() didn't > work, because the new PLL was just barely too fast to get the integer > PLL divider we needed. We may run into similar troubles > elsewhere. --anholt > From mboxrd@z Thu Jan 1 00:00:00 1970 From: stefan.wahren@i2se.com (Stefan Wahren) Date: Mon, 15 May 2017 19:47:39 +0200 (CEST) Subject: [PATCH] clk: bcm2835: Correct the prediv logic In-Reply-To: <20170515173504.11337-1-eric@anholt.net> References: <20170515173504.11337-1-eric@anholt.net> Message-ID: <1373231207.445862.1494870459462@email.1und1.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Eric, > Eric Anholt hat am 15. Mai 2017 um 19:35 geschrieben: > > > From: Phil Elwell > > If a clock has the prediv flag set, both the integer and fractional > parts must be scaled when calculating the resulting frequency. > > Signed-off-by: Phil Elwell > Signed-off-by: Eric Anholt > --- > > While this is a bugfix, I haven't put a "Fixes:" line in here to get > it automatically backported to stable. sorry, i can't follow. How should this happen without "Fixes: "? > We had trouble with the > out-of-tree DSI panel driver, at least: Our old set_rate() didn't > work, because the new PLL was just barely too fast to get the integer > PLL divider we needed. We may run into similar troubles > elsewhere. --anholt >