Skip to content

Conversation

@wipawel
Copy link
Contributor

@wipawel wipawel commented Feb 19, 2022

  • add detection and initialization of a framebuffer (currently RGB color mode is assumed)
  • add simple 8x8 fonts
  • add logo drawing functionality
  • add console output to framebuffer video memory (this allows to use KTF on UEFI systems without serial)

image

@wipawel wipawel added feature New feature or request Priority: 2 Very important feature labels Feb 19, 2022
@wipawel wipawel added this to the v0.5.0 milestone Feb 19, 2022
@wipawel wipawel requested a review from a team as a code owner February 19, 2022 10:12
* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/

unsigned char logo[] = {
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe a comment on how it was generated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's basically a Copy and Past as C code from 010 Editor's opened BMP file.

endif

AFLAGS := $(COMMON_FLAGS) -D__ASSEMBLY__ -nostdlib -nostdinc
AFLAGS := $(COMMON_FLAGS) -DASM_FILE -D__ASSEMBLY__ -nostdlib -nostdinc
Copy link
Contributor

Choose a reason for hiding this comment

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

What is ASM_FILE?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the official multiboot.h file we use uses ASM_FILE to distinguish between C and asm. We use __ASSEMBLY__, but I do not want to modify the original file.

@wipawel wipawel merged commit bdf6272 into KernelTestFramework:mainline Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request Priority: 2 Very important feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants