Re: [coreboot]: Help request with SMSC SCH3114 superio.

[Jose Trujillo]

Dear Michal,

I am sorry for the late reply but on Friday something went wrong with my coreboot installation and was unable to boot and I have to reinstall it again and I just resumed this morning on this issue.

(you) don't think loading the watchdog module will help in any way, actually the opposite.
(Me) I did the modprobe of the watchdog module just to ask linux to give me the base address of the runtime registers "0xa00" of the original bios.

(you) I recommend using https://github.com/adurbin/iotools

(you) Superiotool will simply dump all registers and may alternatively dump IO too.
(Me) Thank you for your great advice I will go by the Superiotool route (patching it) but so far I am dumping using "ioports" in the following way:

on root:
for i in $(seq 0xa00 0xa7f); do echo -n "$i: "; inb $i; done; $i = $((++i))

(you) I assume you have configured runtime registers IOBASE right?

(Me) I found out (because of your advise) that the runtime registers base address was wrong on my devicetree (0xe00 instead 0xa00) I set it correctly then linux was able to "see" this range and dumped the registers with the string shown above.
Now I can compare both dumps and I can see the pins for the UART ports 2-4 are all GPIO inputs (1).

(Me) I set manually the GPIOs from linux using ioport's function outb and all the 4 ports work correctly.
Now I am trying to set from coreboot but I am still unsuccessful.
Looking at you dell Optiplex 9010 you set the following runtime registers under romstage.c -> mainboard_early_init

/* Disable SMIs and clear SMI status */
outb(0, SCH5545_RUNTIME_REG_BASE + SCH5545_RR_SMI_EN);
outb(SCH5545_SMI_GLOBAL_STS, SCH5545_RUNTIME_REG_BASE + SCH5545_RR_SMI_STS);

and I do similar to you under romstage.c -> mainboard_early_init:

outb(0x55, 0x2e);
outb(0x05, 0x0a3f); /* GP50= RI_2 : in */
outb(0x05, 0x0a40); /* GP51= DCD_2 : in */
outb(0x05, 0x0a41); /* GP52= RXD_2 : in */
outb(0x04, 0x0a42); /* GP53= TXD_2 : out */
outb(0x05, 0x0a43); /* GP54= DSR_2 : in */
outb(0x04, 0x0a44); /* GP55= RTS_2 : out */
outb(0x05, 0x0a45); /* GP56= CTS_2 : in */
outb(0x04, 0x0a46); /* GP57= DTR_2 : out */
outb(0xaa, 0x2e);

but It doesn't work.

Also I set the same code under mainboard.c -> mainboard_init but neither work.

Any advice on this? because I cannot find any information on the datasheet on how to set those registers and I suppose I just have to set to the 0xa00 base address + register.

Thank you,
Jose Trujillo.

[coreboot]: Help request with SMSC SCH3114 superio.

[Michal Zygowski]

Hi Jose,
> and I do similar to you under romstage.c -> mainboard_early_init:
>
> outb(0x55, 0x2e);
> outb(0x05, 0x0a3f); /* GP50= RI_2 : in */
> outb(0x05, 0x0a40); /* GP51= DCD_2 : in */
> outb(0x05, 0x0a41); /* GP52= RXD_2 : in */
> outb(0x04, 0x0a42); /* GP53= TXD_2 : out */
> outb(0x05, 0x0a43); /* GP54= DSR_2 : in */
> outb(0x04, 0x0a44); /* GP55= RTS_2 : out */
> outb(0x05, 0x0a45); /* GP56= CTS_2 : in */
> outb(0x04, 0x0a46); /* GP57= DTR_2 : out */
> outb(0xaa, 0x2e);
>
> but It doesn't work.
>
> Also I set the same code under mainboard.c -> mainboard_init but
> neither work.

You should not copy my code for the two major reasons:

1. SCH5545 and SCH3114 are very different.
2. GPIO configuration on SCH5545 is not accessible from SuperIO. The
code I have written communicates with Environmental Controller (ARC
coprocessor) which resides inside the SCH5545, because only the EC had
access to those registers. The SCH5545 did not have GPIO config
registers in the Runtime Registers block.

This will obviously not work.
> Any advice on this? because I cannot find any information on the
> datasheet on how to set those registers and I suppose I just have to
> set to the 0xa00 base address + register.

I suggest to look at TABLE 26-3 in the datasheet of your part
http://ww1.microchip.com/downloads/en/DeviceDoc/00001872A.pdf
and using these runtime registers definitions, write the right code that
will set the GPIOs up for UARTs.  Just use the appropriate 0xa00 base +
REG OFFSET from the table and you should be allright. For simplicity you
may also compare these registers to reference values from original
firmware (but with care! sometimes firmware vendors tend to make stupid
mistakes).
> Thank you,
> Jose Trujillo.

Best regards,

-- 
Michał Żygowski
Firmware Engineer
https://3mdeb.com | @3mdeb_com

[coreboot]: Help request with SMSC SCH3114 superio.

[Jose Trujillo]

Thank you Michal:

(you) Just use the appropriate 0xa00 base +

REG OFFSET from the table and you should be allright.

(Me) This is what I believe I did: 0xa00 + 0x040 = GP51/DCD_2 Function Select. Page (272) top

Maybe I am not setting the code in the right place or executing at the right time?

I am still looking and testing different things to resolve this issue.

Have a good day
Jose Trujillo.

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Tuesday, September 1, 2020 10:02 AM, Michal Zygowski [e-mail] wrote:
> Hi Jose,
> > and I do similar to you under romstage.c -> mainboard_early_init:
> > outb(0x55, 0x2e);
> > outb(0x05, 0x0a3f); /* GP50= RI_2 : in /
> > outb(0x05, 0x0a40); / GP51= DCD_2 : in /
> > outb(0x05, 0x0a41); / GP52= RXD_2 : in /
> > outb(0x04, 0x0a42); / GP53= TXD_2 : out /
> > outb(0x05, 0x0a43); / GP54= DSR_2 : in /
> > outb(0x04, 0x0a44); / GP55= RTS_2 : out /
> > outb(0x05, 0x0a45); / GP56= CTS_2 : in /
> > outb(0x04, 0x0a46); / GP57= DTR_2 : out */outb(0xaa, 0x2e);
> > but It doesn't work.
> > Also I set the same code under mainboard.c -> mainboard_init but
> > neither work.
>
> You should not copy my code for the two major reasons:
>
> - SCH5545 and SCH3114 are very different.
> - GPIO configuration on SCH5545 is not accessible from SuperIO. The
> code I have written communicates with Environmental Controller (ARC
> coprocessor) which resides inside the SCH5545, because only the EC had
> access to those registers. The SCH5545 did not have GPIO config
> registers in the Runtime Registers block.
> This will obviously not work.
> > Any advice on this? because I cannot find any information on the
> > datasheet on how to set those registers and I suppose I just have to
> > set to the 0xa00 base address + register.
>
> I suggest to look at TABLE 26-3 in the datasheet of your part
> http://ww1.microchip.com/downloads/en/DeviceDoc/00001872A.pdf
> and using these runtime registers definitions, write the right code that
> will set the GPIOs up for UARTs. Just use the appropriate 0xa00 base +
> REG OFFSET from the table and you should be allright. For simplicity you
> may also compare these registers to reference values from original
> firmware (but with care! sometimes firmware vendors tend to make stupid
> mistakes).
> > Thank you,
> > Jose Trujillo.
>
> Best regards,
> ---------------------------------------------------------------
>
> Michał Żygowski
> Firmware Engineer
> https://3mdeb.com | @3mdeb_com
> coreboot mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[coreboot]: Help request with SMSC SCH3114 superio.

[Jose Trujillo]

Dear Michal:

Thank you very much for all your guidance.
The problem was resolved.

The code was OK but was not executed at the right time (was executed before LPC and SIO were initialized).

The attached code did the job

Have a great day.
Jose Trujillo.


‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Tuesday, September 1, 2020 10:02 AM, Michal Zygowski <[e-mail]> wrote:
> Hi Jose,
> > and I do similar to you under romstage.c -> mainboard_early_init:
> > outb(0x55, 0x2e);
> > outb(0x05, 0x0a3f); /* GP50= RI_2 : in /
> > outb(0x05, 0x0a40); / GP51= DCD_2 : in /
> > outb(0x05, 0x0a41); / GP52= RXD_2 : in /
> > outb(0x04, 0x0a42); / GP53= TXD_2 : out /
> > outb(0x05, 0x0a43); / GP54= DSR_2 : in /
> > outb(0x04, 0x0a44); / GP55= RTS_2 : out /
> > outb(0x05, 0x0a45); / GP56= CTS_2 : in /
> > outb(0x04, 0x0a46); / GP57= DTR_2 : out */outb(0xaa, 0x2e);
> > but It doesn't work.
> > Also I set the same code under mainboard.c -> mainboard_init but
> > neither work.
>
> You should not copy my code for the two major reasons:
>
> 1.  SCH5545 and SCH3114 are very different.
> 2.  GPIO configuration on SCH5545 is not accessible from SuperIO. The
>     code I have written communicates with Environmental Controller (ARC
>     coprocessor) which resides inside the SCH5545, because only the EC had
>     access to those registers. The SCH5545 did not have GPIO config
>     registers in the Runtime Registers block.
>
> This will obviously not work.
> > Any advice on this? because I cannot find any information on the
> > datasheet on how to set those registers and I suppose I just have to
> > set to the 0xa00 base address + register.
>
> I suggest to look at TABLE 26-3 in the datasheet of your part
> http://ww1.microchip.com/downloads/en/DeviceDoc/00001872A.pdf
> and using these runtime registers definitions, write the right code that
> will set the GPIOs up for UARTs.  Just use the appropriate 0xa00 base +
> REG OFFSET from the table and you should be allright. For simplicity you
> may also compare these registers to reference values from original
> firmware (but with care! sometimes firmware vendors tend to make stupid
> mistakes).
> > Thank you,
> > Jose Trujillo.
>
> Best regards,
>
> ----------------
>
> Michał Żygowski
> Firmware Engineer
> https://3mdeb.com | @3mdeb_com
>
> coreboot mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[coreboot]: Help request with SMSC SCH3114 superio.

[Michal Zygowski]

Dear Jose,

I felt that you are very close to resolving it and it is some minor
issue not exactly in the Super I/O init code.

Great job and good luck with future challenges.

And of course enjoy coreboot!

Best regards,

-- 
Michał Żygowski
Firmware Engineer
https://3mdeb.com | @3mdeb_com

On 02.09.2020 16:12, Jose Trujillo via coreboot wrote:
> Dear Michal:
>
> Thank you very much for all your guidance.
> The problem was resolved.
>
> The code was OK but was not executed at the right time (was executed before LPC and SIO were initialized).
>
> The attached code did the job
>
> Have a great day.
> Jose Trujillo.
>
> ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
> On Tuesday, September 1, 2020 10:02 AM, Michal Zygowski <[e-mail]> wrote:
> > Hi Jose,
> > > and I do similar to you under romstage.c -> mainboard_early_init:
> > > outb(0x55, 0x2e);
> > > outb(0x05, 0x0a3f); /* GP50= RI_2 : in /
> > > outb(0x05, 0x0a40); / GP51= DCD_2 : in /
> > > outb(0x05, 0x0a41); / GP52= RXD_2 : in /
> > > outb(0x04, 0x0a42); / GP53= TXD_2 : out /
> > > outb(0x05, 0x0a43); / GP54= DSR_2 : in /
> > > outb(0x04, 0x0a44); / GP55= RTS_2 : out /
> > > outb(0x05, 0x0a45); / GP56= CTS_2 : in /
> > > outb(0x04, 0x0a46); / GP57= DTR_2 : out */outb(0xaa, 0x2e);
> > > but It doesn't work.
> > > Also I set the same code under mainboard.c -> mainboard_init but
> > > neither work.
> > > You should not copy my code for the two major reasons:
> >
> > 1.  SCH5545 and SCH3114 are very different.
> > 2.  GPIO configuration on SCH5545 is not accessible from SuperIO. The
> >     code I have written communicates with Environmental Controller (ARC
> >     coprocessor) which resides inside the SCH5545, because only the EC had
> >     access to those registers. The SCH5545 did not have GPIO config
> >     registers in the Runtime Registers block.
> >
> > This will obviously not work.
> > > Any advice on this? because I cannot find any information on the
> > > datasheet on how to set those registers and I suppose I just have to
> > > set to the 0xa00 base address + register.
> > > I suggest to look at TABLE 26-3 in the datasheet of your part
> > > http://ww1.microchip.com/downloads/en/DeviceDoc/00001872A.pdf
> > > and using these runtime registers definitions, write the right code that
> > > will set the GPIOs up for UARTs.  Just use the appropriate 0xa00 base +
> > > REG OFFSET from the table and you should be allright. For simplicity you
> > > may also compare these registers to reference values from original
> > > firmware (but with care! sometimes firmware vendors tend to make stupid
> > > mistakes).
> > > Thank you,
> > > Jose Trujillo.
> > > Best regards,
> >
> > ----------------
> >
> > Michał Żygowski
> > Firmware Engineer
> > https://3mdeb.com | @3mdeb_com
> >
> > coreboot mailing list -- [e-mail]
> > To unsubscribe send an email to [e-mail]
>
> _______________________________________________
> coreboot mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]