diff --git a/halo2_gadgets/src/ecc.rs b/halo2_gadgets/src/ecc.rs
index 572952db6..92bb97637 100644
--- a/halo2_gadgets/src/ecc.rs
+++ b/halo2_gadgets/src/ecc.rs
@@ -918,7 +918,7 @@ pub(crate) mod tests {
     }
 
     #[test]
-    fn test_against_stored_ecc_chip() {
+    fn test_ecc_chip_against_stored_circuit() {
         let circuit: MyCircuit<PallasLookupRangeCheckConfig> = MyCircuit {
             test_errors: false,
             _lookup_marker: PhantomData,
diff --git a/halo2_gadgets/src/sinsemilla/chip/hash_to_point.rs b/halo2_gadgets/src/sinsemilla/chip/hash_to_point.rs
index ab06b29d1..b32b15c98 100644
--- a/halo2_gadgets/src/sinsemilla/chip/hash_to_point.rs
+++ b/halo2_gadgets/src/sinsemilla/chip/hash_to_point.rs
@@ -17,7 +17,7 @@ use pasta_curves::{arithmetic::CurveAffine, pallas};
 
 use std::ops::Deref;
 
-/// Define an enum that can hold either a public or a private ECC Point
+/// `EccPointQ` can hold either a public or a private ECC Point
 #[derive(Debug, Clone)]
 #[allow(dead_code)]
 pub enum EccPointQ<'a> {
diff --git a/halo2_gadgets/src/utilities/lookup_range_check.rs b/halo2_gadgets/src/utilities/lookup_range_check.rs
index 0334ccbab..f17b972bf 100644
--- a/halo2_gadgets/src/utilities/lookup_range_check.rs
+++ b/halo2_gadgets/src/utilities/lookup_range_check.rs
@@ -58,7 +58,7 @@ impl<F: PrimeFieldBits> RangeConstrained<F, AssignedCell<F, F>> {
     }
 }
 
-/// Configuration that provides methods for a lookup range check.
+/// Configuration that provides methods for a 10-bit lookup range check.
 #[derive(Eq, PartialEq, Debug, Clone, Copy)]
 pub struct LookupRangeCheckConfig<F: PrimeFieldBits, const K: usize> {
     q_lookup: Selector,