Il dom 15 feb 2026, 10:51 BALATON Zoltan <[email protected]> ha scritto:
> >>> +void ppc_graphic_dimensions(int *width, int *height, int *depth)
> >>> +{
> >>> + *width = graphic_width ?: 800;
> >>> + *height = graphic_height ?: 600;
> >>> + *depth = graphic_depth ?: 32;
> >>> +}
> >>
> >> I think there's no reason to have this common to all ppc machines.
I agree with you here that it's better to just inline this in all the
boards that need it, because there's no reason to have 800x600x32 across
such disparate machines.
> Can you provide a patch showing how you'd prefer it to be done?
>
> Maybe turning these into properties could work. I could look at adding
> such properties but don't know how to change the -g option. But
> considering these are all machines that Mark maintains maybe he should do
> something about it
I am not sure what you'd be requesting of him?
The easiest were if he agreed to drop -g option but he
> never agrees removing anything
Not sure who's he? Is it me since I asked Philippe to keep -g, or Mark? And
if it's Mark, I am not sure what you're trying to obtain by involving (and
in a pretty rude way, too) someone who's not even part of the thread.
so then he should provide patches to clean
> this up the way he prefers. I could try to make patches but I'm afraid
> those would become wasted effort so may I only do it when there's an
> approved way that will be accepted.
>
The -g 800x600x32 option, if desired, can become simply a shortcut for -M
gfxwidth=800,gfxheight=600,gfxdepth=32 (or
gfx.width=800,gfx.height=600,gfx.depth=32), which is more similar to how
smp or boot work). This would actually be easier to implement with
Philippe's move of all graphics_* accesses to board code, so his patches
should go in either way. And then the globals can be removed altogether.
Paolo
> Regards,
> BALATON Zoltan
