From f820ef62df8956e5f56fad375e3ab6f9d819718d Mon Sep 17 00:00:00 2001
From: MatMoul <matmoul@gmail.com>
Date: Wed, 9 Aug 2017 22:06:35 +0200
Subject: [PATCH 1/7] Update README.md

---
 README.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 494a373..c08fb6b 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
 # g810-led</br>
 
-Linux led controller for Logitech G213, G410, G610, G810, G910 and GPRO Keyboards.</br>
+Linux led controller for Logitech G213, G410, g413, G610, G810, G910 and GPRO Keyboards.</br>
 
 ## Compatible keyboards :</br>
 - **G213 Prodigy**</br>
@@ -22,6 +22,7 @@ Linux led controller for Logitech G213, G410, G610, G810, G910 and GPRO Keyboard
 ## Help :</br>
 `g213-led --help`</br>
 `g410-led --help`</br>
+`g413-led --help`</br>
 `g610-led --help`</br>
 `g810-led --help`</br>
 `g910-led --help`</br>

From 7961599cec41a44889d5b63efd4c502502649424 Mon Sep 17 00:00:00 2001
From: GREAT_BIG_BUSHY_BEARD <simon.whyte.lb@gmail.com>
Date: Wed, 25 Oct 2017 23:37:43 +0100
Subject: [PATCH 2/7] Fix spelling mistakes and grammar in README and INSTALL

---
 INSTALL.md | 13 ++++++-------
 README.md  |  4 ++--
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/INSTALL.md b/INSTALL.md
index ceb841d..02c9786 100644
--- a/INSTALL.md
+++ b/INSTALL.md
@@ -11,11 +11,11 @@
 ## hidapi vs libusb :</br>
 hidapi is a newer implementation but needs more testing.</br>
 hidapi is more responsive than libusb (~20ms vs ~150ms).</br>
-hidapi seem not work on CentOS, writing on hidraw is not allowed.</br>
-hidapi is recommended but if you encounter problem on your system, switch to libusb.</br>
+hidapi seems to not work on CentOS, writing to hidraw is not allowed.</br>
+hidapi is recommended but if you encounter a problem on your system, switch to libusb.</br>
 
 
-## Installation by repos :</br>
+## Installation using repos :</br>
 ArchLinux (aur) :</br>
 `yaourt -S g810-led-git` # with yaourt</br>
 `pacaur -S g810-led-git` # with pacaur</br>
@@ -39,7 +39,7 @@ Fedora :</br>
 `make bin LIB=libusb` # for libusb</br>
 `sudo make install`</br>
 
-## Installation of the library (For developpers) :</br>
+## Installation of the library (For developers) :</br>
 `make lib` # for hidapi</br>
 `make lib LIB=libusb` # for libusb</br>
 `sudo make install-lib` to install the libg810-led library.</br>
@@ -53,8 +53,7 @@ Same as install, but your profile and reboot files are preserved.</br>
 
 ## Boot profiles :</br>
 On boot, the keyboard is set with the udev file /etc/udev/rules.d/g810-led.rules</br>
-This file launch the profile stored in /etc/g810-led/profile</br>
-To prevent your keyboard to flash 3 time when you reboot, a systemd unit (g810-led-reboot) is available for this.</br>
-It launch the profile stored in /etc/g810-led/reboot</br>
+This file launches the profile stored in /etc/g810-led/profile</br>
+To prevent your keyboard flashing 3 times when you reboot use the systemd unit (g810-led-reboot).</br>
 
 Samples can be found in /etc/g810-led/samples.</br>
diff --git a/README.md b/README.md
index c08fb6b..0bad66e 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
 # g810-led</br>
 
-Linux led controller for Logitech G213, G410, g413, G610, G810, G910 and GPRO Keyboards.</br>
+Linux led controller for Logitech G213, G410, G413, G610, G810, G910 and GPRO Keyboards.</br>
 
 ## Compatible keyboards :</br>
 - **G213 Prodigy**</br>
@@ -68,7 +68,7 @@ Linux led controller for Logitech G213, G410, g413, G610, G810, G910 and GPRO Ke
 `echo -e "k w ff0000\nk a ff0000\nk s ff0000\nk d ff0000\nc" | g810-led -pp # Set multiple keys`</br>
 
 ## Testing unsuported keyboards :</br>
-Start by retrieve the VendorID and the ProductID of your keyboard with lsusb.</br>
+Start by retrieving the VendorID and the ProductID of your keyboard using lsusb.</br>
 `lsusb`</br>
 Sample return :<br>
 `Bus 001 Device 001: ID 046d:c331 Logitech, Inc.`</br>

From 33912d8dcea61fb0d3e9c9d93183987857b5b6db Mon Sep 17 00:00:00 2001
From: GREAT_BIG_BUSHY_BEARD <simon.whyte.lb@gmail.com>
Date: Wed, 25 Oct 2017 23:38:46 +0100
Subject: [PATCH 3/7] Fixed heading consistency

---
 README.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 0bad66e..9c1e021 100644
--- a/README.md
+++ b/README.md
@@ -54,12 +54,12 @@ Linux led controller for Logitech G213, G410, G413, G610, G810, G910 and GPRO Ke
 `g810-led -kn d ff0000 # Set color of a key with no action`</br>
 `g810-led -c # Commit all changes`</br>
 
-## Samples for g610 :</br>
+## Samples for G610 :</br>
 `g610-led -a 60 # Set intensity of all keys`</br>
 `g610-led -k logo ff # Set intensity of a key`</br>
 `g610-led -g fkeys aa # Set intensity of a group of keys`</br>
 
-## Samples for g213 :</br>
+## Samples for G213 :</br>
 `g213-led -a 00ff00 # Set all keys green`</br>
 `g213-led -r 1 ff0000 # Set region 1 red`</br>
 

From 68551c65ecebcf08415fa73fcd9b67fe92e31c66 Mon Sep 17 00:00:00 2001
From: MatMoul <matmoul@gmail.com>
Date: Wed, 1 Aug 2018 23:37:29 +0200
Subject: [PATCH 4/7] Add support to G512 and G513

---
 README.md                | 6 +++++-
 makefile                 | 5 +++++
 src/classes/Keyboard.cpp | 4 ++++
 src/classes/Keyboard.h   | 2 ++
 udev/g810-led.rules      | 1 +
 5 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 9c1e021..9d653d4 100644
--- a/README.md
+++ b/README.md
@@ -1,11 +1,13 @@
 # g810-led</br>
 
-Linux led controller for Logitech G213, G410, G413, G610, G810, G910 and GPRO Keyboards.</br>
+Linux led controller for Logitech G213, G410, G413, G512, G513, G610, G810, G910 and GPRO Keyboards.</br>
 
 ## Compatible keyboards :</br>
 - **G213 Prodigy**</br>
 - **G410 Atlas Spectrum**</br>
 - **G413 Carbon**</br>
+- **G512 Carbon**</br>
+- **G513 Carbon**</br>
 - **G610 Orion Brown**</br>
 - **G610 Orion Red**</br>
 - **G810 Orion Spectrum**</br>
@@ -23,6 +25,8 @@ Linux led controller for Logitech G213, G410, G413, G610, G810, G910 and GPRO Ke
 `g213-led --help`</br>
 `g410-led --help`</br>
 `g413-led --help`</br>
+`g512-led --help`</br>
+`g513-led --help`</br>
 `g610-led --help`</br>
 `g810-led --help`</br>
 `g910-led --help`</br>
diff --git a/makefile b/makefile
index 019636d..e912e8c 100644
--- a/makefile
+++ b/makefile
@@ -61,6 +61,8 @@ setup:
 	@test -s $(DESTDIR)/usr/bin/g213-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g213-led
 	@test -s $(DESTDIR)/usr/bin/g410-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g410-led
 	@test -s $(DESTDIR)/usr/bin/g413-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g413-led
+	@test -s $(DESTDIR)/usr/bin/g512-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g512-led
+	@test -s $(DESTDIR)/usr/bin/g513-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g513-led
 	@test -s $(DESTDIR)/usr/bin/g610-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g610-led
 	@test -s $(DESTDIR)/usr/bin/g910-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/g910-led
 	@test -s $(DESTDIR)/usr/bin/gpro-led || ln -s /usr/bin/$(PROGN) $(DESTDIR)/usr/bin/gpro-led
