All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "Thomas Zimmermann" <tzimmermann@suse.de>,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	"Michal Suchánek" <msuchanek@suse.de>
Cc: Linux Fbdev development list <linux-fbdev@vger.kernel.org>,
	David Airlie <airlied@linux.ie>,
	Michael Ellerman <mpe@ellerman.id.au>,
	Helge Deller <deller@gmx.de>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	Javier Martinez Canillas <javierm@redhat.com>,
	DRI Development <dri-devel@lists.freedesktop.org>,
	Paul Mackerras <paulus@samba.org>,
	Maxime Ripard <maxime@cerno.tech>,
	Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [PATCH 2/2] drm/tiny: Add ofdrm for Open Firmware framebuffers
Date: Sat, 21 May 2022 12:49:34 +1000	[thread overview]
Message-ID: <615c93392bee43e92f0400cfa51957cd955091d3.camel@kernel.crashing.org> (raw)
In-Reply-To: <29a8201d-3c0c-eeed-81af-92b351880702@suse.de>

On Thu, 2022-05-19 at 09:27 +0200, Thomas Zimmermann wrote:

> to build without PCI to see what happens.

If you bring any of the "heuristic" and palette support code in, you
need PCI. I don't see any reason to take it out.

> Those old Macs use BootX, right? BootX is not supported ATM, as I don't 
> have the HW to test. Is there an emulator for it?

It isn't ? When did it break ? :-)

> If anyone what's to make patches for BootX, I'd be happy to add them. 
> The offb driver also supports a number of special cases for palette 
> handling. That might be necessary for ofdrm as well.

The palette handling is useful when using a real Open Firmware
implementation which tends to boot in 8-bit mode, so without palette
things will look ... bad.

It's not necessary when using 16/32 bpp framebuffers which is typically
... what BootX provides :-)

Cheers,
Ben.

> Best regards
> Thomas
> 
> > Gr{oetje,eeting}s,
> > 
> >                          Geert
> > 
> > --
> > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> > 
> > In personal conversations with technical people, I call myself a hacker. But
> > when I'm talking to journalists I just say "programmer" or something like that.
> >                                  -- Linus Torvalds
> 
> -- 
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Maxfeldstr. 5, 90409 Nürnberg, Germany
> (HRB 36809, AG Nürnberg)
> Geschäftsführer: Ivo Totev


WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "Thomas Zimmermann" <tzimmermann@suse.de>,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	"Michal Suchánek" <msuchanek@suse.de>
Cc: Linux Fbdev development list <linux-fbdev@vger.kernel.org>,
	David Airlie <airlied@linux.ie>, Helge Deller <deller@gmx.de>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	Javier Martinez Canillas <javierm@redhat.com>,
	DRI Development <dri-devel@lists.freedesktop.org>,
	Paul Mackerras <paulus@samba.org>,
	Maxime Ripard <maxime@cerno.tech>,
	Daniel Vetter <daniel@ffwll.ch>, Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [PATCH 2/2] drm/tiny: Add ofdrm for Open Firmware framebuffers
Date: Sat, 21 May 2022 12:49:34 +1000	[thread overview]
Message-ID: <615c93392bee43e92f0400cfa51957cd955091d3.camel@kernel.crashing.org> (raw)
In-Reply-To: <29a8201d-3c0c-eeed-81af-92b351880702@suse.de>

On Thu, 2022-05-19 at 09:27 +0200, Thomas Zimmermann wrote:

> to build without PCI to see what happens.

If you bring any of the "heuristic" and palette support code in, you
need PCI. I don't see any reason to take it out.

> Those old Macs use BootX, right? BootX is not supported ATM, as I don't 
> have the HW to test. Is there an emulator for it?

It isn't ? When did it break ? :-)

> If anyone what's to make patches for BootX, I'd be happy to add them. 
> The offb driver also supports a number of special cases for palette 
> handling. That might be necessary for ofdrm as well.

The palette handling is useful when using a real Open Firmware
implementation which tends to boot in 8-bit mode, so without palette
things will look ... bad.

It's not necessary when using 16/32 bpp framebuffers which is typically
... what BootX provides :-)

Cheers,
Ben.

> Best regards
> Thomas
> 
> > Gr{oetje,eeting}s,
> > 
> >                          Geert
> > 
> > --
> > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> > 
> > In personal conversations with technical people, I call myself a hacker. But
> > when I'm talking to journalists I just say "programmer" or something like that.
> >                                  -- Linus Torvalds
> 
> -- 
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Maxfeldstr. 5, 90409 Nürnberg, Germany
> (HRB 36809, AG Nürnberg)
> Geschäftsführer: Ivo Totev


WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "Thomas Zimmermann" <tzimmermann@suse.de>,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	"Michal Suchánek" <msuchanek@suse.de>
Cc: Javier Martinez Canillas <javierm@redhat.com>,
	David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
	Helge Deller <deller@gmx.de>, Maxime Ripard <maxime@cerno.tech>,
	Sam Ravnborg <sam@ravnborg.org>,
	Michael Ellerman <mpe@ellerman.id.au>,
	Paul Mackerras <paulus@samba.org>,
	DRI Development <dri-devel@lists.freedesktop.org>,
	Linux Fbdev development list <linux-fbdev@vger.kernel.org>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>
