summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--.clang-format7
-rw-r--r--ape/CMakeLists.txt2
-rw-r--r--ape/main.c4
-rw-r--r--ape/rmu.c3
-rw-r--r--ape/vectors.c9
-rw-r--r--cmake/clang-format.cmake79
-rw-r--r--libs/APE/CMakeLists.txt2
-rw-r--r--libs/APE/ape.c16
-rw-r--r--libs/MII/CMakeLists.txt2
-rw-r--r--libs/MII/mii.c21
-rw-r--r--libs/NVRam/CMakeLists.txt2
-rw-r--r--libs/NVRam/nvm.c3
12 files changed, 113 insertions, 37 deletions
diff --git a/.clang-format b/.clang-format
index 3d38985..65e1d76 100644
--- a/.clang-format
+++ b/.clang-format
@@ -43,7 +43,7 @@ BreakConstructorInitializersBeforeComma: false
BreakConstructorInitializers: BeforeColon
BreakAfterJavaFieldAnnotations: false
BreakStringLiterals: true
-ColumnLimit: 80
+ColumnLimit: 160
CommentPragmas: '^ IWYU pragma:'
CompactNamespaces: false
ConstructorInitializerAllOnOneLineOrOnePerLine: false
@@ -67,7 +67,7 @@ IncludeCategories:
- Regex: '.*'
Priority: 1
IncludeIsMainRegex: '(Test)?$'
-IndentCaseLabels: false
+IndentCaseLabels: true
IndentPPDirectives: None
IndentWidth: 4
IndentWrappedFunctionNames: false
@@ -110,9 +110,6 @@ SpacesInCStyleCastParentheses: false
SpacesInParentheses: false
SpacesInSquareBrackets: false
Standard: Cpp11
-StatementMacros:
- - Q_UNUSED
- - QT_REQUIRE_VERSION
TabWidth: 4
UseTab: Never
diff --git a/ape/CMakeLists.txt b/ape/CMakeLists.txt
index 0159e1d..732a8d1 100644
--- a/ape/CMakeLists.txt
+++ b/ape/CMakeLists.txt
@@ -58,6 +58,8 @@ target_link_libraries(${PROJECT_NAME} NVRam-arm MII-arm APE-arm Network-arm NCSI
target_link_libraries(${PROJECT_NAME} bcm5719-arm)
target_compile_options(${PROJECT_NAME} PRIVATE -nodefaultlibs)
+format_target_sources(${PROJECT_NAME})
+
# Simulator add_executable
# simulator_add_executable(sim-${PROJECT_NAME}
# init_hw.c
diff --git a/ape/main.c b/ape/main.c
index 4dfd897..34bdd81 100644
--- a/ape/main.c
+++ b/ape/main.c
@@ -120,10 +120,10 @@ void handleBMCPacket(void)
{
// stat.print();
int32_t words = DIVIDE_RND_UP(bytes, sizeof(uint32_t));
- if(words > ARRAY_ELEMENTS(buffer))
+ if (words > ARRAY_ELEMENTS(buffer))
{
// This should never happen...
- while(words--)
+ while (words--)
{
// Read out the packet, but drop it.
uint32_t word = APE_PERI.BmcToNcReadBuffer.r32;
diff --git a/ape/rmu.c b/ape/rmu.c
index 9407b73..4284b8c 100644
--- a/ape/rmu.c
+++ b/ape/rmu.c
@@ -101,8 +101,7 @@ void initRMU(void)
// START, and setting unknown bit 26 to 1.
RegAPE_PERIArbControl_t arbControl;
arbControl.r32 = (1 << 26);
- arbControl.bits.PackageID =
- 0; /* TODO: allow to be configured as per NC-SI spec. */
+ arbControl.bits.PackageID = 0; /* TODO: allow to be configured as per NC-SI spec. */
arbControl.bits.Start = 1;
arbControl.bits.TKNREL = 0x14;
APE_PERI.ArbControl = arbControl;
diff --git a/ape/vectors.c b/ape/vectors.c
index ff296ab..2eabe34 100644
--- a/ape/vectors.c
+++ b/ape/vectors.c
@@ -48,10 +48,11 @@
typedef void (*vector_t)(void) __attribute__((interrupt));
// Vector table
-typedef struct {
- uint32_t* sp;
- vector_t* __start;
- vector_t* vectors[46];
+typedef struct
+{
+ uint32_t *sp;
+ vector_t *__start;
+ vector_t *vectors[46];
} vector_table_t;
extern uint32_t _estack;
diff --git a/cmake/clang-format.cmake b/cmake/clang-format.cmake
new file mode 100644
index 0000000..d92b3fd
--- /dev/null
+++ b/cmake/clang-format.cmake
@@ -0,0 +1,79 @@
+################################################################################
+###
+### @file clang-format.cmake
+###
+### @project
+###
+### @brief clang-format cmake support
+###
+################################################################################
+###
+################################################################################
+###
+### @copyright Copyright (c) 2018, Evan Lojewski
+### @cond
+###
+### All rights reserved.
+###
+### Redistribution and use in source and binary forms, with or without
+### modification, are permitted provided that the following conditions are met:
+### 1. Redistributions of source code must retain the above copyright notice,
+### this list of conditions and the following disclaimer.
+### 2. Redistributions in binary form must reproduce the above copyright notice,
+### this list of conditions and the following disclaimer in the documentation
+### and/or other materials provided with the distribution.
+### 3. Neither the name of the copyright holder nor the
+### names of its contributors may be used to endorse or promote products
+### derived from this software without specific prior written permission.
+###
+################################################################################
+###
+### THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+### AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+### IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+### ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
+### LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+### CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+### SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+### INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+### CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+### ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+### POSSIBILITY OF SUCH DAMAGE.
+### @endcond
+################################################################################
+
+if(CMAKE_SCRIPT_MODE_FILE)
+ SET(ARGUMENTS )
+ SET(CLANG_FORMAT ${CMAKE_ARGV3})
+ SET(FILES ${CMAKE_ARGV4})
+
+ FOREACH(FILE ${FILES})
+ MESSAGE("Formatting ${FILE}")
+ EXECUTE_PROCESS(COMMAND ${CLANG_FORMAT} -style=file -i "${FILE}")
+ LIST(APPEND ARGUMENTS ${FILE})
+ ENDFOREACH()
+else()
+
+ SET(CLANG_FORMAT ${COMPILER_BASE}/bin/clang-format)
+ add_custom_target(clang-format cmake -P ${CMAKE_CURRENT_LIST_FILE}
+ ${CLANG_FORMAT} $<TARGET_PROPERTY:clang-format,FORMAT_SOURCES>
+ WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
+ VERBATIM)
+
+ function(format_sources)
+ MESSAGE("Formatting sources ${ARGN}")
+
+ set_property(TARGET clang-format APPEND PROPERTY FORMAT_SOURCES ${ARGN})
+ endfunction(format_sources)
+
+ function(format_target_sources target)
+ set(paths )
+ get_target_property(sources ${target} SOURCES)
+ foreach(source ${sources})
+ get_source_file_property(path ${source} LOCATION)
+ LIST(APPEND paths ${path})
+ endforeach()
+
+ format_sources(${paths})
+ endfunction(format_target_sources)
+endif() \ No newline at end of file
diff --git a/libs/APE/CMakeLists.txt b/libs/APE/CMakeLists.txt
index 6b8fa9e..06e42a4 100644
--- a/libs/APE/CMakeLists.txt
+++ b/libs/APE/CMakeLists.txt
@@ -60,3 +60,5 @@ target_include_directories(${PROJECT_NAME}-mips PUBLIC include)
arm_add_library(${PROJECT_NAME}-arm STATIC ape.c)
target_include_directories(${PROJECT_NAME}-arm PUBLIC ../../include)
target_include_directories(${PROJECT_NAME}-arm PUBLIC include)
+
+format_target_sources(${PROJECT_NAME})
diff --git a/libs/APE/ape.c b/libs/APE/ape.c
index e7f0ca8..28230af 100644
--- a/libs/APE/ape.c
+++ b/libs/APE/ape.c
@@ -122,12 +122,12 @@ void APE_releaseAllLocks(void)
lock_release.r32 = 0;
lock_release.bits.Bootcode = 1;
- APE_PERI.PerLockGrantPhy0.r32 = lock_release.r32;
- APE_PERI.PerLockGrantGrc.r32 = lock_release.r32;
- APE_PERI.PerLockGrantPhy1.r32 = lock_release.r32;
- APE_PERI.PerLockGrantPhy2.r32 = lock_release.r32;
- APE_PERI.PerLockGrantMem.r32 = lock_release.r32;
- APE_PERI.PerLockGrantPhy3.r32 = lock_release.r32;
- APE_PERI.PerLockGrantPort6.r32 = lock_release.r32;
- APE_PERI.PerLockGrantGpio.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantPhy0.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantGrc.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantPhy1.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantPhy2.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantMem.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantPhy3.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantPort6.r32 = lock_release.r32;
+ APE_PERI.PerLockGrantGpio.r32 = lock_release.r32;
}
diff --git a/libs/MII/CMakeLists.txt b/libs/MII/CMakeLists.txt
index 568240b..0ddeabc 100644
--- a/libs/MII/CMakeLists.txt
+++ b/libs/MII/CMakeLists.txt
@@ -59,3 +59,5 @@ target_include_directories(${PROJECT_NAME}-mips PUBLIC include)
arm_add_library(${PROJECT_NAME}-arm STATIC mii.c)
target_include_directories(${PROJECT_NAME}-arm PUBLIC ../../include)
target_include_directories(${PROJECT_NAME}-arm PUBLIC include)
+
+format_target_sources(${PROJECT_NAME})
diff --git a/libs/MII/mii.c b/libs/MII/mii.c
index ad8e2ef..05934fd 100644
--- a/libs/MII/mii.c
+++ b/libs/MII/mii.c
@@ -58,14 +58,12 @@ uint8_t MII_getPhy(void)
if (DEVICE.SgmiiStatus.bits.MediaSelectionMode)
{
// SERDES platform
- return DEVICE.Status.bits.FunctionNumber +
- DEVICE_MII_COMMUNICATION_PHY_ADDRESS_SGMII_0;
+ return DEVICE.Status.bits.FunctionNumber + DEVICE_MII_COMMUNICATION_PHY_ADDRESS_SGMII_0;
}
else
{
// GPHY platform
- return DEVICE.Status.bits.FunctionNumber +
- DEVICE_MII_COMMUNICATION_PHY_ADDRESS_PHY_0;
+ return DEVICE.Status.bits.FunctionNumber + DEVICE_MII_COMMUNICATION_PHY_ADDRESS_PHY_0;
}
}
@@ -197,14 +195,12 @@ static void MII_writeShadowRegister18(uint8_t phy, mii_reg_t reg, uint16_t data)
shadow_select.r16 = 0;
shadow_select.bits.ShadowRegisterReadSelector = shadow_reg;
shadow_select.bits.ShadowRegisterSelector = 7;
- MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_AUXILIARY_CONTROL,
- shadow_select.r16);
+ MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_AUXILIARY_CONTROL, shadow_select.r16);
RegMIIMiscellaneousControl_t write_data;
write_data.r16 = data;
write_data.bits.ShadowRegisterSelector = shadow_reg;
- MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_AUXILIARY_CONTROL,
- write_data.r16);
+ MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_AUXILIARY_CONTROL, write_data.r16);
}
static void MII_writeShadowRegister1C(uint8_t phy, mii_reg_t reg, uint16_t data)
@@ -220,16 +216,14 @@ static void MII_writeShadowRegister1C(uint8_t phy, mii_reg_t reg, uint16_t data)
RegMIICabletronLed_t shadow_select;
shadow_select.r16 = 0;
shadow_select.bits.ShadowRegisterSelector = shadow_reg;
- MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_CABLETRON_LED,
- shadow_select.r16);
+ MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_CABLETRON_LED, shadow_select.r16);
RegMIICabletronLed_t write_data;
write_data.r16 = data;
write_data.bits.ShadowRegisterSelector = shadow_reg;
write_data.bits.WriteEnable = 1;
- MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_CABLETRON_LED,
- write_data.r16);
+ MII_writeRegisterInternal(phy, (mii_reg_t)REG_MII_CABLETRON_LED, write_data.r16);
}
void MII_writeRegister(uint8_t phy, mii_reg_t reg, uint16_t data)
@@ -268,6 +262,5 @@ void MII_reset(uint8_t phy)
do
{
// Spin
- } while ((MII_readRegister(phy, (mii_reg_t)REG_MII_CONTROL) &
- MII_CONTROL_RESET_MASK) == MII_CONTROL_RESET_MASK);
+ } while ((MII_readRegister(phy, (mii_reg_t)REG_MII_CONTROL) & MII_CONTROL_RESET_MASK) == MII_CONTROL_RESET_MASK);
}
diff --git a/libs/NVRam/CMakeLists.txt b/libs/NVRam/CMakeLists.txt
index 8f86e85..1ee2c39 100644
--- a/libs/NVRam/CMakeLists.txt
+++ b/libs/NVRam/CMakeLists.txt
@@ -60,3 +60,5 @@ target_include_directories(${PROJECT_NAME}-mips PUBLIC include)
arm_add_library(${PROJECT_NAME}-arm STATIC nvm.c crc.c)
target_include_directories(${PROJECT_NAME}-arm PUBLIC ../../include)
target_include_directories(${PROJECT_NAME}-arm PUBLIC include)
+
+format_target_sources(${PROJECT_NAME})
diff --git a/libs/NVRam/nvm.c b/libs/NVRam/nvm.c
index 33344ed..1813a73 100644
--- a/libs/NVRam/nvm.c
+++ b/libs/NVRam/nvm.c
@@ -159,8 +159,7 @@ static uint32_t NVRam_readWordInternal(uint32_t address, RegNVMCommand_t cmd)
return ntohl(NVM.Read.r32);
}
-static void NVRam_writeWordInternal(uint32_t address, uint32_t data,
- RegNVMCommand_t cmd)
+static void NVRam_writeWordInternal(uint32_t address, uint32_t data, RegNVMCommand_t cmd)
{
address = NVRam_translate(address);
OpenPOWER on IntegriCloud