diff options
Diffstat (limited to 'target/linux/ps3/patches-2.6.28/0002-powerpc-ps3-Replace-the-flip_ctl-logic-in-ps3av-and.patch')
| -rw-r--r-- | target/linux/ps3/patches-2.6.28/0002-powerpc-ps3-Replace-the-flip_ctl-logic-in-ps3av-and.patch | 182 |
1 files changed, 182 insertions, 0 deletions
diff --git a/target/linux/ps3/patches-2.6.28/0002-powerpc-ps3-Replace-the-flip_ctl-logic-in-ps3av-and.patch b/target/linux/ps3/patches-2.6.28/0002-powerpc-ps3-Replace-the-flip_ctl-logic-in-ps3av-and.patch new file mode 100644 index 000000000..40cf93dac --- /dev/null +++ b/target/linux/ps3/patches-2.6.28/0002-powerpc-ps3-Replace-the-flip_ctl-logic-in-ps3av-and.patch @@ -0,0 +1,182 @@ +From 9b82f3e61758ed897200f0244b63a77c1791bcba Mon Sep 17 00:00:00 2001 +From: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com> +Date: Thu, 30 Oct 2008 08:12:58 +0000 +Subject: powerpc/ps3: Replace the flip_ctl logic in ps3av and ps3fb by a mutex + +Introduce ps3_gpu_mutex to synchronizes GPU-related operations, like: + - invoking the L1GPU_CONTEXT_ATTRIBUTE_FB_BLIT command using the + lv1_gpu_context_attribute() hypervisor call, + - handling the PS3AV_CID_AVB_PARAM packet in the PS3 A/V Settings driver. + +Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com> +Signed-off-by: Geoff Levand <geoffrey.levand@am.sony.com> +Signed-off-by: Paul Mackerras <paulus@samba.org> +--- + arch/powerpc/include/asm/ps3.h | 3 +++ + arch/powerpc/include/asm/ps3av.h | 4 ---- + arch/powerpc/platforms/ps3/setup.c | 4 ++++ + drivers/ps3/ps3av.c | 20 -------------------- + drivers/ps3/ps3av_cmd.c | 4 ++-- + drivers/video/ps3fb.c | 17 +++++------------ + 6 files changed, 14 insertions(+), 38 deletions(-) + +diff --git a/arch/powerpc/include/asm/ps3.h b/arch/powerpc/include/asm/ps3.h +index f9e34c4..4299365 100644 +--- a/arch/powerpc/include/asm/ps3.h ++++ b/arch/powerpc/include/asm/ps3.h +@@ -516,4 +516,7 @@ void ps3_sync_irq(int node); + u32 ps3_get_hw_thread_id(int cpu); + u64 ps3_get_spe_id(void *arg); + ++/* mutex synchronizing GPU accesses and video mode changes */ ++extern struct mutex ps3_gpu_mutex; ++ + #endif +diff --git a/arch/powerpc/include/asm/ps3av.h b/arch/powerpc/include/asm/ps3av.h +index 5aa22cf..cd24ac1 100644 +--- a/arch/powerpc/include/asm/ps3av.h ++++ b/arch/powerpc/include/asm/ps3av.h +@@ -740,8 +740,4 @@ extern int ps3av_audio_mute(int); + extern int ps3av_audio_mute_analog(int); + extern int ps3av_dev_open(void); + extern int ps3av_dev_close(void); +-extern void ps3av_register_flip_ctl(void (*flip_ctl)(int on, void *data), +- void *flip_data); +-extern void ps3av_flip_ctl(int on); +- + #endif /* _ASM_POWERPC_PS3AV_H_ */ +diff --git a/arch/powerpc/platforms/ps3/setup.c b/arch/powerpc/platforms/ps3/setup.c +index 77bc330..bfc33fb 100644 +--- a/arch/powerpc/platforms/ps3/setup.c ++++ b/arch/powerpc/platforms/ps3/setup.c +@@ -42,6 +42,10 @@ + #define DBG pr_debug + #endif + ++/* mutex synchronizing GPU accesses and video mode changes */ ++DEFINE_MUTEX(ps3_gpu_mutex); ++EXPORT_SYMBOL_GPL(ps3_gpu_mutex); ++ + #if !defined(CONFIG_SMP) + static void smp_send_stop(void) {} + #endif +diff --git a/drivers/ps3/ps3av.c b/drivers/ps3/ps3av.c +index 06848b2..5324978 100644 +--- a/drivers/ps3/ps3av.c ++++ b/drivers/ps3/ps3av.c +@@ -59,8 +59,6 @@ static struct ps3av { + struct ps3av_reply_hdr reply_hdr; + u8 raw[PS3AV_BUF_SIZE]; + } recv_buf; +- void (*flip_ctl)(int on, void *data); +- void *flip_data; + } *ps3av; + + /* color space */ +@@ -939,24 +937,6 @@ int ps3av_audio_mute(int mute) + + EXPORT_SYMBOL_GPL(ps3av_audio_mute); + +-void ps3av_register_flip_ctl(void (*flip_ctl)(int on, void *data), +- void *flip_data) +-{ +- mutex_lock(&ps3av->mutex); +- ps3av->flip_ctl = flip_ctl; +- ps3av->flip_data = flip_data; +- mutex_unlock(&ps3av->mutex); +-} +-EXPORT_SYMBOL_GPL(ps3av_register_flip_ctl); +- +-void ps3av_flip_ctl(int on) +-{ +- mutex_lock(&ps3av->mutex); +- if (ps3av->flip_ctl) +- ps3av->flip_ctl(on, ps3av->flip_data); +- mutex_unlock(&ps3av->mutex); +-} +- + static int ps3av_probe(struct ps3_system_bus_device *dev) + { + int res; +diff --git a/drivers/ps3/ps3av_cmd.c b/drivers/ps3/ps3av_cmd.c +index 11eb503..716596e 100644 +--- a/drivers/ps3/ps3av_cmd.c ++++ b/drivers/ps3/ps3av_cmd.c +@@ -864,7 +864,7 @@ int ps3av_cmd_avb_param(struct ps3av_pkt_avb_param *avb, u32 send_len) + { + int res; + +- ps3av_flip_ctl(0); /* flip off */ ++ mutex_lock(&ps3_gpu_mutex); + + /* avb packet */ + res = ps3av_do_pkt(PS3AV_CID_AVB_PARAM, send_len, sizeof(*avb), +@@ -878,7 +878,7 @@ int ps3av_cmd_avb_param(struct ps3av_pkt_avb_param *avb, u32 send_len) + res); + + out: +- ps3av_flip_ctl(1); /* flip on */ ++ mutex_unlock(&ps3_gpu_mutex); + return res; + } + +diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c +index 4b5d807..bd3e39b 100644 +--- a/drivers/video/ps3fb.c ++++ b/drivers/video/ps3fb.c +@@ -460,12 +460,16 @@ static void ps3fb_sync_image(struct device *dev, u64 frame_offset, + line_length |= (u64)src_line_length << 32; + + src_offset += GPU_FB_START; ++ ++ mutex_lock(&ps3_gpu_mutex); + status = lv1_gpu_context_attribute(ps3fb.context_handle, + L1GPU_CONTEXT_ATTRIBUTE_FB_BLIT, + dst_offset, GPU_IOIF + src_offset, + L1GPU_FB_BLIT_WAIT_FOR_COMPLETION | + (width << 16) | height, + line_length); ++ mutex_unlock(&ps3_gpu_mutex); ++ + if (status) + dev_err(dev, + "%s: lv1_gpu_context_attribute FB_BLIT failed: %d\n", +@@ -784,15 +788,6 @@ static int ps3fb_wait_for_vsync(u32 crtc) + return 0; + } + +-static void ps3fb_flip_ctl(int on, void *data) +-{ +- struct ps3fb_priv *priv = data; +- if (on) +- atomic_dec_if_positive(&priv->ext_flip); +- else +- atomic_inc(&priv->ext_flip); +-} +- + + /* + * ioctl +@@ -1228,7 +1223,6 @@ static int __devinit ps3fb_probe(struct ps3_system_bus_device *dev) + } + + ps3fb.task = task; +- ps3av_register_flip_ctl(ps3fb_flip_ctl, &ps3fb); + + return 0; + +@@ -1258,10 +1252,9 @@ static int ps3fb_shutdown(struct ps3_system_bus_device *dev) + + dev_dbg(&dev->core, " -> %s:%d\n", __func__, __LINE__); + +- ps3fb_flip_ctl(0, &ps3fb); /* flip off */ ++ atomic_inc(&ps3fb.ext_flip); /* flip off */ + ps3fb.dinfo->irq.mask = 0; + +- ps3av_register_flip_ctl(NULL, NULL); + if (ps3fb.task) { + struct task_struct *task = ps3fb.task; + ps3fb.task = NULL; +-- +1.6.0.4 + |