ggml-qnn: refine code

ggml/src/ggml-qnn/ggml-qnn-impl.h

@@ -373,9 +373,9 @@ class qnn_interface {
     }
 
 private:
-    const QnnInterface_t *_qnn_interface = nullptr;
+    const QnnInterface_t * _qnn_interface           = nullptr;
 
-    const QnnSystemInterface_t *_qnn_sys_interface = nullptr;
+    const QnnSystemInterface_t * _qnn_sys_interface = nullptr;
 };
 
 class qnn_instance {
@@ -582,7 +582,6 @@ const char   * ggmlqnn_get_error_string(Qnn_ErrorHandle_t qnn_error_code);
 Qnn_DataType_t ggmlqnn_datatype_from_ggml_datatype(enum ggml_type ggmltype);
 void         * ggmlqnn_type_trait(ggml_backend_qnn_context * ctx, ggml_tensor * op);
 void           ggmlqnn_get_graphkey_from_op(const ggml_tensor * op, std::string & output);
-bool           ggmlqnn_is_valid_params(ggml_backend_qnn_context * ctx, const ggml_tensor * src0, const ggml_tensor * src1, ggml_tensor * dst);
 uint8_t      * ggmlqnn_create_rpc_buffer(qnn_instance * instance, const ggml_tensor * ggml_tensor, Qnn_Tensor_t * qnn_tensor, bool b_copydata);
 void           ggmlqnn_print_tensors_info(const char * func_name, ggml_backend_qnn_context * ctx, const ggml_tensor * src0, const ggml_tensor * src1, ggml_tensor * dst);
 

ggml/src/ggml-qnn/ggml-qnn-ops.cpp

@@ -23,6 +23,22 @@
 #include "ggml-common.h"
 #include "ggml-qnn-ops.h"
 
+static inline bool ggmlqnn_is_valid_params(ggml_backend_qnn_context * ctx, const ggml_tensor * src0,
+                             const ggml_tensor * src1, ggml_tensor * dst) {
+    if ((nullptr == ctx) || (nullptr == src0) || (nullptr == src1) || (nullptr == dst)) {
+        GGMLQNN_LOG_WARN("invalid params\n");
+        return false;
+    }
+
+    qnn_instance * instance = ctx->instance;
+    if (nullptr == instance) {
+        GGMLQNN_LOG_WARN("invalid params\n");
+        return false;
+    }
+
+    return true;
+}
+
 #define GGMLQNN_CHECK_PARAMS(ctx, src0, src1, dst)                          \
     do {                                                                    \
         if (!ggmlqnn_is_valid_params((ctx), (src0), (src1), (dst))) {       \
@@ -491,11 +507,10 @@ void ggml_qnn_mul_mat(ggml_backend_qnn_context * ctx, ggml_tensor * op) {
     QNN_VER_PTR(*p_tensor2)->dimensions = tensor_2_dimensions;
     op_perf.info();
 }
+
 void ggml_qnn_repeat(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
 }
 
-void ggml_qnn_add(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
-}
 
 void ggml_qnn_div(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
 }
@@ -539,12 +554,6 @@ void ggml_qnn_upsample_nearest2d(ggml_backend_qnn_context * ctx, ggml_tensor * d
 void ggml_qnn_pad(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
 }
 
-static void ggml_qnn_avg_pool2d(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
-}
-
-static void ggml_qnn_max_pool2d(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
-}
-
 void ggml_qnn_pool2d(ggml_backend_qnn_context * ctx, ggml_tensor * dst) {
 }
 

ggml/src/ggml-qnn/ggml-qnn-ops.h

@@ -24,8 +24,8 @@
 #include "ggml-qnn-impl.h"
 void ggml_qnn_general_node(ggml_backend_qnn_context * ctx, ggml_tensor * dst);
 void ggml_qnn_mul_mat(ggml_backend_qnn_context * ctx, ggml_tensor * dst);
+
 void ggml_qnn_repeat(ggml_backend_qnn_context * ctx, ggml_tensor * dst);
-void ggml_qnn_add(ggml_backend_qnn_context * ctx, ggml_tensor * dst);
 void ggml_qnn_div(ggml_backend_qnn_context * ctx, ggml_tensor * dst);
 void ggml_qnn_leaky_relu(ggml_backend_qnn_context * ctx, ggml_tensor * dst);
 void ggml_qnn_concat(ggml_backend_qnn_context * ctx, ggml_tensor * dst);

ggml/src/ggml-qnn/ggml-qnn.cpp

@@ -1139,22 +1139,6 @@ void ggmlqnn_get_graphkey_from_op(const ggml_tensor * op, std::string & output)
     }
 }
 
-bool ggmlqnn_is_valid_params(ggml_backend_qnn_context * ctx, const ggml_tensor * src0,
-                             const ggml_tensor * src1, ggml_tensor * dst) {
-    if ((nullptr == ctx) || (nullptr == src0) || (nullptr == src1) || (nullptr == dst)) {
-        GGMLQNN_LOG_WARN("invalid params\n");
-        return false;
-    }
-
-    qnn_instance * instance = ctx->instance;
-    if (nullptr == instance) {
-        GGMLQNN_LOG_WARN("invalid params\n");
-        return false;
-    }
-
-    return true;
-}
-
 template<typename Fn>
 Fn load_qnn_functionpointers(void * handle, const char * function_name) {
 #if defined(__ANDROID__) || defined(__linux__)