<p><a href="https://review.coreboot.org/c/coreboot/+/30234">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/30234/1/src/soc/intel/broadwell/Kconfig">File src/soc/intel/broadwell/Kconfig:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/30234/1/src/soc/intel/broadwell/Kconfig@39">Patch Set #1, Line 39:</a> <code style="font-family:monospace,monospace">SOC_INTEL_COMMON_BLOCK</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">if you are intended to make use of intel common code, then you should break ur CL into 2 part.</p><p style="white-space: pre-wrap; word-wrap: break-word;">1. make use of Intel common code in BDW context<br>2. Make use of those SA code to create your ASL</p><p style="white-space: pre-wrap; word-wrap: break-word;">but in #1, you have bigger problem because from design standpoint, we have agreed to add support from Intel Gen 6 onwards (skylake) platform but BDW is older generation platform. where we haven't validated those socs.</p><p style="white-space: pre-wrap; word-wrap: break-word;">once you decided to make use of that, you have further 2 challenges </p><p style="white-space: pre-wrap; word-wrap: break-word;">1. Add support in gen 4 and gen 5 platform <br>2. remove redundant code, once you have included SA kconfig, entire library code will be there, you are having duplicate functions now hence you might need to remove those duplicate code from soc</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30234">change 30234</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30234"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I1a10a4f91b787b72f33150031b783d426148c25d </div>
<div style="display:none"> Gerrit-Change-Number: 30234 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matt DeVillier <matt.devillier@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 15 Dec 2018 16:19:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>