<div dir="ltr"><div dir="ltr">Thanks for your help</div><div dir="ltr"><br><div><div>Both mailbox accounts are mine.</div><div>I don't know what to do next.</div><div>Do I need to resubmit the patch[V7]?</div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Thierry Reding <<a href="mailto:thierry.reding@gmail.com">thierry.reding@gmail.com</a>> 于2019年4月23日周二 下午8:08写道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Mar 26, 2019 at 03:09:33PM +0800, Jerry Han wrote:<br>
> Support Boe Himax8279d 8.0" 1200x1920 TFT LCD panel, it is a MIPI DSI<br>
> panel.<br>
> <br>
> V7:<br>
> - Add the information of the reviewer<br>
> - Remove unnecessary delays, The udelay_range code gracefully returns<br>
>     without hitting the scheduler on a delay of 0. (Derek)<br>
> - Merge the same data structures, like display_mode and off_cmds (Derek)<br>
> - Optimize the processing of results returned by<br>
>     devm_gpiod_get_optional (Derek)<br>
> <br>
> V6:<br>
> - Add the information of the reviewer (Sam)<br>
> - Delete unnecessary header files #include <linux/fb.h> (Sam)<br>
> - The config DRM_PANEL_BOE_HIMAX8279D appears twice. Drop one of them (Sam)<br>
> - ADD static, set_gpios function is not used outside this module (Sam)<br>
> <br>
> V5:<br>
> - Added changelog<br>
> <br>
> V4:<br>
> - Frefix all function maes with boe_ (Sam)<br>
> - Fsed "enable_gpio" replace "reset_gpio", Make it look clearer (Sam)<br>
> - Sort include lines alphabetically (Sam)<br>
> - Fixed entries in the makefile must be sorted alphabetically (Sam)<br>
> - Add send_mipi_cmds function to avoid duplicating the code (Sam)<br>
> - Add the necessary delay(reset_delay_t5) between reset and sending<br>
>     the initialization command (Rock wang)<br>
> <br>
> V3:<br>
> - Remove unnecessary delays in sending initialization commands (Jitao Shi)<br>
> <br>
> V2:<br>
> - Use SPDX identifier (Sam)<br>
> - Use necessary header files replace drmP.h (Sam)<br>
> - Delete unnecessary header files #include <linux/err.h> (Sam)<br>
> - Specifies a GPIOs array to control the reset timing,<br>
>     instead of reading "dsi-reset-sequence" data from DTS (Sam)<br>
> - Delete backlight_disable() function when already disabled (Sam)<br>
> - Use devm_of_find_backlight() replace of_find_backlight_by_node() (Sam)<br>
> - Move the necessary data in the DTS to the current file,<br>
>     like porch, display_mode and Init code etc. (Sam)<br>
> - Add compatible device "boe,himax8279d10p" (Sam)<br>
> <br>
> Signed-off-by: Jerry Han <<a href="mailto:hanxu5@huaqin.corp-partner.google.com" target="_blank">hanxu5@huaqin.corp-partner.google.com</a>><br>
<br>
The sign-off identity doesn't match the authorship information in the<br>
patch. You may want to fix your git configuration.<br>
<br>
Thierry<br>
</blockquote></div>