New QL LED driver #3
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| # Copyright (c) 2018 Workaround GmbH | ||
spingaliQL marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| config EOS_S3_LED1 | ||
|
There was a problem hiding this comment. Without There was a problem hiding this comment. We have a simple LED controller and an advanced one. I'll call all "1" as "basic" |
||
| bool "EOS S3 LED driver 1" | ||
| help | ||
| Enable LED driver 1 for QL S3. | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| /* | ||
| * ========================================================== | ||
| * | ||
| * Copyright (C) 2020 QuickLogic Corporation | ||
| * Licensed under the Apache License, Version 2.0 (the "License"); | ||
| * you may not use this file except in compliance with the License. | ||
| * You may obtain a copy of the License at | ||
| * http://www.apache.org/licenses/LICENSE-2.0 | ||
| * Unless required by applicable law or agreed to in writing, software | ||
| * distributed under the License is distributed on an "AS IS" BASIS, | ||
| * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| * See the License for the specific language governing permissions and | ||
| * limitations under the License. | ||
| * | ||
| * File : eos_s3_led_config.h | ||
| * Purpose : This file contains the IO mux definitions for LEDs | ||
| * | ||
| * | ||
| * =========================================================== | ||
| * | ||
| */ | ||
|
|
||
|
|
||
| #ifndef _INC_EOS_S3_LED_CONFIG | ||
| #define _INC_EOS_S3_LED_CONFIG | ||
|
|
||
| #include <soc_pinmap.h> | ||
|
|
||
| /* Set FPGA_LED0 to PAD18 */ | ||
| #define FPGA_LED0_PAD18 (PAD_CTRL_SEL_FPGA | PAD_OEN_NORMAL \ | ||
| | PAD_P_Z | PAD_SR_SLOW | PAD_E_4MA \ | ||
| | PAD_REN_DISABLE | PAD_SMT_DISABLE) | ||
| #define FPGA_LED0_PAD18_FBIO PAD18_FUNC_SEL_FBIO_18 | ||
|
|
||
| /* Set FPGA_LED1 to PAD21 */ | ||
| #define FPGA_LED1_PAD21 (PAD_CTRL_SEL_FPGA | PAD_OEN_NORMAL \ | ||
| | PAD_P_Z | PAD_SR_SLOW | PAD_E_4MA \ | ||
| | PAD_REN_DISABLE | PAD_SMT_DISABLE) | ||
| #define FPGA_LED1_PAD21_FBIO PAD21_FUNC_SEL_FBIO_21 | ||
|
|
||
| /* Set FPGA_LED2 to PAD22 */ | ||
| #define FPGA_LED2_PAD22 (PAD_CTRL_SEL_FPGA | PAD_OEN_NORMAL \ | ||
| | PAD_P_Z | PAD_SR_SLOW | PAD_E_4MA \ | ||
| | PAD_REN_DISABLE | PAD_SMT_DISABLE) | ||
| #define FPGA_LED2_PAD22_FBIO PAD22_FUNC_SEL_FBIO_22 | ||
|
|
||
| #define FPGA_LED0_PAD 18 | ||
| #define FPGA_LED0_PAD_CFG FPGA_LED0_PAD18 | ||
| #define FPGA_LED0_FBIO_SEL FPGA_LED0_PAD18_FBIO | ||
| #define FPGA_LED1_PAD 21 | ||
| #define FPGA_LED1_PAD_CFG FPGA_LED1_PAD21 | ||
| #define FPGA_LED1_FBIO_SEL FPGA_LED1_PAD21_FBIO | ||
| #define FPGA_LED2_PAD 22 | ||
| #define FPGA_LED2_PAD_CFG FPGA_LED2_PAD22 | ||
| #define FPGA_LED2_FBIO_SEL FPGA_LED2_PAD22_FBIO | ||
|
|
||
|
|
||
| #endif /* _INC_EOS_S3_LED_CONFIG */ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| /* | ||
| * ========================================================== | ||
| * | ||
| * Copyright (C) 2020 QuickLogic Corporation | ||
| * Licensed under the Apache License, Version 2.0 (the "License"); | ||
| * you may not use this file except in compliance with the License. | ||
| * You may obtain a copy of the License at | ||
| * http://www.apache.org/licenses/LICENSE-2.0 | ||
| * Unless required by applicable law or agreed to in writing, software | ||
| * distributed under the License is distributed on an "AS IS" BASIS, | ||
| * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| * See the License for the specific language governing permissions and | ||
| * limitations under the License. | ||
| * | ||
| * File : led_eos_s3_1.c | ||
| * Purpose : This is the driver for LED controller IP. | ||
| * | ||
| * | ||
| * =========================================================== | ||
| * | ||
| */ | ||
|
|
||
| #include <zephyr.h> | ||
| #include <soc.h> | ||
| #include <fpga_loader.h> | ||
| #include "eos_s3_led1_ip.h" | ||
| #include "eos_s3_led_config.h" | ||
|
|
||
| static void config_ios() | ||
| { | ||
| eos_s3_io_mux(FPGA_LED0_PAD, FPGA_LED0_PAD_CFG); | ||
| eos_s3_io_mux(FPGA_LED1_PAD, FPGA_LED1_PAD_CFG); | ||
| eos_s3_io_mux(FPGA_LED2_PAD, FPGA_LED2_PAD_CFG); | ||
|
|
||
| eos_s3_fbio_select(FPGA_LED0_PAD,FPGA_LED0_FBIO_SEL); | ||
| eos_s3_fbio_select(FPGA_LED1_PAD,FPGA_LED1_FBIO_SEL); | ||
| eos_s3_fbio_select(FPGA_LED2_PAD,FPGA_LED2_FBIO_SEL); | ||
|
|
||
| } | ||
|
|
||
| void program_fpga_ip() | ||
spingaliQL marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| { | ||
| // Load bitstrem into FPGA | ||
spingaliQL marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| load_fpga(sizeof(axFPGABitStream),axFPGABitStream); | ||
spingaliQL marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| // Configure IOs | ||
spingaliQL marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| config_ios(); | ||
| } | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not change this selection, because
SOC_EOS_S3_FPGAconfiguration tells software if we want to initialize FPGA or not here. Without it clocks won't be set up.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The main change I made in these commits is to program FPGA from M4 program itself, instead of doing it outside using Jlink. To enable this feature I have added EOS_S3_PROGRAM_FPGA. If someone wants to use the old way, one can enable SOC_EOS_S3_FPGA. I have not made any changes in this flow. Since all clocks are enabled with EOS_S3_PROGRAM_FPGA, other is not required. So EOS_S3_PROGRAM_FPGA and SOC_EOS_S3_FPGA are mutually exclusive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does
eos_s3_pwm_ip.hcontainPWM_LITEX? If not you must add separate config entry like explained here.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No it doesn't.
I'll add separate config entry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added new configuration. However I understand from our H/W team that we are using litex pwm IP only, the bit stream is generated using symbi flow tools (with EOS S3 support). Our driver just loads the FPGA IP. It doesn't have full functions. For that I have to use pwm litex driver. So I have included that too.