Re: [PATCH 2/2] dt-bindings: display: bridge: lt8912b: document 'lontium, pn-swap' property

2024-04-11 Thread Alexandru Ardelean
On Sun, Apr 7, 2024 at 11:31 PM Dmitry Baryshkov wrote: > > On Tue, Apr 02, 2024 at 01:59:25PM +0300, Alexandru Ardelean wrote: > > On some HW designs, it's easier for the layout if the P/N pins are swapped. > > The driver currently has a DT property to do that. > > &

Re: [PATCH 1/2] drm/bridge: lt8912b: add support for P/N pin swap

2024-04-03 Thread Alexandru Ardelean
On Tue, Apr 2, 2024 at 7:53 PM Francesco Dolcini wrote: > > Hello Alexandru, thanks for your patch. > > On Tue, Apr 02, 2024 at 01:59:24PM +0300, Alexandru Ardelean wrote: > > On some HW designs, it's easier for the layout if the P/N pins are swapped. > > In those

Re: [PATCH 2/2] dt-bindings: display: bridge: lt8912b: document 'lontium,pn-swap' property

2024-04-03 Thread Alexandru Ardelean
On Tue, Apr 2, 2024 at 9:06 PM Conor Dooley wrote: > > On Tue, Apr 02, 2024 at 01:59:25PM +0300, Alexandru Ardelean wrote: > > On some HW designs, it's easier for the layout if the P/N pins are swapped. > > The driver currently has a DT property to do that. > > "

[PATCH 2/2] dt-bindings: display: bridge: lt8912b: document 'lontium, pn-swap' property

2024-04-02 Thread Alexandru Ardelean
On some HW designs, it's easier for the layout if the P/N pins are swapped. The driver currently has a DT property to do that. This change documents the 'lontium,pn-swap' property. Signed-off-by: Alexandru Ardelean --- .../devicetree/bindings/display/bridge/lontium,lt8912b.yaml | 6 ++ 1

[PATCH 1/2] drm/bridge: lt8912b: add support for P/N pin swap

2024-04-02 Thread Alexandru Ardelean
to configure the MIPI analog registers for P/N swap. Signed-off-by: Alexandru Ardelean --- drivers/gpu/drm/bridge/lontium-lt8912b.c | 25 +--- 1 file changed, 22 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/lontium-lt8912b.c b/drivers/gpu/drm/bridge

Re: [PATCH v4] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-20 Thread Alexandru Ardelean
On Wed, Jul 19, 2023 at 9:02 AM Alexandru Ardelean wrote: > > From: Bogdan Togorean > > For ADV7533 and ADV7535 low refresh rate is selected using > bits [3:2] of 0x4a main register. > So depending on ADV model write 0xfb or 0x4a register. > > Fixes: 2437e7cd88e8 ("

[PATCH v4] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-19 Thread Alexandru Ardelean
y: Nuno Sa Signed-off-by: Bogdan Togorean Signed-off-by: Alexandru Ardelean --- drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c b/drivers/gpu/drm/bridge/adv7511/adv7511_d

Re: [PATCH v3] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-18 Thread Alexandru Ardelean
On Tue, Jul 18, 2023 at 11:50 AM Robert Foss wrote: > > On Tue, Jul 18, 2023 at 10:42 AM Alexandru Ardelean wrote: > > > > From: Bogdan Togorean > > > > For ADV7533 and ADV7535 low refresh rate is selected using > > bits [3:2] of 0x4a main register. >

[PATCH v3] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-18 Thread Alexandru Ardelean
n Togorean Signed-off-by: Alexandru Ardelean --- Changelog v2 -> v3: * https://lore.kernel.org/dri-devel/1c3fde3a873b0f948d3c4d37107c5bb67dc9f7bb.ca...@gmail.com/T/#u * Added my S-o-b tag back Changelog v1 -> v2: * https://lore.kernel.org/dri-devel/20190716131005.761-1-bogdan.tog

Re: [PATCH V2] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-18 Thread Alexandru Ardelean
On Tue, Jul 18, 2023 at 10:00 AM Nuno Sá wrote: > > On Tue, 2023-07-18 at 09:28 +0300, Alexandru Ardelean wrote: > > From: Bogdan Togorean > > > > For ADV7533 and ADV7535 low refresh rate is selected using > > bits [3:2] of 0x4a main register. > > So depen

[PATCH V2] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-18 Thread Alexandru Ardelean
From: Bogdan Togorean For ADV7533 and ADV7535 low refresh rate is selected using bits [3:2] of 0x4a main register. So depending on ADV model write 0xfb or 0x4a register. Fixes: 2437e7cd88e8 ("drm/bridge: adv7533: Initial support for ADV7533") Reviewed-by: Nuno Sa Signed-off-by: Bogdan Togorean

Re: [PATCH] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-18 Thread Alexandru Ardelean
On Fri, Jul 14, 2023 at 10:48 AM Nuno Sá wrote: > > Hey Alex, Hey Nuno :) > > On Thu, 2023-07-13 at 17:19 -0300, Fabio Estevam wrote: > > On Wed, May 17, 2023 at 4:08 AM Alexandru Ardelean wrote: > > > > > > From: Bogdan Togorean > > > &

Re: [PATCH] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-07-13 Thread Alexandru Ardelean
On Tue, May 16, 2023 at 7:44 PM Alexandru Ardelean wrote: > > From: Bogdan Togorean > > For ADV7533 and ADV7535 low refresh rate is selected using > bits [3:2] of 0x4a main register. > So depending on ADV model write 0xfb or 0x4a register. > Ping on this patch :) &g

Re: [PATCH v2 1/2] dt-bindings: backlight: document new property default-brightness-level

2023-06-23 Thread Alexandru Ardelean
On Thu, Jun 22, 2023 at 5:13 AM Rob Herring wrote: > > On Thu, Jun 22, 2023 at 12:54:56AM +0300, Alexandru Ardelean wrote: > > From: Yannick Fertre > > > > Add documentation for new default-brightness-level property. > > Why? I'll admit, I liked the fact that

[PATCH v2 1/2] dt-bindings: backlight: document new property default-brightness-level

2023-06-21 Thread Alexandru Ardelean
From: Yannick Fertre Add documentation for new default-brightness-level property. Signed-off-by: Yannick Fertre Signed-off-by: Alexandru Ardelean --- Link to original patch: https://github.com/STMicroelectronics/linux/commit/c4067d7bd883c6fa14ffd49892c4ce663cdafe98 Changelog v1 ->

[PATCH v2 2/2] backlight: gpio_backlight: add new property default-brightness-level

2023-06-21 Thread Alexandru Ardelean
From: Yannick Fertre Add new property to set a brightness by default at probe. Reviewed-by: Daniel Thompson Signed-off-by: Yannick Fertre Signed-off-by: Alexandru Ardelean --- Link to original patch: https://github.com/STMicroelectronics/linux/commit

