From 7928daf147683df27a8e5010cb7572c08798128b Mon Sep 17 00:00:00 2001
From: Nolen Johnson <johnsonnolen@gmail.com>
Date: Mon, 10 May 2021 13:59:50 -0400
Subject: [PATCH] kane: Cleanup

Change-Id: I9762c939670b4cce5903f9dc77bca0e58dbfce81
---
 BoardConfig.mk        | 18 +++++++++---------
 device.mk             |  5 +++--
 lineage_kane.mk       | 12 ++++++------
 proprietary-files.txt | 12 ++++++------
 vendor.prop           |  5 ++++-
 5 files changed, 28 insertions(+), 24 deletions(-)

diff --git a/BoardConfig.mk b/BoardConfig.mk
index ae8da0c..c6b34cf 100644
--- a/BoardConfig.mk
+++ b/BoardConfig.mk
@@ -1,5 +1,5 @@
 #
-# Copyright (C) 2020 The LineageOS Project
+# Copyright (C) 2020-2021 The LineageOS Project
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -13,19 +13,19 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# inherit from common
--include device/motorola/exynos9610-common/BoardConfigCommon.mk
+## Inherit from the common tree
+include device/motorola/exynos9610-common/BoardConfigCommon.mk
+
+## Inherit from the proprietary configuration
+include vendor/motorola/kane/BoardConfigVendor.mk
 
 DEVICE_PATH := device/motorola/kane
 
-# Assert
+## Assert
 TARGET_OTA_ASSERT_DEVICE := kane,one_vision
 
-# Kernel
+## Kernel
 TARGET_KERNEL_CONFIG := kane_defconfig
 
-# Properties
+## Properties
 TARGET_VENDOR_PROP += $(DEVICE_PATH)/vendor.prop
-
-# inherit from the proprietary version
-include vendor/motorola/kane/BoardConfigVendor.mk
diff --git a/device.mk b/device.mk
index da70092..745591f 100644
--- a/device.mk
+++ b/device.mk
@@ -1,5 +1,5 @@
 #
-# Copyright (C) 2020 The LineageOS Project
+# Copyright (C) 2020-2021 The LineageOS Project
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -14,7 +14,8 @@
 # limitations under the License.
 #
 
-# Inherit from common
+# Inherit from the common tree
 $(call inherit-product, device/motorola/exynos9610-common/common.mk)
 
+# Inherit proprietary files
 $(call inherit-product, vendor/motorola/kane/kane-vendor.mk)
diff --git a/lineage_kane.mk b/lineage_kane.mk
index 6645b20..207e4f9 100644
--- a/lineage_kane.mk
+++ b/lineage_kane.mk
@@ -1,5 +1,5 @@
 #
-# Copyright (C) 2020 The LineageOS Project
+# Copyright (C) 2020-2021 The LineageOS Project
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -14,18 +14,18 @@
 # limitations under the License.
 #
 
-# Inherit from those products. Most specific first.
+## Inherit from generic products, most specific first
 $(call inherit-product, $(SRC_TARGET_DIR)/product/core_64_bit.mk)
 $(call inherit-product, $(SRC_TARGET_DIR)/product/full_base.mk)
 
-# Inherit from kane device
+## Inherit from kane device
 $(call inherit-product, device/motorola/kane/device.mk)
 $(call inherit-product, $(SRC_TARGET_DIR)/product/full_base_telephony.mk)
 
-# Inherit some common Lineage stuff.
+## Inherit some common Lineage stuff
 $(call inherit-product, vendor/lineage/config/common_full_phone.mk)
 
-# Device identifier. This must come after all inclusions
+## Device identifier, this must come after all inclusions
 PRODUCT_DEVICE := kane
 PRODUCT_NAME := lineage_kane
 PRODUCT_BRAND := Motorola
@@ -34,7 +34,7 @@ PRODUCT_MANUFACTURER := Motorola
 
 PRODUCT_GMS_CLIENTID_BASE := android-motorola
 
-# Use the latest approved GMS identifiers
+## Use the latest CTS approved build identifiers
 PRODUCT_BUILD_PROP_OVERRIDES += \
     PRODUCT_NAME=kane_sprout \
     PRIVATE_BUILD_DESC="10/QSAS30.62-33-4/2d332:user/release-keys"
diff --git a/proprietary-files.txt b/proprietary-files.txt
index ae3ecaf..26fa212 100644
--- a/proprietary-files.txt
+++ b/proprietary-files.txt
@@ -1,12 +1,12 @@
-# Device proprietary files - from motorola/kane_retail/kane_sprout:10/QSAS30.62-33-12/e6aee, unless pinned
+## Device proprietary files - from motorola/kane_retail/kane_sprout:10/QSAS30.62-33-12/e6aee, unless pinned
 
-# camera
+## Camera
 vendor/lib/hw/camera.exynos9610.so
 vendor/lib/libexynoscamera3.so
 vendor/lib64/hw/camera.exynos9610.so
 vendor/lib64/libexynoscamera3.so
 
-# firmware
+## Firmware
 vendor/firmware/fimc_is_lib.bin
 vendor/firmware/fimc_is_rta.bin
 vendor/firmware/himax-hix83112a-hx83112a-0C00-208-kane.bin
@@ -27,16 +27,16 @@ vendor/firmware/setfile_5e9.bin
 vendor/firmware/setfile_6b2.bin
 vendor/firmware/setfile_gm1sp.bin
 
-# NFC
+## NFC
 vendor/etc/sec_s3nrn81_rfreg.bin
 
-# sensors
+## Sensors
 vendor/lib/hw/sensors.kane_sprout.so
 vendor/lib/sensors.chub.so
 vendor/lib64/hw/sensors.kane_sprout.so
 vendor/lib64/sensors.chub.so
 
-# ril
+## RIL
 vendor/lib/libril_sitril.so
 vendor/lib/libsitril.so
 vendor/lib64/libril_sitril.so
diff --git a/vendor.prop b/vendor.prop
index 9dba3fb..7ed0cd1 100644
--- a/vendor.prop
+++ b/vendor.prop
@@ -1,2 +1,5 @@
-ro.hardware.sensors=kane_sprout
+## IMEI
 ro.radio.imei.sv=18
+
+## Sensors
+ro.hardware.sensors=kane_sprout
-- 
2.20.1