espressif
diff --git a/‎cores/esp32/esp32-hal-ledc.c
Copy file name to clipboardExpand all lines: cores/esp32/esp32-hal-ledc.c
+13-4Lines changed: 13 additions & 4 deletions b/‎cores/esp32/esp32-hal-ledc.c
Copy file name to clipboardExpand all lines: cores/esp32/esp32-hal-ledc.c
+13-4Lines changed: 13 additions & 4 deletions
diff --git a/‎docs/source/api/ledc.rst
Copy file name to clipboardExpand all lines: docs/source/api/ledc.rst
+22Lines changed: 22 additions & 0 deletions b/‎docs/source/api/ledc.rst
Copy file name to clipboardExpand all lines: docs/source/api/ledc.rst
+22Lines changed: 22 additions & 0 deletions
@@ -222,9 +222,12 @@ void analogWrite(uint8_t pin, int value) {
           log_e("No more analogWrite channels available! You can have maximum %u", LEDC_CHANNELS);
           return;
       }
+      if(ledcSetup(cnt_channel - 1, analog_frequency, analog_resolution) == 0){
+          log_e("analogWrite setup failed (freq = %u, resolution = %u). Try setting different resolution or frequency");
+          return;
+      }
+      ledcAttachPin(pin, cnt_channel - 1);
       pin_to_channel[pin] = cnt_channel--;
-      ledcSetup(cnt_channel, analog_frequency, analog_resolution);
-      ledcAttachPin(pin, cnt_channel);
     }
     ledcWrite(pin_to_channel[pin] - 1, value);
   }
@@ -237,7 +240,10 @@ int8_t analogGetChannel(uint8_t pin) {
 void analogWriteFrequency(uint32_t freq) {
     if (cnt_channel != LEDC_CHANNELS) {
         for (int channel = LEDC_CHANNELS - 1; channel >= cnt_channel; channel--) {
-            ledcChangeFrequency(channel, freq, analog_resolution);
+            if (ledcChangeFrequency(channel, freq, analog_resolution) == 0){
+                log_e("analogWrite frequency cant be set due to selected resolution! Try to adjust resolution first");
+                return;
+            }
         }
     }
     analog_frequency = freq;
@@ -250,7 +256,10 @@ void analogWriteResolution(uint8_t bits) {
     }
     if (cnt_channel != LEDC_CHANNELS) {
         for (int channel = LEDC_CHANNELS - 1; channel >= cnt_channel; channel--) {
-            ledcChangeFrequency(channel, analog_frequency, bits);
+            if (ledcChangeFrequency(channel, analog_frequency, bits) == 0){
+                log_e("analogWrite resolution cant be set due to selected frequency! Try to adjust frequency first");
+                return;
+            }
         }
     }
     analog_resolution = bits;
 
@@ -169,6 +169,28 @@ It is compatible with Arduinos analogWrite function.
 * ``value`` select the duty cycle of pwm.
   * range is from 0 (always off) to 255 (always on).
 
+analogWriteResolution
+*********************
+
+This function is used to set resolution for all analogWrite channels.
+
+.. code-block:: arduino
+
+    void analogWriteResolution(uint8_t bits);
+   
+* ``bits`` select resolution for analog channels. 
+
+analogWriteFrequency
+********************
+
+This function is used to set frequency for all analogWrite channels.
+
+.. code-block:: arduino
+
+    void analogWriteFrequency(uint32_t freq);
+
+* ``freq`` select frequency of pwm.
+
 Example Applications
 ********************
Original file line number	Diff line number	Diff line change
`@@ -222,9 +222,12 @@ void analogWrite(uint8_t pin, int value) {`
`222`	`222`	`log_e("No more analogWrite channels available! You can have maximum %u", LEDC_CHANNELS);`
`223`	`223`	`return;`
`224`	`224`	`}`
	`225`	`+ if(ledcSetup(cnt_channel - 1, analog_frequency, analog_resolution) == 0){`
	`226`	`+ log_e("analogWrite setup failed (freq = %u, resolution = %u). Try setting different resolution or frequency");`
	`227`	`+ return;`
	`228`	`+ }`
	`229`	`+ ledcAttachPin(pin, cnt_channel - 1);`
`225`	`230`	`pin_to_channel[pin] = cnt_channel--;`
`226`		`- ledcSetup(cnt_channel, analog_frequency, analog_resolution);`
`227`		`- ledcAttachPin(pin, cnt_channel);`
`228`	`231`	`}`
`229`	`232`	`ledcWrite(pin_to_channel[pin] - 1, value);`
`230`	`233`	`}`
`@@ -237,7 +240,10 @@ int8_t analogGetChannel(uint8_t pin) {`
`237`	`240`	`void analogWriteFrequency(uint32_t freq) {`
`238`	`241`	`if (cnt_channel != LEDC_CHANNELS) {`
`239`	`242`	`for (int channel = LEDC_CHANNELS - 1; channel >= cnt_channel; channel--) {`
`240`		`- ledcChangeFrequency(channel, freq, analog_resolution);`
	`243`	`+ if (ledcChangeFrequency(channel, freq, analog_resolution) == 0){`
	`244`	`+ log_e("analogWrite frequency cant be set due to selected resolution! Try to adjust resolution first");`
	`245`	`+ return;`
	`246`	`+ }`
`241`	`247`	`}`
`242`	`248`	`}`
`243`	`249`	`analog_frequency = freq;`
`@@ -250,7 +256,10 @@ void analogWriteResolution(uint8_t bits) {`
`250`	`256`	`}`
`251`	`257`	`if (cnt_channel != LEDC_CHANNELS) {`
`252`	`258`	`for (int channel = LEDC_CHANNELS - 1; channel >= cnt_channel; channel--) {`
`253`		`- ledcChangeFrequency(channel, analog_frequency, bits);`
	`259`	`+ if (ledcChangeFrequency(channel, analog_frequency, bits) == 0){`
	`260`	`+ log_e("analogWrite resolution cant be set due to selected frequency! Try to adjust frequency first");`
	`261`	`+ return;`
	`262`	`+ }`
`254`	`263`	`}`
`255`	`264`	`}`
`256`	`265`	`analog_resolution = bits;`