2024-05-16 13:33:23

by Doug Anderson

[permalink] [raw]
Subject: Re: [PATCH v8 0/6] Break out as separate driver and add BOE nv110wum-l60 IVO t109nw41 MIPI-DSI panel

Hi,

On Thu, May 16, 2024 at 12:21 AM Cong Yang
<[email protected]> wrote:
>
> Discussion with Doug and Linus in V1, we need a
> separate driver to enable the hx83102 controller.
>
> So this series this series mainly Break out as separate driver
> for Starry-himax83102-j02 panels from boe tv101wum driver.
>
> Then add BOE nv110wum-l60 and IVO t109nw41 in himax-hx83102 driver.
>
> Add compatible for BOE nv110wum-l60 and IVO t109nw41
> in dt-bindings
>
> Note:this series depend Dous'series [1]
> [1]: https://lore.kernel.org/all/[email protected]/
>
> Changes in v8:
> - Neil think need sent separately to ARM SoC maintainer with "arm64: defconfig: Enable HIMAX_HX83102 panel patch ", so remove it.
> - PATCH 1/6: No change.
> - PATCH 2/6: Fix Doug comment "return ret" change to "goto poweroff".
> - PATCH 3/6: No change.
> - PATCH 4/6: No change.
> - PATCH 5/6: No change.
> - PATCH 6/6: No change.
> - Link to v7:https://lore.kernel.org/all/20240515014643.2715010-1-yangcong5@huaqin.corp-partner.google.com/

This all looks good to me now.

Neil: do you want to apply this series plus Dmitry's [1] atop it, or
would you like me to? Dmitry's series has a fix in it but I don't
think it's critical enough to warrant the merge conflict that would
come with putting it through drm-misc-fixes.

[1] https://lore.kernel.org/r/[email protected]/


2024-05-17 07:23:21

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH v8 0/6] Break out as separate driver and add BOE nv110wum-l60 IVO t109nw41 MIPI-DSI panel

On 16/05/2024 15:31, Doug Anderson wrote:
> Hi,
>
> On Thu, May 16, 2024 at 12:21 AM Cong Yang
> <[email protected]> wrote:
>>
>> Discussion with Doug and Linus in V1, we need a
>> separate driver to enable the hx83102 controller.
>>
>> So this series this series mainly Break out as separate driver
>> for Starry-himax83102-j02 panels from boe tv101wum driver.
>>
>> Then add BOE nv110wum-l60 and IVO t109nw41 in himax-hx83102 driver.
>>
>> Add compatible for BOE nv110wum-l60 and IVO t109nw41
>> in dt-bindings
>>
>> Note:this series depend Dous'series [1]
>> [1]: https://lore.kernel.org/all/[email protected]/
>>
>> Changes in v8:
>> - Neil think need sent separately to ARM SoC maintainer with "arm64: defconfig: Enable HIMAX_HX83102 panel patch ", so remove it.
>> - PATCH 1/6: No change.
>> - PATCH 2/6: Fix Doug comment "return ret" change to "goto poweroff".
>> - PATCH 3/6: No change.
>> - PATCH 4/6: No change.
>> - PATCH 5/6: No change.
>> - PATCH 6/6: No change.
>> - Link to v7:https://lore.kernel.org/all/20240515014643.2715010-1-yangcong5@huaqin.corp-partner.google.com/
>
> This all looks good to me now.
>
> Neil: do you want to apply this series plus Dmitry's [1] atop it, or
> would you like me to? Dmitry's series has a fix in it but I don't
> think it's critical enough to warrant the merge conflict that would
> come with putting it through drm-misc-fixes.


Yes, it's not critical enough, I'll apply this one now and switch to Dmitry's one.

Thanks,
Neil

>
> [1] https://lore.kernel.org/r/[email protected]/