Subject: Re: [PATCH 2/2] drm/tiny: Add ofdrm for Open Firmware framebuffers
Date: Sat, 21 May 2022 12:49:34 +1000	[thread overview]
Message-ID: <615c93392bee43e92f0400cfa51957cd955091d3.camel@kernel.crashing.org> (raw)
In-Reply-To: <29a8201d-3c0c-eeed-81af-92b351880702@suse.de>

On Thu, 2022-05-19 at 09:27 +0200, Thomas Zimmermann wrote:

> to build without PCI to see what happens.

If you bring any of the "heuristic" and palette support code in, you
need PCI. I don't see any reason to take it out.

> Those old Macs use BootX, right? BootX is not supported ATM, as I don't 
> have the HW to test. Is there an emulator for it?

It isn't ? When did it break ? :-)

> If anyone what's to make patches for BootX, I'd be happy to add them. 
> The offb driver also supports a number of special cases for palette 
> handling. That might be necessary for ofdrm as well.

The palette handling is useful when using a real Open Firmware
implementation which tends to boot in 8-bit mode, so without palette
things will look ... bad.

It's not necessary when using 16/32 bpp framebuffers which is typically
... what BootX provides :-)

Cheers,
Ben.

> Best regards
> Thomas
> 
> > Gr{oetje,eeting}s,
> > 
> >                          Geert
> > 
> > --
> > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> > 
> > In personal conversations with technical people, I call myself a hacker. But
> > when I'm talking to journalists I just say "programmer" or something like that.
> >                                  -- Linus Torvalds
> 
> -- 
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Maxfeldstr. 5, 90409 Nürnberg, Germany
> (HRB 36809, AG Nürnberg)
> Geschäftsführer: Ivo Totev


  reply	other threads:[~2022-05-21  2:53 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-18 18:30 [PATCH 0/2] drm: Add driverof PowerPC OF displays Thomas Zimmermann
2022-05-18 18:30 ` Thomas Zimmermann
2022-05-18 18:30 ` [PATCH 1/2] MAINTAINERS: Broaden scope of simpledrm entry Thomas Zimmermann
2022-05-18 18:30   ` Thomas Zimmermann
2022-05-20  5:18   ` Javier Martinez Canillas
2022-05-20  5:18     ` Javier Martinez Canillas
2022-05-18 18:30 ` [PATCH 2/2] drm/tiny: Add ofdrm for Open Firmware framebuffers Thomas Zimmermann
2022-05-18 18:30   ` Thomas Zimmermann
2022-05-18 18:51   ` Michal Suchánek
2022-05-18 18:51     ` Michal Suchánek
2022-05-18 18:51     ` Michal Suchánek
2022-05-19  7:11     ` Geert Uytterhoeven
2022-05-19  7:11       ` Geert Uytterhoeven
2022-05-19  7:11       ` Geert Uytterhoeven
2022-05-19  7:27       ` Thomas Zimmermann
2022-05-19  7:27         ` Thomas Zimmermann
2022-05-19  7:27         ` Thomas Zimmermann
2022-05-21  2:49         ` Benjamin Herrenschmidt [this message]
2022-05-21  2:49           ` Benjamin Herrenschmidt
2022-05-21  2:49           ` Benjamin Herrenschmidt
2022-05-25 16:45           ` Thomas Zimmermann
2022-05-25 16:45             ` Thomas Zimmermann
2022-05-25 16:45             ` Thomas Zimmermann
2022-05-27  0:17             ` Benjamin Herrenschmidt
2022-05-27  0:17               ` Benjamin Herrenschmidt
2022-05-27  0:17               ` Benjamin Herrenschmidt
2022-05-27  0:19             ` Benjamin Herrenschmidt
2022-05-27  0:19               ` Benjamin Herrenschmidt
2022-05-27  0:19               ` Benjamin Herrenschmidt
2022-05-21  1:35       ` Benjamin Herrenschmidt
2022-05-21  1:35         ` Benjamin Herrenschmidt
2022-05-21  1:35         ` Benjamin Herrenschmidt
2022-05-18 21:11   ` Mark Cave-Ayland
2022-05-19  6:34     ` Michal Suchánek
2022-05-19  6:34       ` Michal Suchánek
2022-05-19  6:34       ` Michal Suchánek
2022-05-19  7:39     ` Thomas Zimmermann
2022-05-20  6:19   ` Javier Martinez Canillas
2022-05-20  6:19     ` Javier Martinez Canillas
2022-05-22 19:35     ` Thomas Zimmermann
2022-05-22 19:35       ` Thomas Zimmermann
2022-05-25  3:15       ` Benjamin Herrenschmidt
2022-05-25  3:15         ` Benjamin Herrenschmidt

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=615c93392bee43e92f0400cfa51957cd955091d3.camel@kernel.crashing.org \
    --to=benh@kernel.crashing.org \
    --cc=airlied@linux.ie \
    --cc=deller@gmx.de \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=geert@linux-m68k.org \
    --cc=javierm@redhat.com \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=maxime@cerno.tech \
    --cc=mpe@ellerman.id.au \
    --cc=msuchanek@suse.de \
    --cc=paulus@samba.org \
    --cc=sam@ravnborg.org \
    --cc=tzimmermann@suse.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.