Re: [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level

2023-05-29 Thread Alexandru Ardelean
On Fri, May 26, 2023 at 3:05 PM Philippe CORNU wrote: > > > > On 5/19/23 22:05, Alexandru Ardelean wrote: > > From: Yannick Fertre > > > > Add documentation for new default-brightness-level property. > > > > Reviewed-by: Philippe CORNU > > Hi

Re: [PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level

2023-05-29 Thread Alexandru Ardelean
On Fri, May 26, 2023 at 1:20 PM Daniel Thompson wrote: > > On Fri, May 19, 2023 at 11:05:20PM +0300, Alexandru Ardelean wrote: > > From: Yannick Fertre > > > > Add documentation for new default-brightness-level property. > > > > Reviewed-by: Philippe CORN

Re: [PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level

2023-05-29 Thread Alexandru Ardelean
On Fri, May 26, 2023 at 3:04 PM Philippe CORNU wrote: > > > On 5/19/23 22:05, Alexandru Ardelean wrote: > > From: Yannick Fertre > > > > Add new property to set a brightness by default at probe. > > > > Reviewed-by: Philippe CORNU > > Hi Alexandru,

[PATCH 1/2] backlight: gpio_backlight: add new property default-brightness-level

2023-05-20 Thread Alexandru Ardelean
From: Yannick Fertre Add new property to set a brightness by default at probe. Reviewed-by: Philippe CORNU Signed-off-by: Yannick Fertre Signed-off-by: Alexandru Ardelean --- Link to original patch: https://github.com/STMicroelectronics/linux/commit

[PATCH 2/2] dt-bindings: backlight: document new property default-brightness-level

2023-05-20 Thread Alexandru Ardelean
From: Yannick Fertre Add documentation for new default-brightness-level property. Reviewed-by: Philippe CORNU Signed-off-by: Yannick Fertre Signed-off-by: Alexandru Ardelean --- Link to original patch: https://github.com/STMicroelectronics/linux/commit

[PATCH] drm: adv7511: Fix low refresh rate register for ADV7533/5

2023-05-17 Thread Alexandru Ardelean
From: Bogdan Togorean For ADV7533 and ADV7535 low refresh rate is selected using bits [3:2] of 0x4a main register. So depending on ADV model write 0xfb or 0x4a register. Signed-off-by: Bogdan Togorean Signed-off-by: Alexandru Ardelean --- drivers/gpu/drm/i2c/adv7511.c | 9 +++-- 1 file

Re: [PATCH 08/15] iio: buffer-dma: split iio_dma_buffer_fileio_free() function

2021-11-16 Thread Alexandru Ardelean
On Mon, Nov 15, 2021 at 4:20 PM Paul Cercueil wrote: > > From: Alexandru Ardelean > > A part of the logic in the iio_dma_buffer_exit() is required for the change > to add mmap support to IIO buffers. > This change splits the logic into a separate function, which will

Re: [PATCH 14/15] iio: buffer-dmaengine: Add support for cyclic buffers

2021-11-16 Thread Alexandru Ardelean
On Mon, Nov 15, 2021 at 4:23 PM Paul Cercueil wrote: > > Handle the IIO_BUFFER_DMABUF_CYCLIC flag to support cyclic buffers. > Reviewed-by: Alexandru Ardelean > Signed-off-by: Paul Cercueil > --- > drivers/iio/buffer/industrialio-buffer-dma.c | 1 + > .../iio/buffe

Re: [PATCH 13/15] iio: core: Add support for cyclic buffers

2021-11-16 Thread Alexandru Ardelean
until the buffer is disabled. > Reviewed-by: Alexandru Ardelean > Signed-off-by: Paul Cercueil > --- > drivers/iio/industrialio-buffer.c | 5 + > include/uapi/linux/iio/buffer.h | 3 ++- > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/iio

Re: [PATCH 05/15] iio: buffer-dmaengine: Support specifying buffer direction

2021-11-16 Thread Alexandru Ardelean
On Mon, Nov 15, 2021 at 4:20 PM Paul Cercueil wrote: > > Update the devm_iio_dmaengine_buffer_setup() function to support > specifying the buffer direction. > > Update the iio_dmaengine_buffer_submit() function to handle input > buffers as well as output buffers. > R

Re: [PATCH 01/15] iio: buffer-dma: Get rid of incoming/outgoing queues

2021-11-16 Thread Alexandru Ardelean
blocks. It is much easier to > just check each block's state manually, and keep a counter for the next > block to dequeue. > > Since the new DMABUF based API wouldn't use these incoming and outgoing > queues anyway, getting rid of them now makes the upcoming changes > simpler

Re: [PATCH 03/15] iio: buffer-dma: Use round_down() instead of rounddown()

2021-11-16 Thread Alexandru Ardelean
On Mon, Nov 15, 2021 at 4:19 PM Paul Cercueil wrote: > > We know that the buffer's alignment will always be a power of two; > therefore, we can use the faster round_down() macro. > Reviewed-by: Alexandru Ardelean > Signed-off-by: Paul Cercueil > --- > drivers/iio/buffe

Re: [PATCH 04/15] iio: buffer-dma: Enable buffer write support

2021-11-16 Thread Alexandru Ardelean
nput buffers, since in both cases we > count the exact same thing (the number of bytes in each available > block). > A few comments inline from me. Otherwise fine from me. Reviewed-by: Alexandru Ardelean > Signed-off-by: Paul Cercueil > --- > drivers/iio/buffer/industrialio-buffe

Re: [PATCH 02/15] iio: buffer-dma: Remove unused iio_buffer_block struct

2021-11-16 Thread Alexandru Ardelean
On Mon, Nov 15, 2021 at 4:19 PM Paul Cercueil wrote: > > This structure was never used anywhere, so it can safely be dropped. > > It will later be re-introduced as a different structure in a > different header. Reviewed-by: Alexandru Ardelean > > Signed-off-by: Paul Cercu

Re: [PATCH 06/15] iio: buffer-dmaengine: Enable write support

2021-11-16 Thread Alexandru Ardelean
e comment about iio_dma_buffer_space_available() in patch 4 But otherwise: Reviewed-by: Alexandru Ardelean > Signed-off-by: Paul Cercueil > --- > drivers/iio/buffer/industrialio-buffer-dmaengine.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/iio/buffer/industrialio-buffer-dm

[PATCH] staging: fbtft: fbtft-core: Fix last line displayed on fbcon

2019-10-11 Thread Alexandru Ardelean
From: Michael Hennerich For the special case when fbtft_mkdirty() is called with with -1 for the y coordinate, the height is truncated by 1. This isn't required, and causes the last line to not update. Signed-off-by: Michael Hennerich Signed-off-by: Alexandru Ardelean --- drivers/staging

[PATCH 2/3][V2] treewide: rename match_string() -> __match_string()

2019-05-28 Thread Alexandru Ardelean
differs. Signed-off-by: Alexandru Ardelean --- arch/powerpc/xmon/xmon.c | 2 +- arch/x86/kernel/cpu/mtrr/if.c| 2 +- drivers/ata/pata_hpt366.c| 2 +- drivers/ata/pata_hpt37x.c| 2 +- drivers/base/d

[PATCH 1/3][V2] lib: fix match_string() helper on -1 array size

2019-05-28 Thread Alexandru Ardelean
on doing some cosmetic changes, which are aimed at grouping the users of `match_string()`. Signed-off-by: Alexandru Ardelean --- Changelog v1 -> v2: * split the initial series into just 3 patches that fix the `match_string()` helper and start introducing a new version of this helper, wh

[PATCH 3/3][V2] lib: re-introduce new match_string() helper/macro

2019-05-28 Thread Alexandru Ardelean
, and will be reviewed by each subsystem. Signed-off-by: Alexandru Ardelean --- include/linux/string.h | 9 + 1 file changed, 9 insertions(+) diff --git a/include/linux/string.h b/include/linux/string.h index 7149fcdf62df..34491b075449 100644 --- a/include/linux/string.h +++ b/include/linux/string.h

[PATCH 01/16] lib: fix match_string() helper on -1 array size

2019-05-09 Thread Alexandru Ardelean
on doing some cosmetic changes, which are aimed at grouping the users of `match_string()`. Signed-off-by: Alexandru Ardelean --- lib/string.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/string.c b/lib/string.c index 3ab861c1a857..76edb7bf76cb 100644 --- a/lib

[PATCH 15/16] video: fbdev: pxafb: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
The `lcd_types` array is a static array of strings. Using match_string() (which computes the array size via ARRAY_SIZE()) is possible. This reduces the array by 1 element, since the NULL (at the end of the array) is no longer needed. Signed-off-by: Alexandru Ardelean --- drivers/video/fbdev

[PATCH 10/16] pinctrl: armada-37xx: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
off-by: Alexandru Ardelean --- drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c index 07a5bcaa0067..68b0db5ef5e9 100644 --- a/drivers/pinc

[PATCH 08/16] cpufreq/intel_pstate: remove NULL entry + use match_string()

2019-05-09 Thread Alexandru Ardelean
The change is mostly cosmetic. The `energy_perf_strings` array is static, so match_string() can be used (which will implicitly do a ARRAY_SIZE(energy_perf_strings)). The only small benefit here, is the reduction of the array size by 1 element. Signed-off-by: Alexandru Ardelean --- drivers

[PATCH 12/16] rdmacg: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
The `rdmacg_resource_names` array is a static array of strings. Using match_string() (which computes the array size via ARRAY_SIZE()) is possible. The change is mostly cosmetic. No functionality change. Signed-off-by: Alexandru Ardelean --- kernel/cgroup/rdma.c | 2 +- 1 file changed, 1

[PATCH 13/16] drm/edid: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
The `generic_edid_name` is a static array of strings. Using match_string() (which computes the array size via ARRAY_SIZE()) is possible. The change is mostly cosmetic. No functionality change. Signed-off-by: Alexandru Ardelean --- drivers/gpu/drm/drm_edid_load.c | 2 +- 1 file changed, 1

[PATCH 00/16] treewide: fix match_string() helper when array size

2019-05-09 Thread Alexandru Ardelean
y. The first patch is important, the others can be dropped. Signed-off-by: Alexandru Ardelean Alexandru Ardelean (16): lib: fix match_string() helper when array size is positive treewide: rename match_string() -> __match_string() lib,treewide: add new match_string() helper/macro power

[PATCH 02/16] treewide: rename match_string() -> __match_string()

2019-05-09 Thread Alexandru Ardelean
ositive value does not make any difference: the iteration will stop at the first NULL element. Signed-off-by: Alexandru Ardelean --- arch/powerpc/xmon/xmon.c | 2 +- arch/x86/kernel/cpu/mtrr/if.c| 2 +- drivers/ata/pata_h

[PATCH 05/16] ALSA: oxygen: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
and ignore the NULL. Signed-off-by: Alexandru Ardelean --- sound/pci/oxygen/oxygen_mixer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/oxygen/oxygen_mixer.c b/sound/pci/oxygen/oxygen_mixer.c index 13c2fb75fd71..961fd1cbc712 100644 --- a/sound/pci/oxygen

[PATCH 16/16] sched: debug: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
The `sched_feat_names` array is a static array of strings. Using match_string() (which computes the array size via ARRAY_SIZE()) is possible. The change is mostly cosmetic. No functionality change. Signed-off-by: Alexandru Ardelean --- kernel/sched/debug.c | 2 +- 1 file changed, 1 insertion

[PATCH 14/16] staging: gdm724x: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
The `DRIVER_STRING` array is a static array of strings. Using match_string() (which computes the array size via ARRAY_SIZE()) is possible. The change is mostly cosmetic. No functionality change. Signed-off-by: Alexandru Ardelean --- drivers/staging/gdm724x/gdm_tty.c | 3 +-- 1 file changed, 1

[PATCH 11/16] mm/vmpressure.c: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
will be computed automatically, which would only help if they ever get expanded. Signed-off-by: Alexandru Ardelean --- mm/vmpressure.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/vmpressure.c b/mm/vmpressure.c index d43f33139568..b8149924f078 100644 --- a/mm/vmpressure.c

[PATCH 04/16] powerpc/xmon: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
the NULL. This would technically allow for "reserved" regs, though here it's not the case. Signed-off-by: Alexandru Ardelean --- arch/powerpc/xmon/xmon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c index ef

[PATCH 01/16] lib: fix match_string() helper when array size is positive

2019-05-09 Thread Alexandru Ardelean
in the series will focus on doing some cosmetic changes, which are aimed at grouping the users of `match_string()`. Signed-off-by: Alexandru Ardelean --- lib/string.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/string.c b/lib/string.c index 3ab861c1a857

[PATCH 07/16] device connection: use new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
(on the struct), so using the match_string() (which does an ARRAY_SIZE((con->endpoint)) should be fine. The recent change to match_string() (to ignore NULL entries up to the size of the array) shouldn't affect this. Signed-off-by: Alexandru Ardelean --- drivers/base/devcon.c | 2 +- 1 file changed

[PATCH 03/16] lib,treewide: add new match_string() helper/macro

2019-05-09 Thread Alexandru Ardelean
This change re-introduces `match_string()` as a macro that uses ARRAY_SIZE() to compute the size of the array. The macro is added in all the places that do `match_string(_a, ARRAY_SIZE(_a), s)`, since the change is pretty straightforward. Signed-off-by: Alexandru Ardelean --- drivers/clk/bcm

[PATCH 06/16] x86/mtrr: use new match_string() helper + add gaps == minor fix

2019-05-09 Thread Alexandru Ardelean
n't), but this fixes that. Signed-off-by: Alexandru Ardelean --- arch/x86/kernel/cpu/mtrr/if.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/arch/x86/kernel/cpu/mtrr/if.c b/arch/x86/kernel/cpu/mtrr/if.c index 4ec7a5f7b94c..e67820a044cc 100644 --- a/arch/x86/kernel/cpu/mtrr