Re: [PATCH v4 1/6] serial: Add Goldfish TTY driver

[Kuan-Wei Chiu <visitorckw@gmail.com>]

Hi Tom,

On Mon, Jan 05, 2026 at 08:43:04AM -0600, Tom Rini wrote:
> On Sat, Jan 03, 2026 at 07:14:30AM +0800, Kuan-Wei Chiu wrote:
> 
> > Hi Tom,
> > 
> > On Fri, Jan 02, 2026 at 03:51:25PM -0600, Tom Rini wrote:
> > > On Sat, Jan 03, 2026 at 05:48:14AM +0800, Kuan-Wei Chiu wrote:
> > > > Hi Heinrich,
> > > > 
> > > > On Fri, Jan 02, 2026 at 09:40:39AM +0100, Heinrich Schuchardt wrote:
> > > > > On 1/2/26 09:29, Heinrich Schuchardt wrote:
> > > > > > On 1/1/26 18:54, Kuan-Wei Chiu wrote:
> > > > > > > Add support for the Google Goldfish TTY serial device. This virtual
> > > > > > > device is commonly used in QEMU virtual machines (such as the m68k
> > > > > > > virt machine) and Android emulators.
> > > > > > > 
> > > > > > > The driver implements basic console output and input polling using the
> > > > > > > Goldfish MMIO interface.
> > > > > > > 
> > > > > > > Signed-off-by: Kuan-Wei Chiu <visitorckw@gmail.com>
> > > > > > > Reviewed-by: Yao Zi <me@ziyao.cc>
> > > > > > > Tested-by: Daniel Palmer <daniel@0x0f.com>
> > > > > > 
> > > > > > Acked-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> > > > 
> > > > Thanks for the review.
> > > > 
> > > > > > 
> > > > > > > ---
> > > > > > > Changes in v4:
> > > > > > > - Support DT probing with platform data as a fallback.
> > > > > > > 
> > > > > > >   MAINTAINERS                      |   6 ++
> > > > > > >   drivers/serial/Kconfig           |   8 +++
> > > > > > >   drivers/serial/Makefile          |   1 +
> > > > > > >   drivers/serial/serial_goldfish.c | 109 +++++++++++++++++++++++++++++++
> > > > > > >   include/goldfish_tty.h           |  18 +++++
> > > > > > >   5 files changed, 142 insertions(+)
> > > > > > >   create mode 100644 drivers/serial/serial_goldfish.c
> > > > > > >   create mode 100644 include/goldfish_tty.h
> > > > > > > 
> > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > > > > index 6ce0bbce13d..da4a6e4d518 100644
> > > > > > > --- a/MAINTAINERS
> > > > > > > +++ b/MAINTAINERS
> > > > > > > @@ -1259,6 +1259,12 @@ S:    Maintained
> > > > > > >   F:    drivers/misc/gsc.c
> > > > > > >   F:    include/gsc.h
> > > > > > > +GOLDFISH SERIAL DRIVER
> > > > > > > +M:    Kuan-Wei Chiu <visitorckw@gmail.com>
> > > > > > > +S:    Maintained
> > > > > > > +F:    drivers/serial/serial_goldfish.c
> > > > > > > +F:    include/goldfish_tty.h
> > > > > > > +
> > > > > 
> > > > > Applying: serial: Add Goldfish TTY driver
> > > > > Patch failed at 0001 serial: Add Goldfish TTY driver
> > > > > error: patch failed: MAINTAINERS:1259
> > > > > error: MAINTAINERS: patch does not apply
> > > > > 
> > > > > Should a respin of the series be needed, please, check that the series
> > > > > applies to origin/next before sending.
> > > > 
> > > > To make sure I base the next version on the correct tree, should I base
> > > > it on the ColdFire tree's next branch:
> > > > 
> > > > https://source.denx.de/u-boot/custodians/u-boot-coldfire.git
> > > > 
> > > > Or the main U-Boot repository's next branch:
> > > > 
> > > > https://source.denx.de/u-boot/u-boot.git
> > > 
> > > The main U-Boot tree. Custodian trees are not always up to date (and
> > 
> > Thanks for the clarification.
> > I will base v5 on the main U-Boot tree's next branch.
> > 
> > > with the renewed interest in m68k, I was going to ask if someone else
> > > would like to step up as co-maintainer as part of all of this, Angelo
> > > might like some help).
> > 
> > TBH, being relatively new to the U-Boot community, I'm not sure if I
> > have gained enough trust or shown enough knowledge of the m68k code and
> > development workflow just yet.
> > 
> > However, if help is needed, I'm happy to help review and test
> > m68k-related patches.
> 
> I understand, and yes, starting with reviewing things is fine. But this
> is also the most activity m68k has gotten in a decade or so, so I do
> believe you to be on the forefront of how the code is :)

Thanks for the kind words.

In that case, I'll append a patch to the end of the v5 series to add
myself to the m68k entry in MAINTAINERS (after Angelo), so I will be
notified of m68k patches to help with review and testing.

Regards,
Kuan-Wei