Skip to contentSkip to conversation

package read-edid, corrected patch

PATCH New Dimitry Golubovsky 3 participants 3 messages May 4, 2012

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package read-edid, corrected patch

Hi,
Here is the updated patch per comments by Thomas and Baruch.

Thanks.

---

package/Config.in +1 -0
index caa618b..e39fccf 100644
@@ -210,6 +210,7 @@ source "package/openocd/Config.in"
 source "package/parted/Config.in"
 source "package/pciutils/Config.in"
 source "package/picocom/Config.in"
+source "package/read-edid/Config.in"
 source "package/rng-tools/Config.in"
 source "package/sane-backends/Config.in"
 source "package/sdparm/Config.in"
package/read-edid/Config.in new file +4 -0
new file mode 100644
index 0000000..c66a5e0
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_READ_EDID
+	bool "read-edid"
+	help
+	  Read-edid is a pair of tools (originally by John Fremlin) for

reading the EDID
+ from a monitor. It should work with most monitors made since 1996
(except for
+ newer ones with 256-byte EDID's - WiP), assuming the video card
supports the
+ standard read commands (most do). Read-edid is a set of two tools
- get-edid,
+ which gets the raw edid information from the monitor, and
parse-edid, which
+ turns the raw binary information into an XF86Config-compatible
monitor section.
+
+ http://polypux.org/projects/read-edid/
+
+depends on (BR2_i386)
+
+

package/read-edid/read-edid.mk new file +20 -0
new file mode 100644
index 0000000..6328def
@@ -0,0 +1,20 @@
+############################################
+#
+# read-edid
+#
+############################################
+
+READ_EDID_VERSION = 1.4.2
+READ_EDID_SITE = http://www.polypux.org/projects/read-edid/
+
+define READ_EDID_BUILD_CMDS
+	$(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D)
+endef
+
+define READ_EDID_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/get-edid $(TARGET_DIR)/sbin/get-edid
+	$(INSTALL) -D -m 0755 $(@D)/parse-edid $(TARGET_DIR)/sbin/parse-edid
+endef
+
+$(eval $(call AUTOTARGETS))
+

--

#1 lore raw

PATCH 1/1 package read-edid, corrected patch

Hi Dimitry,

Quoted 4 lines · Dimitry Golubovsky

Hi,
Here is the updated patch per comments by Thomas and Baruch.

Thanks.

The text here should contain the patch changelog information. This goes directly to the git log. Other free text goes below...

Quoted 2 lines

---

here.

Quoted 32 lines
package/Config.in +1 -0
index caa618b..e39fccf 100644
@@ -210,6 +210,7 @@ source "package/openocd/Config.in"
 source "package/parted/Config.in"
 source "package/pciutils/Config.in"
 source "package/picocom/Config.in"
+source "package/read-edid/Config.in"
 source "package/rng-tools/Config.in"
 source "package/sane-backends/Config.in"
 source "package/sdparm/Config.in"
package/read-edid/Config.in new file +4 -0
new file mode 100644
index 0000000..c66a5e0
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_READ_EDID
+	bool "read-edid"
+	help
+	  Read-edid is a pair of tools (originally by John Fremlin) for

reading the EDID

package/read-edid/Config.in new file +1 -0
+	  from a monitor.  It should work with most monitors made since 1996

(except for

package/read-edid/Config.in new file +1 -0
+	  newer ones with 256-byte EDID's - WiP),  assuming the video card

supports the

package/read-edid/Config.in new file +2 -1
+	  standard read commands (most do).  Read-edid is a set of two tools
- get-edid,
+	  which gets  the raw edid  information from the monitor,  and

parse-edid, which

package/read-edid/Config.in new file +1 -0
+	  turns the raw binary information into an XF86Config-compatible
Quoted 1 line

monitor section.

Your patch is line wrapped. I still suggest you to consider 'git send-email'.

package/read-edid/Config.in

Quoted 4 lines
package/read-edid/Config.in new file +4 -0
+
+	  http://polypux.org/projects/read-edid/
+
+depends on (BR2_i386)

This line should be placed just below the 'bool "read-edid"' line, and be tab indented. The help text should be indented with tab + 2 spaces.

Quoted 17 lines
package/read-edid/Config.in new file +2 -0
+
+
package/read-edid/read-edid.mk new file +9 -0
new file mode 100644
index 0000000..6328def
@@ -0,0 +1,20 @@
+############################################
+#
+# read-edid
+#
+############################################
+
+READ_EDID_VERSION = 1.4.2
+READ_EDID_SITE = http://www.polypux.org/projects/read-edid/
+

package/read-edid/read-edid.mk line 19

Quoted 10 lines
package/read-edid/read-edid.mk new file +10 -0
+define READ_EDID_BUILD_CMDS
+	$(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D)
+endef
+
+define READ_EDID_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/get-edid $(TARGET_DIR)/sbin/get-edid
+	$(INSTALL) -D -m 0755 $(@D)/parse-edid $(TARGET_DIR)/sbin/parse-edid
+endef
+
+$(eval $(call AUTOTARGETS))

baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
#2 lore raw

PATCH 1/1 package read-edid, corrected patch

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Hi,

Quoted 6 lines
Quoted 6 lines
package/read-edid/Config.in new file +3 -0
+	bool "read-edid"
+	help
+	  Read-edid is a pair of tools (originally by John Fremlin) for

reading the EDID

(quoted diff) +1 -0
+	  from a monitor.  It should work with most monitors made since 1996

(except for

Quoted 8 lines
Quoted 8 lines
(quoted diff) +1 -0
+	  newer ones with 256-byte EDID's - WiP),  assuming the video card

supports the

(quoted diff) +2 -1
+	  standard read commands (most do).  Read-edid is a set of two tools
- get-edid,
+	  which gets  the raw edid  information from the monitor,  and

parse-edid, which

(quoted diff) +1 -0
+	  turns the raw binary information into an XF86Config-compatible

monitor section.

Baruch> Your patch is line wrapped. I still suggest you to consider 'git send-email'.

And please also wrap the help text at ~75 chars or so.

Signature · 1 line
Bye, Peter Korsgaard