On 16/01/2025 15:49, Tudor Ambarus wrote: > -EXPORT_SYMBOL_GPL(of_qcom_ice_get); > > static void qcom_ice_put(const struct qcom_ice *ice) > { > diff --git a/include/soc/qcom/ice.h b/include/soc/qcom/ice.h > index d5f6a228df65..fdf1b5c21eb9 100644 > --- a/include/soc/qcom/ice.h > +++ b/include/soc/qcom/ice.h > @@ -33,7 +33,6 @@ int qcom_ice_program_key(struct qcom_ice *ice, > const u8 crypto_key[], u8 data_unit_size, > int slot); > int qcom_ice_evict_key(struct qcom_ice *ice, int slot); > -struct qcom_ice *of_qcom_ice_get(struct device *dev); > struct qcom_ice *devm_of_qcom_ice_get(struct device *dev); So this explains why you did not export the put(). Mention the reason for missing put() in the first commit. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof