<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
<html lang="en" style="--code-editor-font: var(--default-mono-font, "GitLab Mono"), JetBrains Mono, Menlo, DejaVu Sans Mono, Liberation Mono, Consolas, Ubuntu Mono, Courier New, andale mono, lucida console, monospace;">
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<title>
GitLab
</title>
<style data-premailer="ignore" type="text/css">
a { color: #1068bf; }
</style>
<style>img {
max-width: 100%; height: auto;
}
body {
font-size: .875rem;
}
body {
-webkit-text-shadow: rgba(255,255,255,.01) 0 0 1px;
}
body {
font-family: "GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji"; font-size: inherit;
}
</style>
</head>
<body style="font-size: inherit; -webkit-text-shadow: rgba(255,255,255,.01) 0 0 1px; font-family: "GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji";">
<div class="content">
<p class="details" style="font-style: italic; color: #626168;">
<a href="https://gitlab.rtems.org/mohamedayman23">Mohamed Ayman</a> created a merge request: <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1161">!1161</a>
</p>
<div class="branch">
Project:Branches: mohamedayman23/rtems:stm32h7-spi-group-cs-gpio to rtems/rtos/rtems:main
</div>
<div class="author">
Author: Mohamed Ayman
</div>
<div class="assignee">
Assignees:
</div>
<div class="reviewer">
Reviewers:
</div>
<div class="md gl-mt-5" style="position: relative; z-index: 1; color: #3a383f; word-wrap: break-word; margin-top: 1rem;">
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">bsps/arm/stm32h7: group SPI CS GPIO initialization by port</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">Group SPI chip select (CS) GPIO pins by GPIO port during MSP initialization. This allows multiple CS pins belonging to the same GPIO block to be configured and updated together using a single operation.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">Previously, each CS pin was initialized and set individually, even when multiple pins shared the same GPIO port. This change combines such pins into a single mask per port, improving efficiency and aligning better with the STM32 GPIO hardware design.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">Additionally, CS pins are set to the inactive (high) state before configuring them as outputs to reduce the risk of unintended active-low glitches during initialization.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">Basic validation checks are also added to guard against NULL SPI handles and missing configuration data.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">This change addresses the existing TODO regarding handling multiple pins within a single GPIO block.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">AI Usage: yes</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">AI Prompt: how can i improve this file neglect the TODO i will do it and till me how SPI is important but also optional.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">AI model: gemini pro</p>
<p dir="auto" style="color: #3a383f; margin: 0;" align="initial">How AI used: to clarify production improvement of code's goals</p>
</div>
</div>
<div class="footer" style="margin-top: 10px;">
<p style="font-size: small; color: #626168;">
—
<br>
<a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1161">View it on GitLab</a>.
<br>
You're receiving this email because of your account on <a target="_blank" rel="noopener noreferrer" href="https://gitlab.rtems.org">gitlab.rtems.org</a>. <a href="https://gitlab.rtems.org/-/sent_notifications/2-86w7bj91ly22viwmpz6f9em5s/unsubscribe" target="_blank" rel="noopener noreferrer">Unsubscribe</a> from this thread · <a href="https://gitlab.rtems.org/-/profile/notifications" target="_blank" rel="noopener noreferrer" class="mng-notif-link">Manage all notifications</a> · <a href="https://gitlab.rtems.org/help" target="_blank" rel="noopener noreferrer" class="help-link">Help</a>
<span style="color: transparent; font-size: 0; display: none; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0;">
Notification message regarding https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1161 at 1774414642
</span>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","action":{"@type":"ViewAction","name":"View Merge request","url":"https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1161"}}</script>
</p>
</div>
</body>
</html>