summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
author AJR <ajrhacker@users.noreply.github.com>2019-09-02 16:18:37 -0400
committer AJR <ajrhacker@users.noreply.github.com>2019-09-02 16:20:23 -0400
commitf3ce368aafe6be885f5d35e267c235791c1e4dff (patch)
treec61366075d8db62e5b12519649b9da85feadfffd
parent6029a46c16cfdf4f00e35a8f60bf5d8caed6d3b2 (diff)
tia_video: Use device_palette_interface; save bitmaps; convert update method to rgb32 (nw)
-rw-r--r--src/mame/drivers/a2600.cpp2
-rw-r--r--src/mame/drivers/tourtabl.cpp1
-rw-r--r--src/mame/drivers/tvboy.cpp1
-rw-r--r--src/mame/video/tia.cpp78
-rw-r--r--src/mame/video/tia.h24
5 files changed, 43 insertions, 63 deletions
diff --git a/src/mame/drivers/a2600.cpp b/src/mame/drivers/a2600.cpp
index 5bb1bd347df..6cbeeef3a4d 100644
--- a/src/mame/drivers/a2600.cpp
+++ b/src/mame/drivers/a2600.cpp
@@ -521,7 +521,6 @@ void a2600_state::a2600(machine_config &config)
SCREEN(config, m_screen, SCREEN_TYPE_RASTER);
m_screen->set_raw(MASTER_CLOCK_NTSC, 228, 26, 26 + 160 + 16, 262, 24 , 24 + 192 + 31);
m_screen->set_screen_update("tia_video", FUNC(tia_video_device::screen_update));
- m_screen->set_palette("tia_video:palette");
/* sound hardware */
SPEAKER(config, "mono").front_center();
@@ -568,7 +567,6 @@ void a2600_state::a2600p(machine_config &config)
SCREEN(config, m_screen, SCREEN_TYPE_RASTER);
m_screen->set_raw(MASTER_CLOCK_PAL, 228, 26, 26 + 160 + 16, 312, 32, 32 + 228 + 31);
m_screen->set_screen_update("tia_video", FUNC(tia_video_device::screen_update));
- m_screen->set_palette("tia_video:palette");
/* sound hardware */
SPEAKER(config, "mono").front_center();
diff --git a/src/mame/drivers/tourtabl.cpp b/src/mame/drivers/tourtabl.cpp
index 34917009195..50e28dd9f87 100644
--- a/src/mame/drivers/tourtabl.cpp
+++ b/src/mame/drivers/tourtabl.cpp
@@ -180,7 +180,6 @@ void tourtabl_state::tourtabl(machine_config &config)
screen_device &screen(SCREEN(config, "screen", SCREEN_TYPE_RASTER));
screen.set_raw(MASTER_CLOCK, 228, 34, 34 + 160, 262, 46, 46 + 200);
screen.set_screen_update("tia_video", FUNC(tia_video_device::screen_update));
- screen.set_palette("tia_video:palette");
/* sound hardware */
SPEAKER(config, "mono").front_center();
diff --git a/src/mame/drivers/tvboy.cpp b/src/mame/drivers/tvboy.cpp
index e276e01f772..52cc083a064 100644
--- a/src/mame/drivers/tvboy.cpp
+++ b/src/mame/drivers/tvboy.cpp
@@ -92,7 +92,6 @@ void tvboy_state::tvboyii(machine_config &config)
SCREEN(config, m_screen, SCREEN_TYPE_RASTER);
m_screen->set_raw(MASTER_CLOCK_PAL, 228, 26, 26 + 160 + 16, 312, 32, 32 + 228 + 31);
m_screen->set_screen_update("tia_video", FUNC(tia_video_device::screen_update));
- m_screen->set_palette("tia_video:palette");
/* sound hardware */
SPEAKER(config, "mono").front_center();
diff --git a/src/mame/video/tia.cpp b/src/mame/video/tia.cpp
index 1c60324d6a7..6fe49fb13e7 100644
--- a/src/mame/video/tia.cpp
+++ b/src/mame/video/tia.cpp
@@ -22,24 +22,23 @@ static const int nusiz[8][3] =
{ 1, 4, 0 }
};
-static void extend_palette(palette_device &palette) {
- int i,j;
-
- for( i = 0; i < 128; i ++ )
+void tia_video_device::extend_palette()
+{
+ for (int i = 0; i < 128; i++)
{
- rgb_t new_rgb = palette.pen_color( i );
+ rgb_t new_rgb = pen_color( i );
uint8_t new_r = new_rgb .r();
uint8_t new_g = new_rgb .g();
uint8_t new_b = new_rgb .b();
- for ( j = 0; j < 128; j++ )
+ for (int j = 0; j < 128; j++)
{
- rgb_t old_rgb = palette.pen_color( j );
+ rgb_t old_rgb = pen_color( j );
uint8_t old_r = old_rgb .r();
uint8_t old_g = old_rgb .g();
uint8_t old_b = old_rgb .b();
- palette.set_pen_color(( ( i + 1 ) << 7 ) | j,
+ set_pen_color(( ( i + 1 ) << 7 ) | j,
( new_r + old_r ) / 2,
( new_g + old_g ) / 2,
( new_b + old_b ) / 2 );
@@ -47,7 +46,7 @@ static void extend_palette(palette_device &palette) {
}
}
-void tia_ntsc_video_device::tia_ntsc_palette(palette_device &palette) const
+void tia_ntsc_video_device::init_palette()
{
/********************************************************************
Atari 2600 NTSC Palette Notes:
@@ -282,18 +281,18 @@ void tia_ntsc_video_device::tia_ntsc_palette(palette_device &palette) const
if (G > 1) G = 1;
if (B > 1) B = 1;
- palette.set_pen_color(
+ set_pen_color(
8 * i + j,
uint8_t(255 * R + 0.5),
uint8_t(255 * G + 0.5),
uint8_t(255 * B + 0.5));
}
}
- extend_palette(palette);
+ extend_palette();
}
-void tia_pal_video_device::tia_pal_palette(palette_device &palette) const
+void tia_pal_video_device::init_palette()
{
static constexpr double color[16][2] =
{
@@ -340,19 +339,20 @@ void tia_pal_video_device::tia_pal_palette(palette_device &palette) const
if (G > 1) G = 1;
if (B > 1) B = 1;
- palette.set_pen_color(
+ set_pen_color(
8 * i + j,
uint8_t(255 * R + 0.5),
uint8_t(255 * G + 0.5),
uint8_t(255 * B + 0.5));
}
}
- extend_palette(palette);
+ extend_palette();
}
tia_video_device::tia_video_device(const machine_config &mconfig, device_type type, const char *tag, device_t *owner, uint32_t clock)
: device_t(mconfig, type, tag, owner, clock)
, device_video_interface(mconfig, *this)
+ , device_palette_interface(mconfig, *this)
, m_read_input_port_cb(*this)
, m_databus_contents_cb(*this)
, m_vsync_cb(*this)
@@ -373,15 +373,6 @@ tia_pal_video_device::tia_pal_video_device(const machine_config &mconfig, const
{
}
-//-------------------------------------------------
-// device_add_mconfig - add device configuration
-//-------------------------------------------------
-
-void tia_pal_video_device::device_add_mconfig(machine_config &config)
-{
- PALETTE(config, "palette", FUNC(tia_pal_video_device::tia_pal_palette), TIA_PALETTE_LENGTH);
-}
-
// device type definition
DEFINE_DEVICE_TYPE(TIA_NTSC_VIDEO, tia_ntsc_video_device, "tia_ntsc_video", "TIA Video (NTSC)")
@@ -395,15 +386,6 @@ tia_ntsc_video_device::tia_ntsc_video_device(const machine_config &mconfig, cons
}
//-------------------------------------------------
-// device_add_mconfig - add device configuration
-//-------------------------------------------------
-
-void tia_ntsc_video_device::device_add_mconfig(machine_config &config)
-{
- PALETTE(config, "palette", FUNC(tia_ntsc_video_device::tia_ntsc_palette), TIA_PALETTE_LENGTH);
-}
-
-//-------------------------------------------------
// device_start - device-specific startup
//-------------------------------------------------
@@ -414,13 +396,14 @@ void tia_video_device::device_start()
m_databus_contents_cb.resolve();
m_vsync_cb.resolve();
+ init_palette();
int cx = screen().width();
screen_height = screen().height();
- helper[0] = std::make_unique<bitmap_ind16>(cx, TIA_MAX_SCREEN_HEIGHT);
- helper[1] = std::make_unique<bitmap_ind16>(cx, TIA_MAX_SCREEN_HEIGHT);
- helper[2] = std::make_unique<bitmap_ind16>(cx, TIA_MAX_SCREEN_HEIGHT);
+ helper[0].allocate(cx, TIA_MAX_SCREEN_HEIGHT);
+ helper[1].allocate(cx, TIA_MAX_SCREEN_HEIGHT);
+ buffer.allocate(cx, TIA_MAX_SCREEN_HEIGHT);
register_save_state();
}
@@ -430,10 +413,10 @@ void tia_video_device::device_start()
// screen_update -
//-------------------------------------------------
-uint32_t tia_video_device::screen_update(screen_device &screen, bitmap_ind16 &bitmap, const rectangle &cliprect)
+uint32_t tia_video_device::screen_update(screen_device &screen, bitmap_rgb32 &bitmap, const rectangle &cliprect)
{
screen_height = screen.height();
- copybitmap(bitmap, *helper[2], 0, 0, 0, 0, cliprect);
+ copybitmap(bitmap, buffer, 0, 0, 0, 0, cliprect);
return 0;
}
@@ -969,7 +952,7 @@ void tia_video_device::update_bitmap(int next_x, int next_y)
if (collision_check(lineM0, lineM1, colx1, x2))
CXPPMM |= 0x40;
- p = &helper[current_bitmap]->pix16(y % screen_height, 34);
+ p = &helper[current_bitmap].pix16(y % screen_height, 34);
for (x = x1; x < x2; x++)
{
@@ -978,17 +961,17 @@ void tia_video_device::update_bitmap(int next_x, int next_y)
if ( x2 == 160 && y % screen_height == (screen_height - 1) ) {
int t_y;
- for ( t_y = 0; t_y < helper[2]->height(); t_y++ ) {
- uint16_t* l0 = &helper[current_bitmap]->pix16(t_y);
- uint16_t* l1 = &helper[1 - current_bitmap]->pix16(t_y);
- uint16_t* l2 = &helper[2]->pix16(t_y);
+ for ( t_y = 0; t_y < buffer.height(); t_y++ ) {
+ uint16_t* l0 = &helper[current_bitmap].pix16(t_y);
+ uint16_t* l1 = &helper[1 - current_bitmap].pix16(t_y);
+ uint32_t* l2 = &buffer.pix32(t_y);
int t_x;
- for( t_x = 0; t_x < helper[2]->width(); t_x++ ) {
+ for( t_x = 0; t_x < buffer.width(); t_x++ ) {
if ( l0[t_x] != l1[t_x] ) {
/* Combine both entries */
- l2[t_x] = ( ( l0[t_x] + 1 ) << 7 ) | l1[t_x];
+ l2[t_x] = pen(( ( l0[t_x] + 1 ) << 7 ) | l1[t_x]);
} else {
- l2[t_x] = l0[t_x];
+ l2[t_x] = pen(l0[t_x]);
}
}
}
@@ -1338,7 +1321,7 @@ WRITE8_MEMBER( tia_video_device::HMOVE_w )
}
if (curr_y < screen_height)
{
- memset(&helper[current_bitmap]->pix16(curr_y, 34), 0, 16);
+ memset(&helper[current_bitmap].pix16(curr_y, 34), 0, 16);
}
prev_x = 8;
@@ -2256,4 +2239,7 @@ void tia_video_device::register_save_state()
save_item(NAME(HMBL_latch));
save_item(NAME(REFLECT));
save_item(NAME(NUSIZx_changed));
+ save_item(NAME(helper[0]));
+ save_item(NAME(helper[1]));
+ save_item(NAME(buffer));
}
diff --git a/src/mame/video/tia.h b/src/mame/video/tia.h
index 6ae40f9a8bd..0947e4ea87f 100644
--- a/src/mame/video/tia.h
+++ b/src/mame/video/tia.h
@@ -6,7 +6,6 @@
#pragma once
#include "sound/tiaintf.h"
-#include "emupal.h"
//**************************************************************************
// MACROS / CONSTANTS
@@ -39,11 +38,10 @@ struct player_gfx {
// ======================> tia_video_device
-class tia_video_device : public device_t,
- public device_video_interface
+class tia_video_device : public device_t, public device_video_interface, public device_palette_interface
{
public:
- uint32_t screen_update(screen_device &screen, bitmap_ind16 &bitmap, const rectangle &cliprect);
+ uint32_t screen_update(screen_device &screen, bitmap_rgb32 &bitmap, const rectangle &cliprect);
auto read_input_port_callback() { return m_read_input_port_cb.bind(); }
auto databus_contents_callback() { return m_databus_contents_cb.bind(); }
@@ -62,6 +60,11 @@ protected:
virtual void device_start() override;
virtual void device_reset() override;
+ // device_palette_interface overrides
+ virtual uint32_t palette_entries() const override { return TIA_PALETTE_LENGTH; }
+
+ void extend_palette();
+ virtual void init_palette() = 0;
void draw_sprite_helper(uint8_t* p, uint8_t *col, struct player_gfx *gfx, uint8_t GRP, uint8_t COLUP, uint8_t REFP);
void draw_missile_helper(uint8_t* p, uint8_t* col, int horz, int skipdelay, int latch, int start, uint8_t RESMP, uint8_t ENAM, uint8_t NUSIZ, uint8_t COLUM);
void draw_playfield_helper(uint8_t* p, uint8_t* col, int horz, uint8_t COLU, uint8_t REFPF);
@@ -196,7 +199,8 @@ private:
uint8_t REFLECT; /* Should playfield be reflected or not */
uint8_t NUSIZx_changed;
- std::unique_ptr<bitmap_ind16> helper[3];
+ bitmap_ind16 helper[2];
+ bitmap_rgb32 buffer;
uint16_t screen_height;
@@ -215,10 +219,7 @@ public:
tia_pal_video_device(const machine_config &mconfig, const char *tag, device_t *owner, uint32_t clock);
protected:
- virtual void device_add_mconfig(machine_config &config) override;
-
-private:
- void tia_pal_palette(palette_device &palette) const;
+ virtual void init_palette() override;
};
class tia_ntsc_video_device : public tia_video_device
@@ -233,10 +234,7 @@ public:
tia_ntsc_video_device(const machine_config &mconfig, const char *tag, device_t *owner, uint32_t clock);
protected:
- virtual void device_add_mconfig(machine_config &config) override;
-
-private:
- void tia_ntsc_palette(palette_device &palette) const;
+ virtual void init_palette() override;
};