Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
On Tue, 24 Mar 2020, Uwe Kleine-König wrote: > On Tue, Mar 24, 2020 at 02:24:41PM +, Lee Jones wrote: > > On Tue, 24 Mar 2020, Daniel Thompson wrote: > > > > > On Tue, Mar 24, 2020 at 01:57:35PM +0100, Uwe Kleine-König wrote: > > > > Hello Lee, > > > > > > > > On Tue, Mar 24, 2020 at 11:07:10AM +, Lee Jones wrote: > > > > > On Fri, 20 Mar 2020, Lee Jones wrote: > > > > > > > > > > > On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > > > > > > > > > > > > > Since the PWM framework is switching struct pwm_state.period's > > > > > > > datatype > > > > > > > to u64, prepare for this transition by using div_u64 to handle a > > > > > > > 64-bit > > > > > > > dividend instead of a straight division operation. > > > > > > > > > > > > > > Cc: Lee Jones > > > > > > > Cc: Daniel Thompson > > > > > > > Cc: Jingoo Han > > > > > > > Cc: Bartlomiej Zolnierkiewicz > > > > > > > Cc: linux-...@vger.kernel.org > > > > > > > Cc: dri-devel@lists.freedesktop.org > > > > > > > Cc: linux-fb...@vger.kernel.org > > > > > > > > > > > > > > Signed-off-by: Guru Das Srinagesh > > > > > > > Reviewed-by: Daniel Thompson > > > > > > > --- > > > > > > > drivers/video/backlight/pwm_bl.c | 3 ++- > > > > > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > > > > > > > Can this patch be taken on its own? > > > > > > > > > > Hello ... > > > > > > > > Conceptually it can. As the last patch depends on this one (and the > > > > others) some coordination might be beneficial. But that's up to Thierry > > > > to decide how (and if) he want this series to be applied. > > > > > > ... and on the backlight side we definitely need to know about the "if" > > > otherwise there's no point in taking it. > > > > Right. > > > > I'm happy to wait for Thierry. Although this isn't the only set he's > > currently blocking. Is he okay? On holiday perhaps? > > The newest commit by him in next is from last week. My guess is he > just didn't come around yet to care for the PWM duties. Looks like we missed the last release. Let's hope we don't miss the next one also. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
On Tue, Mar 24, 2020 at 02:24:41PM +, Lee Jones wrote: > On Tue, 24 Mar 2020, Daniel Thompson wrote: > > > On Tue, Mar 24, 2020 at 01:57:35PM +0100, Uwe Kleine-König wrote: > > > Hello Lee, > > > > > > On Tue, Mar 24, 2020 at 11:07:10AM +, Lee Jones wrote: > > > > On Fri, 20 Mar 2020, Lee Jones wrote: > > > > > > > > > On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > > > > > > > > > > > Since the PWM framework is switching struct pwm_state.period's > > > > > > datatype > > > > > > to u64, prepare for this transition by using div_u64 to handle a > > > > > > 64-bit > > > > > > dividend instead of a straight division operation. > > > > > > > > > > > > Cc: Lee Jones > > > > > > Cc: Daniel Thompson > > > > > > Cc: Jingoo Han > > > > > > Cc: Bartlomiej Zolnierkiewicz > > > > > > Cc: linux-...@vger.kernel.org > > > > > > Cc: dri-devel@lists.freedesktop.org > > > > > > Cc: linux-fb...@vger.kernel.org > > > > > > > > > > > > Signed-off-by: Guru Das Srinagesh > > > > > > Reviewed-by: Daniel Thompson > > > > > > --- > > > > > > drivers/video/backlight/pwm_bl.c | 3 ++- > > > > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > > > > > Can this patch be taken on its own? > > > > > > > > Hello ... > > > > > > Conceptually it can. As the last patch depends on this one (and the > > > others) some coordination might be beneficial. But that's up to Thierry > > > to decide how (and if) he want this series to be applied. > > > > ... and on the backlight side we definitely need to know about the "if" > > otherwise there's no point in taking it. > > Right. > > I'm happy to wait for Thierry. Although this isn't the only set he's > currently blocking. Is he okay? On holiday perhaps? The newest commit by him in next is from last week. My guess is he just didn't come around yet to care for the PWM duties. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König| Industrial Linux Solutions | https://www.pengutronix.de/ | ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
On Tue, 24 Mar 2020, Daniel Thompson wrote: > On Tue, Mar 24, 2020 at 01:57:35PM +0100, Uwe Kleine-König wrote: > > Hello Lee, > > > > On Tue, Mar 24, 2020 at 11:07:10AM +, Lee Jones wrote: > > > On Fri, 20 Mar 2020, Lee Jones wrote: > > > > > > > On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > > > > > > > > > Since the PWM framework is switching struct pwm_state.period's > > > > > datatype > > > > > to u64, prepare for this transition by using div_u64 to handle a > > > > > 64-bit > > > > > dividend instead of a straight division operation. > > > > > > > > > > Cc: Lee Jones > > > > > Cc: Daniel Thompson > > > > > Cc: Jingoo Han > > > > > Cc: Bartlomiej Zolnierkiewicz > > > > > Cc: linux-...@vger.kernel.org > > > > > Cc: dri-devel@lists.freedesktop.org > > > > > Cc: linux-fb...@vger.kernel.org > > > > > > > > > > Signed-off-by: Guru Das Srinagesh > > > > > Reviewed-by: Daniel Thompson > > > > > --- > > > > > drivers/video/backlight/pwm_bl.c | 3 ++- > > > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > > > Can this patch be taken on its own? > > > > > > Hello ... > > > > Conceptually it can. As the last patch depends on this one (and the > > others) some coordination might be beneficial. But that's up to Thierry > > to decide how (and if) he want this series to be applied. > > ... and on the backlight side we definitely need to know about the "if" > otherwise there's no point in taking it. Right. I'm happy to wait for Thierry. Although this isn't the only set he's currently blocking. Is he okay? On holiday perhaps? -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
On Tue, Mar 24, 2020 at 01:57:35PM +0100, Uwe Kleine-König wrote: > Hello Lee, > > On Tue, Mar 24, 2020 at 11:07:10AM +, Lee Jones wrote: > > On Fri, 20 Mar 2020, Lee Jones wrote: > > > > > On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > > > > > > > Since the PWM framework is switching struct pwm_state.period's datatype > > > > to u64, prepare for this transition by using div_u64 to handle a 64-bit > > > > dividend instead of a straight division operation. > > > > > > > > Cc: Lee Jones > > > > Cc: Daniel Thompson > > > > Cc: Jingoo Han > > > > Cc: Bartlomiej Zolnierkiewicz > > > > Cc: linux-...@vger.kernel.org > > > > Cc: dri-devel@lists.freedesktop.org > > > > Cc: linux-fb...@vger.kernel.org > > > > > > > > Signed-off-by: Guru Das Srinagesh > > > > Reviewed-by: Daniel Thompson > > > > --- > > > > drivers/video/backlight/pwm_bl.c | 3 ++- > > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > Can this patch be taken on its own? > > > > Hello ... > > Conceptually it can. As the last patch depends on this one (and the > others) some coordination might be beneficial. But that's up to Thierry > to decide how (and if) he want this series to be applied. ... and on the backlight side we definitely need to know about the "if" otherwise there's no point in taking it. Daniel. ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
Hello Lee, On Tue, Mar 24, 2020 at 11:07:10AM +, Lee Jones wrote: > On Fri, 20 Mar 2020, Lee Jones wrote: > > > On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > > > > > Since the PWM framework is switching struct pwm_state.period's datatype > > > to u64, prepare for this transition by using div_u64 to handle a 64-bit > > > dividend instead of a straight division operation. > > > > > > Cc: Lee Jones > > > Cc: Daniel Thompson > > > Cc: Jingoo Han > > > Cc: Bartlomiej Zolnierkiewicz > > > Cc: linux-...@vger.kernel.org > > > Cc: dri-devel@lists.freedesktop.org > > > Cc: linux-fb...@vger.kernel.org > > > > > > Signed-off-by: Guru Das Srinagesh > > > Reviewed-by: Daniel Thompson > > > --- > > > drivers/video/backlight/pwm_bl.c | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > Can this patch be taken on its own? > > Hello ... Conceptually it can. As the last patch depends on this one (and the others) some coordination might be beneficial. But that's up to Thierry to decide how (and if) he want this series to be applied. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König| Industrial Linux Solutions | https://www.pengutronix.de/ | ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
On Fri, 20 Mar 2020, Lee Jones wrote: > On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > > > Since the PWM framework is switching struct pwm_state.period's datatype > > to u64, prepare for this transition by using div_u64 to handle a 64-bit > > dividend instead of a straight division operation. > > > > Cc: Lee Jones > > Cc: Daniel Thompson > > Cc: Jingoo Han > > Cc: Bartlomiej Zolnierkiewicz > > Cc: linux-...@vger.kernel.org > > Cc: dri-devel@lists.freedesktop.org > > Cc: linux-fb...@vger.kernel.org > > > > Signed-off-by: Guru Das Srinagesh > > Reviewed-by: Daniel Thompson > > --- > > drivers/video/backlight/pwm_bl.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > Can this patch be taken on its own? Hello ... > > diff --git a/drivers/video/backlight/pwm_bl.c > > b/drivers/video/backlight/pwm_bl.c > > index efb4efc..3e5dbcf 100644 > > --- a/drivers/video/backlight/pwm_bl.c > > +++ b/drivers/video/backlight/pwm_bl.c > > @@ -625,7 +625,8 @@ static int pwm_backlight_probe(struct platform_device > > *pdev) > > pb->scale = data->max_brightness; > > } > > > > - pb->lth_brightness = data->lth_brightness * (state.period / pb->scale); > > + pb->lth_brightness = data->lth_brightness * (div_u64(state.period, > > + pb->scale)); > > > > props.type = BACKLIGHT_RAW; > > props.max_brightness = data->max_brightness; > -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
On Thu, 19 Mar 2020, Guru Das Srinagesh wrote: > Since the PWM framework is switching struct pwm_state.period's datatype > to u64, prepare for this transition by using div_u64 to handle a 64-bit > dividend instead of a straight division operation. > > Cc: Lee Jones > Cc: Daniel Thompson > Cc: Jingoo Han > Cc: Bartlomiej Zolnierkiewicz > Cc: linux-...@vger.kernel.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-fb...@vger.kernel.org > > Signed-off-by: Guru Das Srinagesh > Reviewed-by: Daniel Thompson > --- > drivers/video/backlight/pwm_bl.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Can this patch be taken on its own? > diff --git a/drivers/video/backlight/pwm_bl.c > b/drivers/video/backlight/pwm_bl.c > index efb4efc..3e5dbcf 100644 > --- a/drivers/video/backlight/pwm_bl.c > +++ b/drivers/video/backlight/pwm_bl.c > @@ -625,7 +625,8 @@ static int pwm_backlight_probe(struct platform_device > *pdev) > pb->scale = data->max_brightness; > } > > - pb->lth_brightness = data->lth_brightness * (state.period / pb->scale); > + pb->lth_brightness = data->lth_brightness * (div_u64(state.period, > + pb->scale)); > > props.type = BACKLIGHT_RAW; > props.max_brightness = data->max_brightness; -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH v11 10/12] backlight: pwm_bl: Use 64-bit division function
Since the PWM framework is switching struct pwm_state.period's datatype to u64, prepare for this transition by using div_u64 to handle a 64-bit dividend instead of a straight division operation. Cc: Lee Jones Cc: Daniel Thompson Cc: Jingoo Han Cc: Bartlomiej Zolnierkiewicz Cc: linux-...@vger.kernel.org Cc: dri-devel@lists.freedesktop.org Cc: linux-fb...@vger.kernel.org Signed-off-by: Guru Das Srinagesh Reviewed-by: Daniel Thompson --- drivers/video/backlight/pwm_bl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index efb4efc..3e5dbcf 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -625,7 +625,8 @@ static int pwm_backlight_probe(struct platform_device *pdev) pb->scale = data->max_brightness; } - pb->lth_brightness = data->lth_brightness * (state.period / pb->scale); + pb->lth_brightness = data->lth_brightness * (div_u64(state.period, + pb->scale)); props.type = BACKLIGHT_RAW; props.max_brightness = data->max_brightness; -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel