Re: [coreboot] Bayley Bay FSP-based CRB

2018-06-18 Thread Jose Trujillo via coreboot
Hello Zvika:
1.- Usually it is not necessary to change the CBFS size unless the compiler 
complain of lack of space.
2.- You should not worry about this setting to make your system to work.
3.- You should not use FSP_PACKAGE_DEFAULT if your plan is to use SIO because 
it will enable SOC internal COM1, instead (if your plan is to use FSP) enable 
FSP and add  a VGA bios bin manually (The use of FSP is optional but I 
never tried without FSP).
4.- You need to add them yourself, there are examples to follow in this mail 
list.
Good luck!
J.Trujillo

‐‐‐ Original Message ‐‐‐
On June 18, 2018 6:24 AM, Zvi Vered  wrote:

> Hello,
> I inspected the "Bayley Baay FSP-based CRB" sample in coreboot. after make 
> distclean I chose:
> Mainboard vendor: Intel
> Mainboard model: Bayley Bay FSP-based CRB
> 1. The size of CBFS is: 0x20. Is it a fix size or should I change it 
> according to my board (which is also bay trail) ?
> 2. According to "dmidecode" I ran on my target, "Address: 0xE"
> Should I set this address in coreboot configuration ? How ?
>
> 3. In this board default configuration, "Configure defaults for the Intel FSP 
> package" is not selected. Is it possible that this board does not use Intel 
> FSP at all ?
> Under "Generic Drivers", "Use Intel firmware Support Package' is also not 
> selected.
>
> 4. Under "chipset", there is no option to set "Super I/O". Can you please 
> tell why ?
> 5. I noticed that the minimum ROM size is 2MB. If I set 4MB, the size of 
> coreboot.rom is also 4MB.
>
> Thank you in advance,
> Zvika-- 
coreboot mailing list: coreboot@coreboot.org
https://mail.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] Bayley Bay FSP-based CRB

2018-06-20 Thread Zvi Vered
Hello,

Thank you very much for the detailed reply.
Vendor's BIOS contains few peripherals initialization.
For example: PCIe enumeration, SATA controller, USB etc.
In case of PCIe enumeration, the generation of PCIe (1,2,3) can be set.
Should vendor supply code for this ? or any other information ?
How can I write it from scratch ?  Can Intel provide information on how to
implement this initialization ?

Thank you,
Zvika


On Mon, Jun 18, 2018 at 11:22 AM Jose Trujillo via coreboot <
coreboot@coreboot.org> wrote:

> Hello Zvika:
> 1.- Usually it is not necessary to change the CBFS size unless the
> compiler complain of lack of space.
> 2.- You should not worry about this setting to make your system to work.
> 3.- You should not use FSP_PACKAGE_DEFAULT if your plan is to use SIO
> because it will enable SOC internal COM1, instead (if your plan is to use
> FSP) enable FSP and add  a VGA bios bin manually (The use of FSP is
> optional but I never tried without FSP).
> 4.- You need to add them yourself, there are examples to follow in this
> mail list.
> Good luck!
> J.Trujillo
>
> ‐‐‐ Original Message ‐‐‐
> On June 18, 2018 6:24 AM, Zvi Vered  wrote:
>
> Hello,
> I inspected the "Bayley Baay FSP-based CRB" sample in coreboot. after make
> distclean I chose:
> Mainboard vendor: Intel
> Mainboard model: Bayley Bay FSP-based CRB
> 1. The size of CBFS is: 0x20. Is it a fix size or should I change it
> according to my board (which is also bay trail) ?
> 2. According to "dmidecode" I ran on my target, "Address: 0xE"
> Should I set this address in coreboot configuration ? How ?
>
> 3. In this board default configuration, "Configure defaults for the Intel
> FSP package" is not selected. Is it possible that this board does not use
> Intel FSP at all ?
> Under "Generic Drivers", "Use Intel firmware Support Package' is also not
> selected.
>
> 4. Under "chipset", there is no option to set "Super I/O". Can you please
> tell why ?
> 5. I noticed that the minimum ROM size is 2MB. If I set 4MB, the size of
> coreboot.rom is also 4MB.
>
> Thank you in advance,
> Zvika
>
>
> --
> coreboot mailing list: coreboot@coreboot.org
> https://mail.coreboot.org/mailman/listinfo/coreboot
-- 
coreboot mailing list: coreboot@coreboot.org
https://mail.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] Bayley Bay FSP-based CRB

2018-06-21 Thread Nico Huber
Hello Zvika,

On 18.06.2018 05:24, Zvi Vered wrote:
> 1. The size of CBFS is: 0x20. Is it a fix size or should I change it
> according to my board (which is also bay trail) ?

on Intel platforms, the SPI flash is shared with other chipset compo-
nents. The CBFS_SIZE should be at most the size of the "BIOS" region
of the flash.

> 3. In this board default configuration, "Configure defaults for the Intel
> FSP package" is not selected. Is it possible that this board does not use
> Intel FSP at all ?
>
> Under "Generic Drivers", "Use Intel firmware Support Package' is also not
> selected.
>

FSP is mandatory for a bootable image. It is not enabled by default
because Intel does not (always) publish the matching binaries, but our
build tests require the defaults to build. This leaves us with unfor-
tunate default settings for all Intel FSP platforms that will never
boot. If you think that is a bad idea (I do) please let Intel know that
you are interested in working defaults and that it requires them to
publish all the binaries that are integrated into coreboot.

> 4. Under "chipset", there is no option to set "Super I/O". Can you please
> tell why ?

Our config options do not allow board specific changes. For such
settings, each board has it's own directory and Kconfig file
(src/mainboard///Kconfig).

> 
> 5. I noticed that the minimum ROM size is 2MB. If I set 4MB, the size of
> coreboot.rom is also 4MB.

That is correct, but keep in mind that the flash is shared. The
resulting coreboot.rom might not contain everything that is required
to boot. So it is best advised to only flash the "BIOS" region of
the coreboot.rom.

Hope that helps,
Nico

-- 
coreboot mailing list: coreboot@coreboot.org
https://mail.coreboot.org/mailman/listinfo/coreboot


Re: [coreboot] Bayley Bay FSP-based CRB

2018-06-21 Thread Jose Trujillo via coreboot
Hello Zvika,
Look for the list of Linux commands to dump many of the information from your 
original BIOS running, maybe there you will find this information.

Also, some configuration can be seen from your original BIOS running Intel FIT 
for Baytrail in Windows.

About configuring those settings in coreboot look for other examples in the 
coreboot tree about configuring PCI devices in devicetree.cb and other places.

I Am new to coreboot too and please correct me if I Am wrong.
Good Luck!
J. Trujillo

‐‐‐ Original Message ‐‐‐
On June 21, 2018 3:34 AM, Zvi Vered  wrote:

> Hello,
>
> Thank you very much for the detailed reply.
> Vendor's BIOS contains few peripherals initialization.
> For example: PCIe enumeration, SATA controller, USB etc.
> In case of PCIe enumeration, the generation of PCIe (1,2,3) can be set.
> Should vendor supply code for this ? or any other information ?
> How can I write it from scratch ?  Can Intel provide information on how to 
> implement this initialization ?
>
> Thank you,
> Zvika
>
> On Mon, Jun 18, 2018 at 11:22 AM Jose Trujillo via coreboot 
>  wrote:
>
>> Hello Zvika:
>> 1.- Usually it is not necessary to change the CBFS size unless the compiler 
>> complain of lack of space.
>> 2.- You should not worry about this setting to make your system to work.
>> 3.- You should not use FSP_PACKAGE_DEFAULT if your plan is to use SIO 
>> because it will enable SOC internal COM1, instead (if your plan is to use 
>> FSP) enable FSP and add  a VGA bios bin manually (The use of FSP is 
>> optional but I never tried without FSP).
>> 4.- You need to add them yourself, there are examples to follow in this mail 
>> list.
>> Good luck!
>> J.Trujillo
>>
>> ‐‐‐ Original Message ‐‐‐
>> On June 18, 2018 6:24 AM, Zvi Vered  wrote:
>>
>>> Hello,
>>> I inspected the "Bayley Baay FSP-based CRB" sample in coreboot. after make 
>>> distclean I chose:
>>> Mainboard vendor: Intel
>>> Mainboard model: Bayley Bay FSP-based CRB
>>> 1. The size of CBFS is: 0x20. Is it a fix size or should I change it 
>>> according to my board (which is also bay trail) ?
>>> 2. According to "dmidecode" I ran on my target, "Address: 0xE"
>>> Should I set this address in coreboot configuration ? How ?
>>>
>>> 3. In this board default configuration, "Configure defaults for the Intel 
>>> FSP package" is not selected. Is it possible that this board does not use 
>>> Intel FSP at all ?
>>> Under "Generic Drivers", "Use Intel firmware Support Package' is also not 
>>> selected.
>>>
>>> 4. Under "chipset", there is no option to set "Super I/O". Can you please 
>>> tell why ?
>>> 5. I noticed that the minimum ROM size is 2MB. If I set 4MB, the size of 
>>> coreboot.rom is also 4MB.
>>>
>>> Thank you in advance,
>>> Zvika
>>
>> --
>> coreboot mailing list: coreboot@coreboot.org
>> https://mail.coreboot.org/mailman/listinfo/coreboot-- 
coreboot mailing list: coreboot@coreboot.org
https://mail.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] Bayley Bay FSP-based CRB

2018-06-21 Thread Nico Huber
On 21.06.2018 02:34, Zvi Vered wrote:
> In case of PCIe enumeration, the generation of PCIe (1,2,3) can be set> 
> Should vendor supply code for this ? or any other information ?

PCIe configuration is SoC specific and should be done by FSP. However,
I can't find any PCIe specific settings for the BayTrail FSP.

As this is one of the earlier FSP releases, it is not unlikely that
it does not expose all the option Intel's reference code provides. I'll
add the maintainers of the BayTrail FSP hook-up in CC. Maybe they can
tell us more about it.

> How can I write it from scratch ?  Can Intel provide information on how to
> implement this initialization ?

AFAIK, Intel does not document PCIe initialization and encourages every-
one to use their reference code (which is compiled into FSP). It's a
shame that they don't allow open-source implementations because we can't
make any alterations to FSP once Intel thinks it's finished (which is
almost always too early, IMHO).

