Changes the elida-kd35t133 panel to use multi style functions for
improved error handling.
Reviewed-by: Jessica Zhang
Reviewed-by: Dmitry Baryshkov
Signed-off-by: Tejas Vipin
---
Changes in v3:
- Added back bytes that were removed
- Replaced mipi_dsi_dcs_write_buffer_multi with
On 9/25/24 1:25 AM, Doug Anderson wrote:
> Hi,
>
> On Mon, Sep 23, 2024 at 5:33 AM Tejas Vipin wrote:
>>
>> -static int kd35t133_init_sequence(struct kd35t133 *ctx)
>> +static void kd35t133_init_sequence(struct mipi_dsi_multi_context *dsi_ctx)
>> {
>&
Changes the elida-kd35t133 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
Changes in v2:
- Changed mipi_dsi_dcs_write to mipi_dsi_dcs_write_buffer_multi
- Cleaned up error handling
Link to v1:
https://lore.kernel.org/all
On 9/20/24 9:59 PM, Dmitry Baryshkov wrote:
> On Tue, Sep 17, 2024 at 12:47:10PM GMT, Tejas Vipin wrote:
>> Changes the elida-kd35t133 panel to use multi style functions for
>> improved error handling.
>>
>> Signed-off-by: Tejas Vipin
>> ---
>> drive
Changes the elida-kd35t133 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-elida-kd35t133.c | 107 ---
1 file changed, 45 insertions(+), 62 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-elida
Changes the raydium-rm69380 panel to use multi style functions for
improved error handling.
Reviewed-by: Douglas Anderson
Signed-off-by: Tejas Vipin
---
Changes in v2:
- Fix whitespace issues
Link to v1:
https://lore.kernel.org/all/20240907140130.410349-1-tejasvipi...@gmail.com
On 9/12/24 12:49 PM, Thomas Zimmermann wrote:
> Hi
>
> Am 11.09.24 um 20:06 schrieb Tejas Vipin:
>> Replace drm_detect_hdmi_monitor() with drm_display_info.is_hdmi since
>> monitor HDMI information is available after EDID is parsed. Additionally
>> rewrite the
Replace drm_detect_hdmi_monitor() with drm_display_info.is_hdmi since
monitor HDMI information is available after EDID is parsed. Additionally
rewrite the code the code to have fewer indentation levels.
Signed-off-by: Tejas Vipin
---
Changes in v2:
- Use drm_edid instead of edid
Link to v1
Replace drm_detect_hdmi_monitor() with drm_display_info.is_hdmi since
monitor HDMI information is available after EDID is parsed. Additionally
rewrite the code the code to have fewer indentation levels.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 18
Changes the raydium-rm69380 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-raydium-rm69380.c | 95 ++-
1 file changed, 30 insertions(+), 65 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-raydium
On 9/7/24 3:53 AM, Jessica Zhang wrote:
>
>
> On 9/6/2024 3:14 PM, Jessica Zhang wrote:
>>
>>
>> On 9/4/2024 7:15 AM, Tejas Vipin wrote:
>>> Changes the himax-hx83112a panel to use multi style functions for
>>> improved error handling.
>>
Changes the himax-hx83112a panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-himax-hx83112a.c | 297 +--
1 file changed, 136 insertions(+), 161 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-himax
Changes the samsung-s6e3fa7 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-samsung-s6e3fa7.c | 71 ++-
1 file changed, 21 insertions(+), 50 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-samsung
Changes the novatek-nt35950 panel to use multi style functions for
improved error handling.
Reviewed-by: Neil Armstrong
Signed-off-by: Tejas Vipin
---
Changes in v2:
- Style changes
- Fixed changes in logic
v1: https://lore.kernel.org/all/20240824084422.202946-1-tejasvipi...@gmail.com
Changes the novatek-nt35950 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-novatek-nt35950.c | 214 ++
1 file changed, 70 insertions(+), 144 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-novatek
Converts uppercase hex to lowercase hex for cleanup.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-mantix-mlaf057we51.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-mantix-mlaf057we51.c
b/drivers/gpu/drm/panel/panel
Changes the mantix-mlaf057we51 panel to use multi style functions for
improved error handling.
Reviewed-by: Guido Günther
Reviewed-by: Douglas Anderson
Signed-off-by: Tejas Vipin
---
.../gpu/drm/panel/panel-mantix-mlaf057we51.c | 77 +++
1 file changed, 26 insertions(+), 51
Uses mipi_dsi_*_multi function in the mantix-mlaf057we51 panel and
converting uppercase hex to lowercase hex for cleanup.
---
Changes in v2:
- Split cleanup changes into separate patch
v1: https://lore.kernel.org/all/20240818072356.870465-1-tejasvipi...@gmail.com/
---
Tejas Vipin (2):
drm
Changes the mantix-mlaf057we51 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
.../gpu/drm/panel/panel-mantix-mlaf057we51.c | 79 +++
1 file changed, 27 insertions(+), 52 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel
Changes the jdi-fhd-r63452 panel to use multi style functions for
improved error handling.
Reviewed-by: Douglas Anderson
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-jdi-fhd-r63452.c | 149 ++-
1 file changed, 48 insertions(+), 101 deletions(-)
diff --git a
mipi_dsi_dcs_set_tear_scanline_multi can heavily benefit from being
converted to a multi style function as it is often called in the context of
similar functions.
Reviewed-by: Douglas Anderson
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 31
/
---
Tejas Vipin (2):
drm/mipi-dsi: Add mipi_dsi_dcs_set_tear_scanline_multi
drm/panel: jdi-fhd-r63452: transition to mipi_dsi wrapped functions
drivers/gpu/drm/drm_mipi_dsi.c | 31
drivers/gpu/drm/panel/panel-jdi-fhd-r63452.c | 149 ++-
include/drm
Changes the jdi-fhd-r63452 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-jdi-fhd-r63452.c | 150 ++-
1 file changed, 49 insertions(+), 101 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-jdi-fhd
mipi_dsi_dcs_set_tear_scanline_multi can heavily benefit from being
converted to a multi style function as it is often called in the context of
similar functions.
Reviewed-by: Douglas Anderson
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 31
...@gmail.com/
---
Changes in v2:
- Fixed return values
- Removed extra error messages
v1: https://lore.kernel.org/all/20240810045404.188146-1-tejasvipi...@gmail.com/
---
Tejas Vipin (2):
drm/mipi-dsi: Add mipi_dsi_dcs_set_tear_scanline_multi
drm/panel: jdi-fhd-r63452: transition to mipi_dsi
Changes the jdi-fhd-r63452 panel to use multi style functions for
improved error handling.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-jdi-fhd-r63452.c | 125 ++-
1 file changed, 39 insertions(+), 86 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-jdi-fhd
mipi_dsi_dcs_set_tear_scanline_multi can heavily benefit from being
converted to a multi style function as it is often called in the context of
similar functions.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 31 +++
include/drm/drm_mipi_dsi.h
This patch adds mipi_dsi_dcs_set_tear_scanline_multi to the list of multi
functions and uses it with other multi functions in the jdi-fhd-r63452
panel.
Tejas Vipin (2):
drm/mipi-dsi: Add mipi_dsi_dcs_set_tear_scanline_multi
drm/panel: jdi-fhd-r63452: transition to mipi_dsi wrapped functions
Use multi style wrapped functions for mipi_dsi in the
startek-kd070fhfid015 panel.
Signed-off-by: Tejas Vipin
---
.../drm/panel/panel-startek-kd070fhfid015.c | 115 ++
1 file changed, 35 insertions(+), 80 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-startek
...@gmail.com/
---
Tejas Vipin (2):
drm/mipi-dsi: add more multi functions for better error handling
drm/panel: startek-kd070fhfid015: transition to mipi_dsi wrapped
functions
drivers/gpu/drm/drm_mipi_dsi.c| 194 ++
.../drm/panel/panel-startek-kd070fhfid015.c
Add more functions that can benefit from being multi style and mark
older variants as deprecated to eventually convert all mipi_dsi functions
to multi style.
Acked-by: Maxime Ripard
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 194 +
include
On 8/1/24 4:39 PM, Jani Nikula wrote:
> On Tue, 30 Jul 2024, Tejas Vipin wrote:
>> Add more functions that can benefit from being multi style and mark
>> older variants as deprecated to eventually convert all mipi_dsi functions
>> to multi style.
>
> What?
>
&g
On 8/1/24 2:59 AM, Doug Anderson wrote:
> Hi,
>
> On Mon, Jul 29, 2024 at 11:07 PM Tejas Vipin wrote:
>> +/**
>> + * mipi_dsi_dcs_get_display_brightness_multi() - gets the current
>> brightness value
>> + *of the display
>> + * @ctx
Use multi style wrapped functions for mipi_dsi in the
startek-kd070fhfid015 panel.
Signed-off-by: Tejas Vipin
---
.../drm/panel/panel-startek-kd070fhfid015.c | 123 ++
1 file changed, 39 insertions(+), 84 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-startek
Add more functions that can benefit from being multi style and mark
older variants as deprecated to eventually convert all mipi_dsi functions
to multi style.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 226 +
include/drm/drm_mipi_dsi.h
transition
---
Tejas Vipin (2):
drm/mipi-dsi: add more multi functions for better error handling
drm/panel: startek-kd070fhfid015: transition to mipi_dsi wrapped
functions
drivers/gpu/drm/drm_mipi_dsi.c| 226 ++
.../drm/panel/panel-startek-kd070fhfid015.c
Changes all the multi functions to check if the current context requires
errors to be printed or not using the quiet member.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 20
1 file changed, 20 insertions(+)
diff --git a/drivers/gpu/drm/drm_mipi_dsi.c b
multi functions without any loss of
functionality once all the panels are changed to use multi functions.
Tejas Vipin (2):
drm/mipi-dsi: Add quiet member to mipi_dsi_multi_context struct
drm/mipi-dsi: Change multi functions to use quiet member of
mipi_dsi_multi_context
drivers/gpu/drm
A "quiet" member is added to mipi_dsi_multi_context which allows
silencing all the errors printed by the multi functions.
Signed-off-by: Tejas Vipin
---
include/drm/drm_mipi_dsi.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/include/drm/drm_mipi_dsi.h b/i
or message and just stringify the function name for the
> error message?
>
The problem with any macro solution that defines new multi functions is
that it creates a lot of bloat. Defining the function through macros
might be smaller than defining them manually, but there are still twice
as many function declarations as there would be if we went all multi.
The generated code is still just as big as if we manually defined
everything. I think a macro that defines functions should be more of a
last resort if we don't have any other options.
> -Doug
--
Tejas Vipin
On 7/16/24 10:31 PM, Dmitry Baryshkov wrote:
> On Tue, Jul 16, 2024 at 07:01:17PM GMT, Tejas Vipin wrote:
>> Introduce 2 new macros, DSI_CTX_NO_OP and MIPI_DSI_ADD_MULTI_VARIANT.
>>
>> DSI_CTX_NO_OP calls a function only if the context passed to it hasn't
>>
i_dsi function that follows a certain style. This allows us to
greatly reduce the amount of redundant code written for each multi
function. It reduces the overhead for a developer introducing new
mipi_dsi_*_multi functions.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c
On 7/9/24 5:50 AM, Doug Anderson wrote:
> Hi,
>
> On Fri, Jun 28, 2024 at 11:25 AM Tejas Vipin wrote:
>>
>> +/**
>> + * mipi_dsi_dcs_set_page_address_multi() - define the column extent of the
>> + * frame memory accessed by the host processor
>
Use multi style wrapped functions for mipi_dsi in the
startek-kd070fhfid015 panel.
Signed-off-by: Tejas Vipin
---
.../drm/panel/panel-startek-kd070fhfid015.c | 107 ++
1 file changed, 35 insertions(+), 72 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-startek
Add more functions that can benefit from being multi style, which
reduces code size in panels where they appear.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/drm_mipi_dsi.c | 164 +
include/drm/drm_mipi_dsi.h | 10 ++
2 files changed, 174 insertions
This series adds more multi style functions and uses them in the
startek-kd070fhfid015 panel effectively.
Tejas Vipin (2):
drm/mipi-dsi: add more multi functions for better error handling
drm/panel: startek-kd070fhfid015: transition to mipi_dsi wrapped
functions
drivers/gpu/drm
Sorry, didn't intend to send this. please ignore it :p
On 6/28/24 11:42 PM, Tejas Vipin wrote:
> Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi:
> Introduce mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
> ("drm/mipi-dsi: wrap more functions
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi:
Introduce mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
asus-z00t-tm5p5-n35596 panel.
Signed-off-by: Tejas Vipin
---
.../drm/panel/p
Hi,
On 6/26/24 5:55 PM, Guido Günther wrote:
> Hi,
> On Wed, Jun 26, 2024 at 10:22:41AM +0530, Tejas Vipin wrote:
>> Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi:
>> Introduce mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
>> (&quo
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi:
Introduce mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for
sitronix-st7703 based panels.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi:
Introduce mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
asus-z00t-tm5p5-n35596 panel.
Signed-off-by: Tejas Vipin
---
.../drm/panel/p
pare() is changed to
properly power the panel off in the case of a wider range of
initialization commands failing than before.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Douglas Anderson
Signed-off-by: Tejas Vipin
---
Changes in v3:
- Remove extra blank line
- Expanded on error handling fu
On 6/19/24 12:06 PM, Dmitry Baryshkov wrote:
> On Wed, Jun 19, 2024 at 09:03:49AM GMT, Tejas Vipin wrote:
>> Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi: Introduce
>> mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
>> ("drm/mipi-dsi
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi: Introduce
mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
raydium rm692e5 panel.
Signed-off-by: Tejas Vipin
---
Changes in v2:
-
On 6/18/24 7:06 PM, Doug Anderson wrote:
> Hi,
>
> On Tue, Jun 18, 2024 at 5:25 AM Tejas Vipin wrote:
>>
>>>> rm692e5_reset(ctx);
>>>>
>>>> - ret = rm692e5_on(ctx);
>>>> - if (ret < 0) {
>>
On 6/18/24 1:36 AM, Doug Anderson wrote:
> Hi,
>
> On Sat, Jun 15, 2024 at 2:40 AM Tejas Vipin wrote:
>>
>> @@ -168,48 +147,38 @@ static int rm692e5_prepare(struct drm_panel *panel)
>> struct rm692e5_panel *ctx = to_rm692e5_panel(panel);
>> str
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi: Introduce
mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
raydium rm692e5 panel.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi: Introduce
mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
sony tulip truly nt35521 panel.
Reviewed-by: Douglas Anderson
Signed
s ctx by reference (similar to
other functions), mipi_dsi_msleep can be modified to treat ctx as a
pointer and do it correctly, and the other calls to this macro can be
adjusted accordingly.
Tejas Vipin (2):
drm/panel : himax-hx83102: fix incorrect argument to mipi_dsi_msleep
drm/mipi-dsi
mipi_dsi_msleep should be modified to accept ctx as a pointer and the
function call should be adjusted accordingly.
Fixes: a2ab7cb169da3 ("drm/panel: himax-hx83102: use wrapped MIPI DCS
functions")
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-himax-hx83102.c | 6 +++-
ctx would be better off treated as a pointer to account for most of its
usage so far, and brackets should be added to account for operator
precedence for correct evaluation.
Fixes: f79d6d28d8fe7 ("drm/mipi-dsi: wrap more functions for streamline
handling")
Signed-off-by: T
mipi_dsi_msleep expects struct mipi_dsi_multi_context to be passed as a
value and not as a reference.
Fixes: a2ab7cb169da ("drm/panel: himax-hx83102: use wrapped MIPI DCS functions")
Signed-off-by: Tejas Vipin
---
Changes in v2:
- Add Fixes tag
v1: https://lore.kernel.org/al
Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi: Introduce
mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
sony tulip truly nt35521 panel.
Signed-off-by: Tejas Vipin
---
Changes in
Use functions introduced in 966e397e4f603 ("drm/mipi-dsi: Introduce
mipi_dsi_*_write_seq_multi()") and f79d6d28d8fe
("drm/mipi-dsi: wrap more functions for streamline handling") for the
sony tulip truly nt35521 panel.
Signed-off-by: Tejas Vipin
---
.../panel/panel-sony-t
From: Tejas Vipin
Subject: [PATCH] drm/panel : himax-hx83102: fix incorrect argument to
mipi_dsi_msleep
mipi_dsi_msleep expects struct mipi_dsi_multi_context to be passed as a
value and not as a reference.
Signed-off-by: Tejas Vipin
---
drivers/gpu/drm/panel/panel-himax-hx83102.c | 2 +-
1
65 matches
Mail list logo
