[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
<PH0PR22MB3789923F70E128DB14DCD073FA1F2@PH0PR22MB3789.namprd22.prod.outlook.com>
Date: Fri, 3 May 2024 07:35:44 +0000
From: FLAVIO SULIGOI <f.suligoi@...m.it>
To: 'Lee Jones' <lee@...nel.org>
CC: Daniel Thompson <daniel.thompson@...aro.org>, Jingoo Han
<jingoohan1@...il.com>, Helge Deller <deller@....de>, Dan Carpenter
<dan.carpenter@...aro.org>, "dri-devel@...ts.freedesktop.org"
<dri-devel@...ts.freedesktop.org>, "linux-fbdev@...r.kernel.org"
<linux-fbdev@...r.kernel.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>
Subject: RE: EXTERNAL: Re: (subset) [PATCH v1 1/1] backlight: mp3309c: fix
leds flickering in pwm mode
Hi Lee,
...
> Subject: Re: EXTERNAL: Re: (subset) [PATCH v1 1/1] backlight: mp3309c:
> fix leds flickering in pwm mode
>
> On Fri, 03 May 2024, FLAVIO SULIGOI wrote:
>
> > Hi Lee,
> >
> > ...
> >
> > > Subject: EXTERNAL: Re: (subset) [PATCH v1 1/1] backlight: mp3309c:
> > > fix leds flickering in pwm mode
> > >
> > > [Use caution with links & attachments]
> > >
> > >
> > >
> > > On Thu, 02 May 2024, Lee Jones wrote:
> > >
> > > > On Wed, 17 Apr 2024 17:31:05 +0200, Flavio Suligoi wrote:
> > > > > The mp3309 has two configuration registers, named according to
> > > > > their address (0x00 and 0x01).
> > > > > In the second register (0x01), the bit DIMS (Dimming Mode
> > > > > Select) must be always 0 (zero), in both analog (via i2c
> > > > > commands) and pwm dimming mode.
> > > > >
> > > > > In the initial driver version, the DIMS bit was set in pwm mode
> > > > > and reset in analog mode.
> > > > > But if the DIMS bit is set in pwm dimming mode and other devices
> > > > > are connected on the same i2c bus, every i2c commands on the bus
> > > > > generates a flickering on the LEDs powered by the mp3309c.
> > > > >
> > > > > [...]
> > > >
> > > > Applied, thanks!
> > > >
> > > > [1/1] backlight: mp3309c: fix leds flickering in pwm mode
> > > > commit: ce60cddc2abf61902dfca71d630624db95315124
> > >
> > > Applied, but in future it's; I2C, PWM and LED, thanks.
> >
> > Sorry for my question, but do you mean that I also have to add the
> > I2C, PWM and LED mailing lists in my messages related to the mp33309c
> patches?
>
> Just use proper capitalisation when you abbreviate the names these
> subsystems please.
Ah, ok, I got it!
>
> --
> Lee Jones [李琼斯]
Thanks for your explanation,
Flavio
Powered by blists - more mailing lists