Nico

-- 
coreboot mailing list: coreboot@coreboot.org
https://mail.coreboot.org/mailman/listinfo/coreboot


Re: [coreboot] Bayley Bay FSP-based CRB

2018-06-26 Thread Zvi Vered
Dear Jose & Nico,

Thank you very much for your help !

Best regards,
Zvika

On Thu, Jun 21, 2018 at 11:28 AM Jose Trujillo 
wrote:

> Hello Zvika,
> Look for the list of Linux commands to dump many of the information from
> your original BIOS running, maybe there you will find this information.
>
> Also, some configuration can be seen from your original BIOS running Intel
> FIT for Baytrail in Windows.
>
> About configuring those settings in coreboot look for other examples in
> the coreboot tree about configuring PCI devices in devicetree.cb and other
> places.
>
> I Am new to coreboot too and please correct me if I Am wrong.
> Good Luck!
> J. Trujillo
>
> ‐‐‐ Original Message ‐‐‐
> On June 21, 2018 3:34 AM, Zvi Vered  wrote:
>
> Hello,
>
> Thank you very much for the detailed reply.
> Vendor's BIOS contains few peripherals initialization.
> For example: PCIe enumeration, SATA controller, USB etc.
> In case of PCIe enumeration, the generation of PCIe (1,2,3) can be set.
> Should vendor supply code for this ? or any other information ?
> How can I write it from scratch ?  Can Intel provide information on how to
> implement this initialization ?
>
> Thank you,
> Zvika
>
>
> On Mon, Jun 18, 2018 at 11:22 AM Jose Trujillo via coreboot <
> coreboot@coreboot.org> wrote:
>
>> Hello Zvika:
>> 1.- Usually it is not necessary to change the CBFS size unless the
>> compiler complain of lack of space.
>> 2.- You should not worry about this setting to make your system to work.
>> 3.- You should not use FSP_PACKAGE_DEFAULT if your plan is to use SIO
>> because it will enable SOC internal COM1, instead (if your plan is to use
>> FSP) enable FSP and add  a VGA bios bin manually (The use of FSP is
>> optional but I never tried without FSP).
>> 4.- You need to add them yourself, there are examples to follow in this
>> mail list.
>> Good luck!
>> J.Trujillo
>>
>> ‐‐‐ Original Message ‐‐‐
>> On June 18, 2018 6:24 AM, Zvi Vered  wrote:
>>
>> Hello,
>> I inspected the "Bayley Baay FSP-based CRB" sample in coreboot. after
>> make distclean I chose:
>> Mainboard vendor: Intel
>> Mainboard model: Bayley Bay FSP-based CRB
>> 1. The size of CBFS is: 0x20. Is it a fix size or should I change it
>> according to my board (which is also bay trail) ?
>> 2. According to "dmidecode" I ran on my target, "Address: 0xE"
>> Should I set this address in coreboot configuration ? How ?
>>
>> 3. In this board default configuration, "Configure defaults for the Intel
>> FSP package" is not selected. Is it possible that this board does not use
>> Intel FSP at all ?
>> Under "Generic Drivers", "Use Intel firmware Support Package' is also not
>> selected.
>>
>> 4. Under "chipset", there is no option to set "Super I/O". Can you please
>> tell why ?
>> 5. I noticed that the minimum ROM size is 2MB. If I set 4MB, the size of
>> coreboot.rom is also 4MB.
>>
>> Thank you in advance,
>> Zvika
>>
>>
>> --
>> coreboot mailing list: coreboot@coreboot.org
>> https://mail.coreboot.org/mailman/listinfo/coreboot
>>
>
>
-- 
coreboot mailing list: coreboot@coreboot.org
https://mail.coreboot.org/mailman/listinfo/coreboot