Merge branch 'bugfix/log_macro_statements' into 'master'

log: Make ESP_LOGx macros into single statements

See merge request idf/esp-idf!2275

Merge branch 'bugfix/ci_retry_build_example' into 'master'

ci: Try to build examples in second time in case of error

See merge request idf/esp-idf!2302

log: Make ESP_LOGx macros into single statements

Requires a semicolon after any use (previously this was optional due to trailing } after expansion)

Closes https://github.com/espressif/esp-idf/issues/1757

Merge branch 'feature/btdm_set_bredr_txpower' into 'master'

component/bt : add BR/EDR TX power set

See merge request idf/esp-idf!2267

component/bt : add BR/EDR TX power set

Merge branch 'bugfix/btdm_fix_gatt_cache_error' into 'master'

Component/bt: fix gattc some errors

See merge request idf/esp-idf!2104

Merge branch 'feature/btdm_cherry-pick_ble_hid_device' into 'master'

Feature/btdm cherry pick ble hid device

See merge request idf/esp-idf!1701

Merge branch 'doc/openocd_release_update' into 'master'

docs: update download links for OpenOCD

See merge request idf/esp-idf!2259

CI: Try to build examples in second time in case of error

    It will help to work around the weird bug during buillding a batch of examples

Merge branch 'bugfix/fatfs_readwrite_test_failure' into 'master'

fatfs/test: increase stack size in read-write unit test

See merge request idf/esp-idf!2256

Merge branch 'feature/nvs_coverage' into 'master'

NVS: add power off recovery and coverage test into CI

See merge request idf/esp-idf!2246

Merge branch 'bugfix/sdspi_wp_cd_pins' into 'master'

sdmmc, sdspi: fix handling of CD and WP

See merge request idf/esp-idf!2285

sdmmc: add tests for CD and WP pins for SD and SPI mode

sdmmc: don’t print sdio_reset warning when CD is idle

sdmmc host: add handling of CD and WP pins

Previous version of the code only connected CD and WP to the
peripheral, in fact the hardware does not use the values of these
signals automatically. This adds code to read CD and WP values when
command is executed and return errors if card is not present, or
write command is executed when WP signal is active.

sdmmc host: when parameter validation fails, exit cleanly

This fixes lock-up which happened when sending a command, if the
previous command has failed.

Merge branch 'bugfix/minor_fix_for_wifi_tx' into 'master'

esp32: minor fix for WiFi TX

See merge request idf/esp-idf!2281

Merge branch 'bugfix/esp_tls_error_logging' into 'master'

esp-tls: Expand error logging

See merge request idf/esp-idf!2284

more

Merge branch 'bugfix/esp_tls_memory_leak' into 'master'

esp-tls: fix memory leak when using CA certification validation

See merge request idf/esp-idf!2283

Merge branch 'bugfix/authmode_change_event_peg_core_in_wep_mode' into 'master'

bugfix/wrong_authmode_change_event_is_reported_in_wep_mode

See merge request idf/esp-idf!2282

Component/bt: fix gatt cache error
- fix sev == NULL error
- fix start_handle == end_handle

sdspi: fix CD and WP incorrectly configured as outputs

sdmmc, sdspi: fix initializers to be C++-compatible, add test

Closes https://github.com/espressif/esp-idf/issues/1861
Closes https://github.com/espressif/arduino-esp32/issues/1312

Bug fix of authmode change event peg core in wep mode

auth mode change event will report continous in wep mode due to wrong original auth record

close github issue https://github.com/espressif/esp-idf/issues/1804

bugfix/sdspi_host.c: wrong CD/WP pin configuration

Looks like the configuration of CP and WP pins are wrong, since the check for `gpio_cd` and `gpio_wp` in `slot_config` seems to be swapped.

Merge branch 'feature/disable_ble_service_change_pr1673' into 'master'

Add option to ena/dis BLE service change registration

See merge request idf/esp-idf!2210

Merge branch 'bugfix/typo_in_aws_iot_config' into 'master'

Typo in aws_iot_config.h

See merge request idf/esp-idf!2280

Merge branch 'bugfix/tw18453_fix_ap_eapol_retry_bug' into 'master'

esp32: fix soft-AP eapol retry issue

See merge request idf/esp-idf!2264

esp32: fix the bug that soft-AP fails to retry eapol

Fix the issue that soft-AP fails to retry eapol frame when multiple
stations connects it at the same time.

More

esp-tls: fix memory leak when using CA certification validation

Merge branch 'feature/idf_size_report_symbols' into 'master'

Feature/idf-size: report per-archive symbols and their sizes

See merge request idf/esp-idf!1956

Merge branch 'feature/esp-tls' into 'master'

esp-tls: Fix HTTP2 failure

See merge request idf/esp-idf!2277

Typo in aws_iot_config.h

-breaks compilation when 'Override Shadow RX buffer size' is enabled via
menuconfig

Merges https://github.com/espressif/esp-idf/pull/1869

example/hid: Added the readme file.

esp-tls: Fix HTTP2 failure

Closes https://github.com/espressif/esp-idf/issues/1874

Merge branch 'bugfix/add_NULL_value_check_bta_gattc_get_services' into 'master'

components/bt: Add NULL value check in bta_gattc_get_service_with_uuid()

See merge request idf/esp-idf!2212

Merge branch 'bugfix/bt_correct_inquiry_mode_name' into 'master'

components/bt: Correct the enquiry mode name

See merge request idf/esp-idf!2217

Merge branch 'feature/btdm_gattc_cache' into 'master'

Feature/btdm gattc cache

See merge request idf/esp-idf!1083

Merge branch 'feature/amazon_freertos_compat_v2' into 'master'

Changes for Amazon Freertos compatibility

See merge request idf/esp-idf!2123

component/bt: For unity, Change all the associat & associa to abbreviated form(assoc).

Merge branch 'revert-88d0d6ff' into 'master'

Revert "Merge branch 'feature/vfs_select' into 'master'"

See merge request idf/esp-idf!2273

Revert "Merge branch 'feature/vfs_select' into 'master'"

This reverts merge request !2074

component/bt: Change the cache 'associat' name for unify.

esp32: fix few components with dependency on networking stack

* Move smartconfig to its component directory, it should be possible to override
  this as whole component
* Fix few header includes related to lwIP networking stack

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

driver/spi: add _ISR counterparts if invoked from interrupt for critical section

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

freertos: portCLEAN_UP_TCB should be performed before freeing tcb memory

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

pthread: implement local storage using pvTaskGetThreadLocalStoragePointer

If static task cleanup option is enabled, then before invoking application
defined `vPortCleanUpTCB` hook, pthread specific cleanup is performed.

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

esp_timer: use _ISR version of port{ENTER,EXIT}_CRITICAL from ISR

newlib: define _REENT_INIT_PTR correctly for ESP_PLATFORM

multi_heap: use portENTER/EXIT_CRITICAL instead of taskENTER/EXIT_CRITICAL

crosscore_int: use _ISR version of portENTER/EXIT_CRITICAL in ISR

Merge branch 'feature/mesh' into 'master'

mesh: update mesh libs

See merge request idf/esp-idf!2244

nvs: run all test by default, exclude long tests by tag

Reduces the chance of new tests being skipped in CI

nvs: print progress in nvs API tests, reduce number of iterations

nvs: add long ci test for power off recovery and coverage

This test can be triggered manually:
`bot test name: test_nvs_coverage`

ci: support jobs which don’t run unless triggered by name

By default, any job will run unless a filter is given, in that case
the filter will determine if the job should run or not. Some jobs do
not need to be run by default, and should only be triggered using the
bot. For such jobs, BOT_NEEDS_TRIGGER_BY_NAME can added to
environment variables.

Merge branch 'feature/vfs_select' into 'master'

Allow VFS file descriptors in select()

See merge request idf/esp-idf!2074

Merge branch 'bugfix/fix_gpio_install_isr_service_check_value' into 'master'

driver(gpio): modify return value when gpio ISR service already exists.

See merge request idf/esp-idf!2125

Merge branch 'feature/bootloader_refactoring_common' into 'master'

