U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Simon Glass <sjg@chromium.org>
Cc: U-Boot Mailing List <u-boot@lists.denx.de>,
	Alexander Graf <agraf@csgraf.de>,
	Alexander Sverdlin <alexander.sverdlin@siemens.com>,
	Alper Nebi Yasak <alpernebiyasak@gmail.com>,
	Anatolij Gustschin <agust@denx.de>, Bin Meng <bmeng.cn@gmail.com>,
	Caleb Connolly <caleb.connolly@linaro.org>,
	Christian Marangi <ansuelsmth@gmail.com>,
	Devarsh Thakkar <devarsht@ti.com>,
	Guillaume La Roque <glaroque@baylibre.com>,
	Kever Yang <kever.yang@rock-chips.com>,
	Mattijs Korpershoek <mkorpershoek@baylibre.com>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Nikhil M Jain <n-jain1@ti.com>,
	Patrick Rudolph <patrick.rudolph@9elements.com>,
	Peter Robinson <pbrobinson@gmail.com>,
	Sughosh Ganu <sughosh.ganu@linaro.org>
Subject: Re: [PATCH 00/13] video: Enhancements related to truetype and console
Date: Wed, 2 Apr 2025 18:27:54 -0600	[thread overview]
Message-ID: <20250403002754.GH5495@bill-the-cat> (raw)
In-Reply-To: <CAFLszThRQi2o5g5S+=eciOS8+XC8TV9AyBpYf9LQKmH-2kkmvw@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2053 bytes --]

On Thu, Apr 03, 2025 at 12:54:55PM +1300, Simon Glass wrote:
> Hi Tom,
> 
> On Thu, 3 Apr 2025 at 10:11, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Thu, Apr 03, 2025 at 08:19:03AM +1300, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Thu, 3 Apr 2025 at 04:00, Tom Rini <trini@konsulko.com> wrote:
> > > >
> > > > On Wed, Apr 02, 2025 at 01:08:56PM +1300, Simon Glass wrote:
> > > > > Hi Tom,
> > > > >
> > > > > On Wed, 2 Apr 2025 at 10:41, Tom Rini <trini@konsulko.com> wrote:
> > > > > >
> > > > > > On Wed, Apr 02, 2025 at 06:29:31AM +1300, Simon Glass wrote:
> > > > > >
> > > > > > > This series includes some precursor patches needed for forthcoming expo
> > > > > > > enhancements.
> > > > > > >
> > > > > > > - truetype support for multiple lines
> > > > > > > - make white-on-black a runtime option
> > > > > > > - support drawing a rectangle
> > > > > > >
> > > > > > > This series was split out from schd2
> > > > > > [snip]
> > > > > > > base-commit: 3f76d803db9b500f43bc534465945a8d2836bb3e
> > > > > > > branch: schg
> > > > > >
> > > > > > I get as far as patch 4 before it doesn't apply to -next.
> > > > >
> > > > > Your tree is missing the video-damage series which adds some more
> > > > > tests. It is here if you want to pick it up:
> > > > >
> > > > > https://patchwork.ozlabs.org/project/uboot/list/?series=369663&state=*
> > > > >
> > > > > I can rebase and send a PR if you prefer.
> > > >
> > > > Reading the cover letter again, that doesn't seem ready to merge either.
> > >
> > > It seems OK to me. What are your concerns?
> >
> > The cover letter read to me like it should be further reworked to
> > address some of the problems it still has / exposed. That's likely why I
> > marked it as Changes Requested a year and a half ago.
> 
> Which bit of it? Are you talking about my conversation with Alex, or
> the actual cover letter?

Yes, I'm talking about the lengthy discussion in the cover letter which
sounds to me as a summary of "changes requested".

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  reply	other threads:[~2025-04-03  0:28 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-01 17:29 [PATCH 00/13] video: Enhancements related to truetype and console Simon Glass
2025-04-01 17:29 ` [PATCH 01/13] console: Support a format string for stderr output Simon Glass
2025-04-01 17:29 ` [PATCH 02/13] video: Make white-on-black a video-device property Simon Glass
2025-04-01 17:29 ` [PATCH 03/13] sandbox: Select white-on-black Simon Glass
2025-04-01 17:29 ` [PATCH 04/13] video: Add a test for font measurement Simon Glass
2025-04-01 17:29 ` [PATCH 05/13] video: Begin support for measuring multiple lines of text Simon Glass
2025-04-01 17:29 ` [PATCH 06/13] video: truetype: Fill in the measured line Simon Glass
2025-04-01 17:29 ` [PATCH 07/13] video: truetype: Support newlines in the measured string Simon Glass
2025-04-01 17:29 ` [PATCH 08/13] video: truetype: Support a limit on the width of a line Simon Glass
2025-04-01 17:29 ` [PATCH 09/13] video: Add a way to write a partial string to the console Simon Glass
2025-04-01 17:29 ` [PATCH 10/13] test: video: Export the video-checking functions Simon Glass
2025-04-01 17:29 ` [PATCH 11/13] video: Allow console output to be silenced Simon Glass
2025-04-01 17:29 ` [PATCH 12/13] video: Add a function to draw a rectangle Simon Glass
2025-04-01 17:29 ` [PATCH 13/13] vidconsole: Avoid kerning against an unrelated character Simon Glass
2025-04-01 21:41 ` [PATCH 00/13] video: Enhancements related to truetype and console Tom Rini
2025-04-02  0:08   ` Simon Glass
2025-04-02 15:00     ` Tom Rini
2025-04-02 19:19       ` Simon Glass
2025-04-02 21:11         ` Tom Rini
2025-04-02 23:54           ` Simon Glass
2025-04-03  0:27             ` Tom Rini [this message]
2025-04-03 17:57               ` Simon Glass
2025-04-12 15:51                 ` Tom Rini
2025-05-02 19:59 ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250403002754.GH5495@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=agraf@csgraf.de \
    --cc=agust@denx.de \
    --cc=alexander.sverdlin@siemens.com \
    --cc=alpernebiyasak@gmail.com \
    --cc=ansuelsmth@gmail.com \
    --cc=bmeng.cn@gmail.com \
    --cc=caleb.connolly@linaro.org \
    --cc=devarsht@ti.com \
    --cc=glaroque@baylibre.com \
    --cc=kever.yang@rock-chips.com \
    --cc=miquel.raynal@bootlin.com \
    --cc=mkorpershoek@baylibre.com \
    --cc=n-jain1@ti.com \
    --cc=patrick.rudolph@9elements.com \
    --cc=pbrobinson@gmail.com \
    --cc=sjg@chromium.org \
    --cc=sughosh.ganu@linaro.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox