I have made the following changes intended for :
  CE:Adaptation:N9xx-common / ti-omap3-sgx

Please review and accept or decline.
BOSS has already run some checks on this request.
See the "Messages from BOSS" section below.

https://build.pub.meego.com//request/show/7239

Thank You,
Islam Amer

[This message was auto-generated]

---

Request # 7239:

Messages from BOSS:
     WARNING check_package_is_complete_sources (ti-omap3-sgx) failed: No dsc 
file found
Extra source files: boss.conf, ti-omap3-sgx-update.sh. 

State: review at 2012-11-01T17:14:14 by bossbot

Reviews:
       accepted by bossbot : Prechecks succeeded.
       new for CE-maintainers : Please replace this text with a review and 
approve/reject the review (not the SR). BOSS will take care of the rest

Changes:
  submit: home:iamer:branches:CE:Adaptation:N9xx-common / ti-omap3-sgx -> 
CE:Adaptation:N9xx-common / ti-omap3-sgx
  
changes files:
--------------
--- ti-omap3-sgx.changes
+++ ti-omap3-sgx.changes
@@ -0,0 +1,4 @@
+* Thu Nov 1 2012 Islam Amer <[email protected]> - 1.4.268
+- Use systemd module loading for pvrsrvkm to workaround startup race condition
+- Move modprobe omaplfb to after pvrsrvinit is done 
+

new:
----
  pvrsrvkm.conf

spec files:
-----------
--- ti-omap3-sgx.spec
+++ ti-omap3-sgx.spec
@@ -21,7 +21,7 @@
 Source3:    %{name}.udev
 Source4:    %{name}.service
 Source5:    Meego_IMG-08172010-license_txt.doc
-Source6:    boss.conf
+Source6:    pvrsrvkm.conf
 Source100:  ti-omap3-sgx.yaml
 Requires:   coreutils
 Requires:   grep
@@ -168,6 +168,7 @@
 install -D -m 644 %{SOURCE4} $RPM_BUILD_ROOT/lib/systemd/system/%{name}.service
 install -d $RPM_BUILD_ROOT/lib/systemd/system/basic.target.wants/
 ln -s ../%{name}.service 
$RPM_BUILD_ROOT/lib/systemd/system/basic.target.wants/%{name}.service
+install -D -m 644 %{SOURCE6} $RPM_BUILD_ROOT/etc/modules-load.d/pvrsrvkm.conf
 # << install pre
 
 # >> install post
@@ -221,6 +222,7 @@
 %attr(755,root,root) %{_sbindir}/pvrsrvinit
 /lib/systemd/system/%{name}.service
 /lib/systemd/system/basic.target.wants/%{name}.service
+%{_sysconfdir}/modules-load.d/pvrsrvkm.conf
 
 # Should be in it's own package.
 %{_libdir}/libOpenVG*.so

other changes:
--------------

++++++ pvrsrvkm.conf (new)
--- pvrsrvkm.conf
+++ pvrsrvkm.conf
@@ -0,0 +1 @@
+pvrsrvkm

++++++ ti-omap3-sgx.service
--- ti-omap3-sgx.service
+++ ti-omap3-sgx.service
@@ -5,8 +5,8 @@
 [Service]
 Type=oneshot
 RemainAfterExit=yes
-ExecStartPre=/sbin/modprobe omaplfb
 ExecStart=/usr/sbin/pvrsrvinit
+ExecStartPost=/sbin/modprobe omaplfb
 
 [Install]
 WantedBy=basic.target

++++++ ti-omap3-sgx.yaml
--- ti-omap3-sgx.yaml
+++ ti-omap3-sgx.yaml
@@ -11,7 +11,7 @@
     - "%{name}.udev"
     - "%{name}.service"
     - "Meego_IMG-08172010-license_txt.doc"
-    - "boss.conf"
+    - "pvrsrvkm.conf"
 
 Description: OMAP3 drivers for SGX
 Configure: none



Reply via email to