@@ -106,6 +108,9 @@ uninstall:
 	@rm /usr/bin/g213-led
 	@rm /usr/bin/g410-led
 	@rm /usr/bin/g413-led
+	@rm /usr/bin/g413-led
+	@rm /usr/bin/g512-led
+	@rm /usr/bin/g513-led
 	@rm /usr/bin/g610-led
 	@rm /usr/bin/g910-led
 	@rm /usr/bin/gpro-led
diff --git a/src/classes/Keyboard.cpp b/src/classes/Keyboard.cpp
index f657b2d..9c54f9d 100644
--- a/src/classes/Keyboard.cpp
+++ b/src/classes/Keyboard.cpp
@@ -358,6 +358,7 @@ bool LedKeyboard::commit() {
 		case KeyboardModel::g413:
 			return true; // Keyboard is non-transactional
 		case KeyboardModel::g410:
+    case KeyboardModel::g513:
 		case KeyboardModel::g610:
 		case KeyboardModel::g810:
 		case KeyboardModel::gpro:
@@ -569,6 +570,7 @@ bool LedKeyboard::setAllKeys(LedKeyboard::Color color) {
 			setNativeEffect(NativeEffect::color, NativeEffectPart::keys, 0, color);
 			return true;
 		case KeyboardModel::g410:
+    case KeyboardModel::g513:
 		case KeyboardModel::g610:
 		case KeyboardModel::g810:
 		case KeyboardModel::g910:
@@ -731,6 +733,7 @@ bool LedKeyboard::setNativeEffect(NativeEffect effect, NativeEffectPart part, ui
 			if (part == NativeEffectPart::logo) return true; //Does not have logo component
 			break;
 		case KeyboardModel::g410:
+    case KeyboardModel::g513:
 		case KeyboardModel::g610: // Unconfirmed
 		case KeyboardModel::g810:
 		case KeyboardModel::gpro:
@@ -856,6 +859,7 @@ LedKeyboard::byte_buffer_t LedKeyboard::getKeyGroupAddress(LedKeyboard::KeyAddre
 		case KeyboardModel::g413:
 		  return {}; // Device doesn't support per-key setting
 		case KeyboardModel::g410:
+    case KeyboardModel::g513:
 		case KeyboardModel::g610:
 		case KeyboardModel::g810:
 		case KeyboardModel::gpro:
diff --git a/src/classes/Keyboard.h b/src/classes/Keyboard.h
index 5e6dca8..76e178f 100644
--- a/src/classes/Keyboard.h
+++ b/src/classes/Keyboard.h
@@ -31,6 +31,7 @@ class LedKeyboard {
 			{ 0x46d, 0xc336, (u_int16_t)KeyboardModel::g213 },
 			{ 0x46d, 0xc330, (u_int16_t)KeyboardModel::g410 },
 			{ 0x46d, 0xc33a, (u_int16_t)KeyboardModel::g413 },
+			{ 0x46d, 0xc33c, (u_int16_t)KeyboardModel::g513 },
 			{ 0x46d, 0xc333, (u_int16_t)KeyboardModel::g610 },
 			{ 0x46d, 0xc338, (u_int16_t)KeyboardModel::g610 },
 			{ 0x46d, 0xc331, (u_int16_t)KeyboardModel::g810 },
@@ -45,6 +46,7 @@ class LedKeyboard {
 			g213,
 			g410,
 			g413,
+      g513,
 			g610,
 			g810,
 			g910,
diff --git a/udev/g810-led.rules b/udev/g810-led.rules
index d0935f3..08c0d63 100644
--- a/udev/g810-led.rules
+++ b/udev/g810-led.rules
@@ -1,6 +1,7 @@
 ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c336", MODE="666" RUN+="/usr/bin/g213-led -p /etc/g810-led/profile"
 ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c330", MODE="666" RUN+="/usr/bin/g410-led -p /etc/g810-led/profile"
 ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c33a", MODE="666" RUN+="/usr/bin/g413-led -p /etc/g810-led/profile"
+ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c33c", MODE="666" RUN+="/usr/bin/g513-led -p /etc/g810-led/profile"
 ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c333", MODE="666" RUN+="/usr/bin/g610-led -p /etc/g810-led/profile"
 ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c338", MODE="666" RUN+="/usr/bin/g610-led -p /etc/g810-led/profile"
 ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="046d", ATTRS{idProduct}=="c331", MODE="666" RUN+="/usr/bin/g810-led -p /etc/g810-led/profile"

From 5fa7760e53a0a0234983bfad622f6614374989c2 Mon Sep 17 00:00:00 2001
From: "Haelwenn (lanodan) Monnier" <contact@hacktivis.me>
Date: Fri, 31 Aug 2018 01:25:29 +0200
Subject: [PATCH 5/7] makefile: Use standard variables and reuse the
 environment

---
 makefile | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/makefile b/makefile
index 019636d..bb3f368 100644
--- a/makefile
+++ b/makefile
@@ -1,18 +1,18 @@
-CC=g++
-CFLAGS=-Wall -O2 -std=gnu++11
+CXX?=g++
+CXXFLAGS?=-Wall -O2 -std=gnu++11
 LIB?=hidapi
 ifeq ($(LIB),libusb)
 	CPPFLAGS=-Dlibusb
-	LDFLAGS=-lusb-1.0
+	LDFLAGS+=-lusb-1.0
 else
 	CPPFLAGS=-Dhidapi
-	LDFLAGS=-lhidapi-hidraw
+	LDFLAGS+=-lhidapi-hidraw
 endif
 SYSTEMDDIR?=/usr/lib/systemd
 
-prefix?=$(DESTDIR)/usr
-libdir?=$(prefix)/lib
-includedir?=$(prefix)/include
+PREFIX?=$(DESTDIR)/usr
+libdir?=$(PREFIX)/lib
+includedir?=$(PREFIX)/include
 
 # Program & versioning information
 PROGN=g810-led
@@ -20,7 +20,7 @@ MAJOR=0
 MINOR=2
 MICRO=7
 
-CFLAGS+=-DVERSION=\"$(MAJOR).$(MINOR).$(MICRO)\"
+CXXFLAGS+=-DVERSION=\"$(MAJOR).$(MINOR).$(MICRO)\"
 APPSRCS=src/main.cpp src/helpers/*.cpp src/helpers/*.h
 LIBSRCS=src/classes/*.cpp src/classes/*.h
 
@@ -32,19 +32,19 @@ bin: bin/$(PROGN)
 
 bin/$(PROGN): $(APPSRCS) $(LIBSRCS)
 	@mkdir -p bin
-	$(CC) $(CPPFLAGS) $(CFLAGS) $^ -o $@ $(LDFLAGS)
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $^ -o $@ $(LDFLAGS)
 	
-debug: CFLAGS += -g -Wextra -pedantic
+debug: CXXFLAGS += -g -Wextra -pedantic
 debug: bin/$(PROGN)
 
 lib/lib$(PROGN).so: $(LIBSRCS)
 	@mkdir -p lib
-	$(CC) $(CPPFLAGS) $(CFLAGS) -fPIC -shared -Wl,-soname,lib$(PROGN).so -o lib/lib$(PROGN).so.$(MAJOR).$(MINOR).$(MICRO) $^ $(LDFLAGS)
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) -fPIC -shared -Wl,-soname,lib$(PROGN).so -o lib/lib$(PROGN).so.$(MAJOR).$(MINOR).$(MICRO) $^ $(LDFLAGS)
 	@ln -sf lib$(PROGN).so.$(MAJOR).$(MINOR).$(MICRO) lib/lib$(PROGN).so
 
 bin-linked: lib/lib$(PROGN).so
 	@mkdir -p bin
-	$(CC) $(CPPFLAGS) $(CFLAGS) $(APPSRCS) -o bin/$(PROGN) $(LDFLAGS) -L./lib -l$(PROGN)
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $(APPSRCS) -o bin/$(PROGN) $(LDFLAGS) -L./lib -l$(PROGN)
 
 lib: lib/lib$(PROGN).so
 

From 87b2de95c94f8828b7d1f1b570d2bab55649b783 Mon Sep 17 00:00:00 2001
From: "Haelwenn (lanodan) Monnier" <contact@hacktivis.me>
Date: Sun, 9 Sep 2018 05:15:17 +0200
Subject: [PATCH 6/7] makefile: Fix issues on code review

Code review: https://github.com/gentoo/gentoo/pull/9476#discussion_r216123340
---
 makefile | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/makefile b/makefile
index bb3f368..707517d 100644
--- a/makefile
+++ b/makefile
@@ -1,12 +1,12 @@
 CXX?=g++
-CXXFLAGS?=-Wall -O2 -std=gnu++11
+CXXFLAGS?=-Wall -O2
 LIB?=hidapi
 ifeq ($(LIB),libusb)
 	CPPFLAGS=-Dlibusb
-	LDFLAGS+=-lusb-1.0
+	LIBS=-lusb-1.0
 else
 	CPPFLAGS=-Dhidapi
-	LDFLAGS+=-lhidapi-hidraw
+	LIBS=-lhidapi-hidraw
 endif
 SYSTEMDDIR?=/usr/lib/systemd
 
@@ -20,7 +20,7 @@ MAJOR=0
 MINOR=2
 MICRO=7
 
-CXXFLAGS+=-DVERSION=\"$(MAJOR).$(MINOR).$(MICRO)\"
+CXXFLAGS+=-std=gnu++11 -DVERSION=\"$(MAJOR).$(MINOR).$(MICRO)\"
 APPSRCS=src/main.cpp src/helpers/*.cpp src/helpers/*.h
 LIBSRCS=src/classes/*.cpp src/classes/*.h
 
@@ -32,19 +32,19 @@ bin: bin/$(PROGN)
 
 bin/$(PROGN): $(APPSRCS) $(LIBSRCS)
 	@mkdir -p bin
-	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $^ -o $@ $(LDFLAGS)
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $(LDFLAGS) $^ -o $@ $(LIBS)
 	
 debug: CXXFLAGS += -g -Wextra -pedantic
 debug: bin/$(PROGN)
 
 lib/lib$(PROGN).so: $(LIBSRCS)
 	@mkdir -p lib
-	$(CXX) $(CPPFLAGS) $(CXXFLAGS) -fPIC -shared -Wl,-soname,lib$(PROGN).so -o lib/lib$(PROGN).so.$(MAJOR).$(MINOR).$(MICRO) $^ $(LDFLAGS)
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $(LDFLAGS) -fPIC -shared -Wl,-soname,lib$(PROGN).so -o lib/lib$(PROGN).so.$(MAJOR).$(MINOR).$(MICRO) $^ $(LIBS)
 	@ln -sf lib$(PROGN).so.$(MAJOR).$(MINOR).$(MICRO) lib/lib$(PROGN).so
 
 bin-linked: lib/lib$(PROGN).so
 	@mkdir -p bin
-	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $(APPSRCS) -o bin/$(PROGN) $(LDFLAGS) -L./lib -l$(PROGN)
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) $(LDFLAGS) $(APPSRCS) -o bin/$(PROGN) $(LIBS) -L./lib -l$(PROGN)
 
 lib: lib/lib$(PROGN).so
 

From b707deb6d3eee419aaa0d27db7b0a538ee4d835a Mon Sep 17 00:00:00 2001
From: MatMoul <matmoul@gmail.com>
Date: Thu, 13 Sep 2018 22:07:08 +0200
Subject: [PATCH 7/7] Version 0.2.8

---
 makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/makefile b/makefile
index e76a786..3741c14 100644
--- a/makefile
+++ b/makefile
@@ -18,7 +18,7 @@ includedir?=$(PREFIX)/include
 PROGN=g810-led
 MAJOR=0
 MINOR=2
-MICRO=7
+MICRO=8
 
 CXXFLAGS+=-std=gnu++11 -DVERSION=\"$(MAJOR).$(MINOR).$(MICRO)\"
 APPSRCS=src/main.cpp src/helpers/*.cpp src/helpers/*.h