bootloader: Refactoring

See merge request idf/esp-idf!2251

Merge branch 'feature/remove_make_dependency_on_git' into 'master'

make: remove build system dependency on `git` using some hooks

See merge request idf/esp-idf!2211

Merge branch 'bugfix/fix_github_reported_1841_and_1845' into 'master'

Fix few minor github reported issues

See merge request idf/esp-idf!2252

component/bt: Change the btc gattc callback function with the compile error.

1. Change all the gattc API && bta gattc layer.
2. Debug the code and change the btc_ble_gattc_get_db method.
3. Change the gatt read API interface.
4. Reconstruction the BTA_gattc_cache code.
5. Change back the bluedroid_get_status to marco.
6. Change the gattc docs format.
7. Change the docs format.
8. fix the read char value bug.
9. change the gattc_get_attr_count method.
10. Change back the bta_gattc write ccc code.
11. Change the gattc api docs format
12. Change the gattc API docs.
13. Change the prepare write descriptor method to avoid the exection.
14. modify gatt clinet demo with new API
15. Change the p_src_data->read.p_value to avoid exection.
16. Change the bugfix of gattc unreg for the notify.

    component/bt: Added the serch service res start_handle & end_handle to the result.

component/bt: Added the bta_gattc_cache_write when gatt discovery complete.

component/bt: Added the bta_gattc_cache_write declaration.

component/bt: Added the comments for esp_ble_gattc_cache_refresh API.

component/bt: Change the spelling errors & some comment error.

component/bt: fix bug of get gattc cache address list error.

1. Change the esp_bluedroid_get_status to macro;
2. added the malloc & free for the get_addr_list

component/bt: Added the addr_info->ass_addr == NULL Judgment to prevent crashes in the bta_gattc_co_cache_find_src_addr function.

component/bt: Fixed following gattc cache bugs

1. gattc can't refresh the gattc cache in the gatt discover state;
2. remove the nvs_get_blob in the cache address init function;
3. added the nvs_set_blob return value judgment in the cache address save function;
4. added the list_new when ass_address is NULL;
5. Change the ass_address list remove method to fix the ass address can't remove bug.

example/hid: Added the BLE hid device profile.

demo/hid: Change the license validity period & some unreasonable LOG print.

example/hid: Added the Vendor Report output support.

esp-tls: Log errors for all "connection failed" code paths

esp-tls: Allow esp_tls_conn_delete() to be called with a NULL pointer

docs: add note regarding ESP-IDF version

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

docs: update download links for OpenOCD

Point to the latest release on Github, which can now be produced by CI.

Ref. https://github.com/espressif/openocd-esp32/issues/30
Ref. https://github.com/espressif/openocd-esp32/issues/18

fatfs/test: increase stack size in read-write unit test

2048 byte stack was not sufficient when some error was logged by SD
card driver, leading to stack overflows.

Merge branch 'bugfix/heap_tracing_newlib_allocations' into 'master'

heap tracing: Fix tracing of newlib internal allocations

See merge request idf/esp-idf!1992

Merge branch 'feature/esp-tls' into 'master'

esp-tls: Fix connection failure

See merge request idf/esp-idf!2241

Merge branch 'bugfix/nvs_failed_crc_assert' into 'master'

nvs: don’t expect items with bad CRC to be in cache

See merge request idf/esp-idf!2239

Merge branch 'bugfix/nvs_power_off_recovery' into 'master'

nvs: Fix recovery after power-off during erase operation

See merge request idf/esp-idf!2238

mesh: update mesh libs

1. fix issues on sending beacon caused by too much tx retries on other packets.
2. modify not to scan if rc exists when connect.
3. modify scan dwell time to default 120ms fo root.

bootloader: Refactoring

Need to make the bootloader modular so that users can redefine its functional part.

- refactoring and moving functions to the bootloader_support component
- Changed function to `void` bootloader_utility_load_image(...);

TW19596

Allow VFS file descriptors in select()

nvs: Fix recovery after power-off during erase operation

Current code for recovery after power-off do not clean-up partially
erased items for FULL pages. If the erasure was part of modification
operation, this gets luckily cleaned-up because of duplicate detection
logic. For erase-only operation, the problem still exists. This patch
adds the recovery for FULL pages also.

Closes TW<20284>

Merge branch 'bugfix/build_v0_output' into 'master'

build: remove some debug lines printed when V=0

See merge request idf/esp-idf!2237

Merge branch 'feature/btdm_get_and_set_cod' into 'master'

component/bt: Add API: get cod & set cod

See merge request idf/esp-idf!2127

Merge branch 'bugfix/btdm_a2dp_source_cleanup_bug' into 'master'

component/bt: Fix bug of a2dp source cleanup bug when connected

See merge request idf/esp-idf!2042

Merge branch 'bugfix/btdm_add_assert_when_no_device_name' into 'master'

component/bt: Fix exception when there is no device name

See merge request idf/esp-idf!2184

bt: fix incorrect comments for error codes

Closes https://github.com/espressif/esp-idf/issues/1845

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

rtc_module: fix build failure if assertions are disabled

Closes https://github.com/espressif/esp-idf/issues/1841

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

build: remove some debug lines printed when V=0

With V=0, build process would print “including .../Makefile.projbuild" lines, causing problems for print_flash_cmd target.
The issue was due to the way macro expansion works in make. To delay evaluation of info function until the execution of expanded block, two dollar signs are required.
Test for print_flash_cmd target added.

Merge branch 'bugfix/nvs_unit_test_state' into 'master'

nvs: fix nvs_get_stats unit test

See merge request idf/esp-idf!2250

nvs: fix nvs_get_stats unit test

Changed check for the returned parameter from ESP_ERR_NVS_PART_NOT_FOUND to ESP_ERR_NVS_NOT_INITIALIZED.

Merge branch 'test/fix_ci_test_fail_in_jap_case' into 'master'

test: fix CI fail when join AP

See merge request idf/esp-idf!2245

Merge branch 'chore/issue_template' into 'master'

add github issue template for users to report issues easily.

See merge request idf/esp-idf!2148

nvs: don’t expect items with bad CRC to be in cache

When erasing a variable length item with an incorrect CRC32, the span
value of the item can not be trusted, so the item will be erased with
span = 1. Subsequent entries represent the data of the variable
length item, and these will be treated as separate items. For each
entry CRC32 is checked, the check most likely fails (because the
entry contains arbitrary data, and not a proper NVS item), and the
entry is erased. Erase function assumed that every item should be
present in cache, but it is not the case for the entries which are
just parts of item’s payload. This change allows for the item to be
not found in the hashlist, if the CRC32 check fails.

add github issue template for users to report issues easily

Merge branch 'bugfix/nvs_stat_test' into 'master'

NVS: fix nvs_get_stats unit test

See merge request idf/esp-idf!2248

Merge branch 'bugfix/spi_miso_on_input_only_pins' into 'master'

fix(spi): allow to use MISO on GPIO34-39.

See merge request idf/esp-idf!2136

nvs: make nvs_flash_deinit useable from unit tests

nvs: deinit NVS in nvs_stats unit test

This fixes the failure when this test runs after another test which
leaves NVS initialized.

Merge branch 'bugfix/nvs_page_selection' into 'master'

nvs: Fix page selection algo to consider free entry counts as well

See merge request idf/esp-idf!2240

Merge branch 'feature/nvs_get_count_used_entries' into 'master'

NVS: add API to get used entries count

See merge request idf/esp-idf!1982

test: fix join AP fail during AP power on

Merge branch 'feature/uart_add_api_get_cmd_position' into 'master'

driver(uart): Add API to get the position of cmd_char

See merge request idf/esp-idf!2177

tools/ci: add test case for build without dependency on `git`

Signed-off-by: Mahavir Jain <mahavir@espressif.com>

make: remove build system dependency on `git` using some hooks

Currently for checking IDF version and submodules existence,
build system uses `git` commands. But, it could be possible use-case
where `git` is not installed (assuming IDF is flattened in source format)
on system and build happens without any warnings.

Signed-off-by: Mahavir Jain <mahavir@espressif.com>