Hi,
On Mon, Jun 26, 2023 at 10:01 PM Cong Yang
<[email protected]> wrote:
>
> Because the setting of hproch is too small, there will be warning in
> kernel log[1]. After fine tune the HFP and HBP, this warning can be
> solved. The actual measurement frame rate is 60.1Hz.
>
> [1]: WARNING kernel:[drm] HFP + HBP less than d-phy, FPS will under 60Hz
Thanks for including the warming. This looks like something that's
only printed on Mediatek display controllers. Just out of curiosity:
is that because Mediatek controllers have special requirements, or is
this something generic and the Mediatek controller is the only one
that warns about it?
> Fixes: 8716a6473e6c ("drm/panel: Support for Starry-ili9882t TDDI
> MIPI-DSI panel")
Ideally the tool you have to send mail wouldn't wrap the Fixes line.
Probably not worth resending just for this, but see if there's
something you can do to fix this in the future.
Since this is a tiny fix, I don't think we need to wait too long. I'll
plan to land it on Thursday unless Neil wants to land it himself
sooner.
Reviewed-by: Douglas Anderson <[email protected]>
Hi, Doug
On Wed, Jun 28, 2023 at 5:17 AM Doug Anderson <[email protected]> wrote:
>
> Hi,
>
> On Mon, Jun 26, 2023 at 10:01 PM Cong Yang
> <[email protected]> wrote:
> >
> > Because the setting of hproch is too small, there will be warning in
> > kernel log[1]. After fine tune the HFP and HBP, this warning can be
> > solved. The actual measurement frame rate is 60.1Hz.
> >
> > [1]: WARNING kernel:[drm] HFP + HBP less than d-phy, FPS will under 60Hz
>
> Thanks for including the warming. This looks like something that's
> only printed on Mediatek display controllers. Just out of curiosity:
> is that because Mediatek controllers have special requirements, or is
> this something generic and the Mediatek controller is the only one
> that warns about it?
It seems to be a generic calculation formula for MTK controllers.
At least I have see this warning in both MT8186 and MT8188.
By the way, the same warning will also appear under coreboot.
>
>
> > Fixes: 8716a6473e6c ("drm/panel: Support for Starry-ili9882t TDDI
> > MIPI-DSI panel")
>
> Ideally the tool you have to send mail wouldn't wrap the Fixes line.
> Probably not worth resending just for this, but see if there's
> something you can do to fix this in the future.
>
> Since this is a tiny fix, I don't think we need to wait too long. I'll
> plan to land it on Thursday unless Neil wants to land it himself
> sooner.
Thank you, Doug.
>
> Reviewed-by: Douglas Anderson <[email protected]>
Hi,
On Tue, Jun 27, 2023 at 2:17 PM Doug Anderson <[email protected]> wrote:
>
> Hi,
>
> On Mon, Jun 26, 2023 at 10:01 PM Cong Yang
> <[email protected]> wrote:
> >
> > Because the setting of hproch is too small, there will be warning in
I realized that hproch should be hporch. I fixed when applying.
> > kernel log[1]. After fine tune the HFP and HBP, this warning can be
> > solved. The actual measurement frame rate is 60.1Hz.
> >
> > [1]: WARNING kernel:[drm] HFP + HBP less than d-phy, FPS will under 60Hz
>
> Thanks for including the warming. This looks like something that's
> only printed on Mediatek display controllers. Just out of curiosity:
> is that because Mediatek controllers have special requirements, or is
> this something generic and the Mediatek controller is the only one
> that warns about it?
>
>
> > Fixes: 8716a6473e6c ("drm/panel: Support for Starry-ili9882t TDDI
> > MIPI-DSI panel")
>
> Ideally the tool you have to send mail wouldn't wrap the Fixes line.
> Probably not worth resending just for this, but see if there's
> something you can do to fix this in the future.
>
> Since this is a tiny fix, I don't think we need to wait too long. I'll
> plan to land it on Thursday unless Neil wants to land it himself
> sooner.
>
> Reviewed-by: Douglas Anderson <[email protected]>
OK, landed. This is a weird time to land since this is a fix and the
fix has already made its way to Linus's tree but -rc1 isn't ready yet.
From reading the flowchart in the committer guidelines:
https://drm.pages.freedesktop.org/maintainer-tools/committer-drm-misc.html
...this means that it lands in drm-misc-next-fixes, so I've pushed it there.
59bba51ec2a5 drm/panel: Fine tune Starry-ili9882t panel HFP and HBP
-Doug
On Fri, Jun 30, 2023 at 2:42 AM Doug Anderson <[email protected]> wrote:
> ...this means that it lands in drm-misc-next-fixes, so I've pushed it there.
>
> 59bba51ec2a5 drm/panel: Fine tune Starry-ili9882t panel HFP and HBP
I guess that means we need to merge drm-misc-next-fixes back to the
drm-misc-next branch so that I can rebase my series breaking out the
ili9882t driver to its own file?
Yours,
Linus Walleij
Hi,
On Tue, Jul 4, 2023 at 12:39 AM Linus Walleij <[email protected]> wrote:
>
> On Fri, Jun 30, 2023 at 2:42 AM Doug Anderson <[email protected]> wrote:
>
> > ...this means that it lands in drm-misc-next-fixes, so I've pushed it there.
> >
> > 59bba51ec2a5 drm/panel: Fine tune Starry-ili9882t panel HFP and HBP
>
> I guess that means we need to merge drm-misc-next-fixes back to the
> drm-misc-next branch so that I can rebase my series breaking out the
> ili9882t driver to its own file?
It looks like this fix is on its way to the main DRM tree:
https://lore.kernel.org/r/20230706112203.GA30555@linux-uq9g
Presumably if we wait a few days things will sort themselves out. If
something needs to happen sooner then we'll have to get the drm-misc
maintainers involved. Probably at this point it makes sense to wait?
-Doug
On Thu, Jul 6, 2023 at 9:26 PM Doug Anderson <[email protected]> wrote:
> Presumably if we wait a few days things will sort themselves out. If
> something needs to happen sooner then we'll have to get the drm-misc
> maintainers involved. Probably at this point it makes sense to wait?
Yeah there is certainly no hurry :D
Yours,
Linus Walleij