Re: [PATCH] hw/display/sm501: Implement more 2D raster operations

[Daniel Henrique Barboza]

On 2/16/23 11:40, BALATON Zoltan wrote:
> Add simple implementation for two raster operations that are used by
> AmigaOS which fixes graphics problems in some progtams using these.
>
> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> ---

Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>


And queued in ppc-next. Thanks,


Daniel

> For definitions of these see:
> https://learn.microsoft.com/en-us/windows/win32/gdi/ternary-raster-operations
>
>   hw/display/sm501.c | 30 +++++++++++++++++++++++++++++-
>   1 file changed, 29 insertions(+), 1 deletion(-)
>
> diff --git a/hw/display/sm501.c b/hw/display/sm501.c
> index e1d0591d36..58bc9701ee 100644
> --- a/hw/display/sm501.c
> +++ b/hw/display/sm501.c
> @@ -753,7 +753,7 @@ static void sm501_2d_operation(SM501State *s)
>           }
>   
>           if ((rop_mode && rop == 0x5) || (!rop_mode && rop == 0x55)) {
> -            /* Invert dest, is there a way to do this with pixman? */
> +            /* DSTINVERT, is there a way to do this with pixman? */
>               unsigned int x, y, i;
>               uint8_t *d = s->local_mem + dst_base;
>   
> @@ -763,6 +763,34 @@ static void sm501_2d_operation(SM501State *s)
>                       stn_he_p(&d[i], bypp, ~ldn_he_p(&d[i], bypp));
>                   }
>               }
> +        } else if (!rop_mode && rop == 0x99) {
> +            /* DSxn, is there a way to do this with pixman? */
> +            unsigned int x, y, i, j;
> +            uint8_t *sp = s->local_mem + src_base;
> +            uint8_t *d = s->local_mem + dst_base;
> +
> +            for (y = 0; y < height; y++) {
> +                i = (dst_x + (dst_y + y) * dst_pitch) * bypp;
> +                j = (src_x + (src_y + y) * src_pitch) * bypp;
> +                for (x = 0; x < width; x++, i += bypp, j += bypp) {
> +                    stn_he_p(&d[i], bypp,
> +                             ~(ldn_he_p(&sp[j], bypp) ^ ldn_he_p(&d[i], 
> bypp)));
> +                }
> +            }
> +        } else if (!rop_mode && rop == 0xee) {
> +            /* SRCPAINT, is there a way to do this with pixman? */
> +            unsigned int x, y, i, j;
> +            uint8_t *sp = s->local_mem + src_base;
> +            uint8_t *d = s->local_mem + dst_base;
> +
> +            for (y = 0; y < height; y++) {
> +                i = (dst_x + (dst_y + y) * dst_pitch) * bypp;
> +                j = (src_x + (src_y + y) * src_pitch) * bypp;
> +                for (x = 0; x < width; x++, i += bypp, j += bypp) {
> +                    stn_he_p(&d[i], bypp,
> +                             ldn_he_p(&sp[j], bypp) | ldn_he_p(&d[i], bypp));
> +                }
> +            }
>           } else {
>               /* Do copy src for unimplemented ops, better than unpainted area 
> */
>               if ((rop_mode && (rop != 0xc || rop2_source_is_pattern)) ||