diff --git a/hal/src/main/java/edu/wpi/first/hal/CTREPCMJNI.java b/hal/src/main/java/edu/wpi/first/hal/CTREPCMJNI.java
index 0abe084dd9..88ba3cebdc 100644
--- a/hal/src/main/java/edu/wpi/first/hal/CTREPCMJNI.java
+++ b/hal/src/main/java/edu/wpi/first/hal/CTREPCMJNI.java
@@ -150,7 +150,7 @@ public class CTREPCMJNI extends JNIWrapper {
    * Gets a bitmask of solenoid values.
    *
    * @param handle the PCM handle
-   * @return solenoid values
+   * @return Bitmask containing the state of the solenoids. The LSB represents solenoid 0.
    * @see "HAL_GetCTREPCMSolenoids"
    */
   public static native int getSolenoids(int handle);
@@ -159,8 +159,9 @@ public class CTREPCMJNI extends JNIWrapper {
    * Sets solenoids on a pneumatics module.
    *
    * @param handle the PCM handle
-   * @param mask bitmask to set
-   * @param values solenoid values
+   * @param mask Bitmask indicating which solenoids to set. The LSB represents solenoid 0.
+   * @param values Bitmask indicating the desired states of the solenoids. The LSB represents
+   *     solenoid 0.
    * @see "HAL_SetCTREPCMSolenoids"
    */
   public static native void setSolenoids(int handle, int mask, int values);
@@ -169,7 +170,7 @@ public class CTREPCMJNI extends JNIWrapper {
    * Get a bitmask of disabled solenoids.
    *
    * @param handle the PCM handle
-   * @return bitmask of disabled solenoids
+   * @return Bitmask indicating disabled solenoids. The LSB represents solenoid 0.
    * @see "HAL_GetCTREPCMSolenoidDisabledList"
    */
   public static native int getSolenoidDisabledList(int handle);
diff --git a/hal/src/main/java/edu/wpi/first/hal/REVPHJNI.java b/hal/src/main/java/edu/wpi/first/hal/REVPHJNI.java
index 1180b9b107..a345416453 100644
--- a/hal/src/main/java/edu/wpi/first/hal/REVPHJNI.java
+++ b/hal/src/main/java/edu/wpi/first/hal/REVPHJNI.java
@@ -169,7 +169,7 @@ public class REVPHJNI extends JNIWrapper {
    * Gets a bitmask of solenoid values.
    *
    * @param handle the PH handle
-   * @return solenoid values
+   * @return Bitmask containing the state of the solenoids. The LSB represents solenoid 0.
    * @see "HAL_GetREVPHSolenoids"
    */
   public static native int getSolenoids(int handle);
@@ -178,8 +178,9 @@ public class REVPHJNI extends JNIWrapper {
    * Sets solenoids on a PH.
    *
    * @param handle the PH handle
-   * @param mask bitmask to set
-   * @param values solenoid values
+   * @param mask Bitmask indicating which solenoids to set. The LSB represents solenoid 0.
+   * @param values Bitmask indicating the desired states of the solenoids. The LSB represents
+   *     solenoid 0.
    * @see "HAL_SetREVPHSolenoids"
    */
   public static native void setSolenoids(int handle, int mask, int values);
diff --git a/hal/src/main/native/include/hal/CTREPCM.h b/hal/src/main/native/include/hal/CTREPCM.h
index 932ed3471c..fc7c46e084 100644
--- a/hal/src/main/native/include/hal/CTREPCM.h
+++ b/hal/src/main/native/include/hal/CTREPCM.h
@@ -179,7 +179,8 @@ HAL_Bool HAL_GetCTREPCMCompressorNotConnectedFault(HAL_CTREPCMHandle handle,
  *
  * @param[in] handle  the PCM handle
  * @param[out] status Error status variable. 0 on success.
- * @return solenoid values
+ * @return Bitmask containing the state of the solenoids. The LSB represents
+ * solenoid 0.
  */
 int32_t HAL_GetCTREPCMSolenoids(HAL_CTREPCMHandle handle, int32_t* status);
 
@@ -187,8 +188,10 @@ int32_t HAL_GetCTREPCMSolenoids(HAL_CTREPCMHandle handle, int32_t* status);
  * Sets solenoids on a pneumatics module.
  *
  * @param[in] handle  the PCM handle
- * @param[in] mask bitmask to set
- * @param[in] values solenoid values
+ * @param[in] mask Bitmask indicating which solenoids to set. The LSB represents
+ * solenoid 0.
+ * @param[in] values Bitmask indicating the desired states of the solenoids. The
+ * LSB represents solenoid 0.
  * @param[out] status Error status variable. 0 on success.
  */
 void HAL_SetCTREPCMSolenoids(HAL_CTREPCMHandle handle, int32_t mask,
@@ -199,7 +202,7 @@ void HAL_SetCTREPCMSolenoids(HAL_CTREPCMHandle handle, int32_t mask,
  *
  * @param[in] handle  the PCM handle
  * @param[out] status Error status variable. 0 on success.
- * @return bitmask of disabled solenoids
+ * @return Bitmask indicating disabled solenoids. The LSB represents solenoid 0.
  */
 int32_t HAL_GetCTREPCMSolenoidDisabledList(HAL_CTREPCMHandle handle,
                                            int32_t* status);
diff --git a/hal/src/main/native/include/hal/REVPH.h b/hal/src/main/native/include/hal/REVPH.h
index 754c0f23ed..545adc5789 100644
--- a/hal/src/main/native/include/hal/REVPH.h
+++ b/hal/src/main/native/include/hal/REVPH.h
@@ -311,7 +311,8 @@ void HAL_GetREVPHVersion(HAL_REVPHHandle handle, HAL_REVPHVersion* version,
  *
  * @param[in] handle  the PH handle
  * @param[out] status Error status variable. 0 on success.
- * @return solenoid values
+ * @return Bitmask containing the state of the solenoids. The LSB represents
+ * solenoid 0.
  */
 int32_t HAL_GetREVPHSolenoids(HAL_REVPHHandle handle, int32_t* status);
 
@@ -319,8 +320,10 @@ int32_t HAL_GetREVPHSolenoids(HAL_REVPHHandle handle, int32_t* status);
  * Sets solenoids on a PH.
  *
  * @param[in] handle  the PH handle
- * @param[in] mask bitmask to set
- * @param[in] values solenoid values
+ * @param[in] mask Bitmask indicating which solenoids to set. The LSB represents
+ * solenoid 0.
+ * @param[in] values Bitmask indicating the desired states of the solenoids. The
+ * LSB represents solenoid 0.
  * @param[out] status Error status variable. 0 on success.
  */
 void HAL_SetREVPHSolenoids(HAL_REVPHHandle handle, int32_t mask, int32_t values,