Skip to content
This repository was archived by the owner on Jun 5, 2019. It is now read-only.

Add SpiDevinceInfo native layer support with reference STM32F4 implementation #177

Merged
merged 3 commits into from
Jun 17, 2015

Conversation

smaillet-ms
Copy link
Member

Fix for #143

  • updated Windows.Devices interop table
  • Added SpiBusInfo Man/Max Frequency properties as internal calls to pass on to the HAL/PAL to provide hardware specific values.
  • Implemented New CPU_SPI_xxx APIs for STM32F4. Currently hardcoded to standard values as there's not a lot of value in anything more sophisticated.
  • removed redundant code used to eliminate warnings in favor of #pragma as code is a storage and perf overhead not worth the price to silence a bening warning.

- Added SpiBusInfo Man/Max Frequency properties as internal calls to pass on to the HAL/PAL to provide hardware specific values.
- Implemented New CPU_SPI_xxx APIs for STM32F4. Currently hardcoded to standard values as there's not a lot of value in anything more sophisticated.
- removed redundant code used to eliminate warnings in favor of #pragma as code is a storage and perf overhead not worth the price to silence a bening warning.
@@ -2,15 +2,26 @@
// Copyright (c) Microsoft Corporation. All rights reserved.
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

#include "windows_devices.h"
#include "Windows_Devices.h"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Original casing matches what's in the file system. This should be reverted.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm... Why did we change that from what's generated?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a generated file. Casing matches other core libraries.

@ckarenz
Copy link

ckarenz commented Jun 17, 2015

👍

smaillet-ms added a commit that referenced this pull request Jun 17, 2015
Add SpiDevinceInfo native layer support with reference STM32F4 implementation
@smaillet-ms smaillet-ms merged commit a5aa38e into NETMF:dev Jun 17, 2015
@smaillet-ms smaillet-ms deleted the Add-SpiDevinceInfo-native branch June 17, 2015 23:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants