Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

I/O slots para o driver do Patinho Feio #1

Open
wants to merge 2 commits into
base: patinhofeio
Choose a base branch
from

Conversation

affonsoamendola
Copy link
Collaborator

Primeiro rascunho da interface para o patinho feio

WIP
Primeiro rascunho da interface para o patinho feio
@felipesanches felipesanches changed the title WIP I/O slots para o driver do Patinho Feio May 7, 2018
@@ -0,0 +1,64 @@
#include "emu.h"
#include "patinhofeio_io.h"

Choose a reason for hiding this comment

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

Acho que é bom remover o sufixo _io do nome do diretório e também dos arquivos .cpp e .h.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Entao, eu pus assim por causa do hp80, nele tem o driver la hp80.h e hp80.cpp na pasta mame/drivers, e tem hp80_io.h e hp80_io.cpp no /devices/bus, mas sei la

Choose a reason for hiding this comment

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

é... pode ser. Mas o nome do diretório tem que ser sem _io. Acho que no mínimo deveríamos mudar para src/devices/bus/patinhofeio/patinhofeio_io.cpp e src/devices/bus/patinhofeio/patinhofeio_io.h

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

Eu acho que você pode fazer:

git remote add fsanches https://github.com/felipesanches/mame.git
git fetch fsanches
git log fsanches/patinhofeio_io # só pra ver qual é o hash do ultimo commit que eu fiz lá
git cherry-pick c91f5e2 # pra pegar o commit e adicionar ao seu branch local
git log # pra ver que deu certo
git push origin/patinhofeio_io # pra enviar pro github e aparecer aqui nesse pull request

É claro que esse commit é super simples, mas é um bom momento pra treinar esses métodos de colaboração com o git.

Choose a reason for hiding this comment

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

E pra ser sincero com você, eu nem sei se essa é a forma mais fácil, mas é um dos jeitos de se fazer.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

bus/patinhofeio/patinhofeio_io ficaria bom, vo tentar fazer isso aí que ce falou

@felipesanches
Copy link

Vou ler o resto depois. Agora to indo dormir. Valeu, @affonsoamendola !

@affonsoamendola
Copy link
Collaborator Author

Acredito que tenha funcionado

@felipesanches
Copy link

sim, perfeito

@affonsoamendola
Copy link
Collaborator Author

Agora o q?, Da Merge pull request e parte pra proxima?

@felipesanches
Copy link

não... podemos continuar aprimorando o código aqui até ter algo que funcione.

@felipesanches
Copy link

felipesanches commented May 8, 2018

Eu ainda não li seu código com calma, não tive tempo de fazer isso hoje.

Mas veja no livro do Edson Fregni a descrição da interface de I/O. A arquitetura do Patinho Feio é descrita no capítulo 5. E o livro está disponível aqui: https://github.com/ArqueologiaDigital/PatinhoFeio/blob/master/Projeto_de_Computadores_Digitais_2a_ed__EFregni_GLangdonJr_1974__reimpressao_de_1979.pdf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants