Re: [coreboot] Status Thinkpad T410

[kinky_nekoboi]

      ok the problems start when i try to merge the patch found on

https://review.coreboot.org/c/coreboot/+/11791

into a coreboot master clone ...

any guide? i am not that advanced in git

Am 08.11.18 um 17:40 schrieb Mike Banon:
  Go to review.coreboot.org, search for the recently
updated T410
 patches and apply them to your locally cloned coreboot, then build and
 test it. Let us know if you have a trouble at any step
 On Thu, Nov 8, 2018 at 7:35 PM kinky_nekoboi
 <kinky_nekoboi(a)bluetardis.de> wrote:
> I have hardware to test,
> T410 with intel gpu + little flashing tool collection.
> i am waiting for instructions
>
> Am 8. November 2018 17:30:00 MEZ schrieb Peter Lemenkov <lemenkov(a)gmail.com>om>:
>> Hello!
>>
>> чт, 8 нояб. 2018 г. в 10:50, Angel Pons <th3fanbus(a)gmail.com>om>:
>>>
>>>  Hello nekoboi,
>>>
>>>  On Thu, Nov 8, 2018, 10:39 kinky_nekoboi <kinky_nekoboi(a)bluetardis.de
wrote:
>>>>
>>>>  is there any work going on in the T410 Port ?
>>>>
>>>>  Hardware is simular to X201
>>>>
>>>>  I heard some people succesfully flashed x201 roms.
>>>
>>>
>>>  I recall seeing a gerrit patch for it, though it is old. I would work on it
but I don't have any Thinkpad (let alone a T410).
>>
>> Btw I've updated recently both t410 and t410s patches from gerrit
>> according to the recent coreboot changes, but I've got the same story
>> - I don't have hardware to test.
>>
> --
> coreboot mailing list: coreboot(a)coreboot.org
> https://mail.coreboot.org/mailman/listinfo/coreboot

Status Thinkpad T410

[kinky_nekoboi]

      ok this issue was caused by a merge conflict... i downloaded the tar
file and extracted it.

compilation fails with:

src/mainboard/lenovo/t410/romstage.c:320:2: error: implicit declaration
of function 'init_tpm' [-Werror=implicit-function-declaration]
  init_tpm(s3resume);
  ^~~~~~~~
    CC         romstage/northbridge/intel/nehalem/raminit.o
    CC         romstage/southbridge/intel/bd82x6x/early_me.o
    CC         romstage/southbridge/intel/bd82x6x/early_rcba.o
cc1: all warnings being treated as errors
make: *** [Makefile:338:
build/romstage/mainboard/lenovo/t410/romstage.o] Fehler 1

Am 08.11.18 um 20:29 schrieb kinky_nekoboi:
  ok the problems start when i try to merge the patch
found on

 https://review.coreboot.org/c/coreboot/+/11791

 into a coreboot master clone ...

 any guide? i am not that advanced in git

 Am 08.11.18 um 17:40 schrieb Mike Banon:
> Go to review.coreboot.org, search for the recently updated T410
> patches and apply them to your locally cloned coreboot, then build and
> test it. Let us know if you have a trouble at any step
> On Thu, Nov 8, 2018 at 7:35 PM kinky_nekoboi
> <kinky_nekoboi(a)bluetardis.de> wrote:
>> I have hardware to test,
>> T410 with intel gpu + little flashing tool collection.
>> i am waiting for instructions
>>
>> Am 8. November 2018 17:30:00 MEZ schrieb Peter Lemenkov
<lemenkov(a)gmail.com>om>:
>>> Hello!
>>>
>>> чт, 8 нояб. 2018 г. в 10:50, Angel Pons <th3fanbus(a)gmail.com>om>:
>>>>  Hello nekoboi,
>>>>
>>>>  On Thu, Nov 8, 2018, 10:39 kinky_nekoboi <kinky_nekoboi(a)bluetardis.de
wrote:
>>>>>  is there any work going on in the T410 Port ?
>>>>>
>>>>>  Hardware is simular to X201
>>>>>
>>>>>  I heard some people succesfully flashed x201 roms.
>>>>
>>>>  I recall seeing a gerrit patch for it, though it is old. I would work on
it but I don't have any Thinkpad (let alone a T410).
>>> Btw I've updated recently both t410 and t410s patches from gerrit
>>> according to the recent coreboot changes, but I've got the same story
>>> - I don't have hardware to test.
>>>
>> --
>> coreboot mailing list: coreboot(a)coreboot.org
>> https://mail.coreboot.org/mailman/listinfo/coreboot

Status Thinkpad T410

[Nico Huber]

      On 11/8/18 8:41 PM, kinky_nekoboi wrote:
  ok this issue was caused by a merge conflict... i
downloaded the tar
 file and extracted it.
You can checkout (not cherry-pick) the patch from Gerrit, then have a
look at the following:

  $ git diff HEAD^..origin/master -- src/mainboard/lenovo/x201/

This should show you the changes that were made to the X201 (closest
supported relative) in the meantime. You might have to replicate most
(maybe all) of these changes for the T410.

Now that I thought about it, you can just look what was the parent of
the commit on Gerrit (what I just did) and use that (no need to checkout
the commit):

  $ git diff 2819a40ed3..origin/master -- src/mainboard/lenovo/x201/

Nico

Status Thinkpad T410

[kinky_nekoboi]

      just found out that the author of this patch/port is also german
speaking and lifes in the same city area as me. i will mail him, than i
dont have to spam this mailing list with my broken english anymore about
the T410.

Am 08.11.18 um 20:59 schrieb Nico Huber:
  On 11/8/18 8:41 PM, kinky_nekoboi wrote:
  ok this issue was caused by a merge conflict... i
downloaded the tar
 file and extracted it.
 You can checkout (not cherry-pick) the patch from Gerrit,
then have a
 look at the following:

   $ git diff HEAD^..origin/master -- src/mainboard/lenovo/x201/

 This should show you the changes that were made to the X201 (closest
 supported relative) in the meantime. You might have to replicate most
 (maybe all) of these changes for the T410.

 Now that I thought about it, you can just look what was the parent of
 the commit on Gerrit (what I just did) and use that (no need to checkout
 the commit):

   $ git diff 2819a40ed3..origin/master -- src/mainboard/lenovo/x201/

 Nico

Status Thinkpad T410

[Peter Lemenkov]

      Hey!
Please try the latest patch from Gerrit - it should apply and compile cleanly.

чт, 8 нояб. 2018 г. в 21:19, kinky_nekoboi <kinky_nekoboi(a)bluetardis.de>de>:
 
 just found out that the author of this patch/port is also german
 speaking and lifes in the same city area as me. i will mail him, than i
 dont have to spam this mailing list with my broken english anymore about
 the T410.

 Am 08.11.18 um 20:59 schrieb Nico Huber:
  On 11/8/18 8:41 PM, kinky_nekoboi wrote:
  ok this issue was caused by a merge conflict... i
downloaded the tar
 file and extracted it.
 You can checkout (not cherry-pick) the patch from Gerrit,
then have a
 look at the following:

   $ git diff HEAD^..origin/master -- src/mainboard/lenovo/x201/

 This should show you the changes that were made to the X201 (closest
 supported relative) in the meantime. You might have to replicate most
 (maybe all) of these changes for the T410.

 Now that I thought about it, you can just look what was the parent of
 the commit on Gerrit (what I just did) and use that (no need to checkout
 the commit):

   $ git diff 2819a40ed3..origin/master -- src/mainboard/lenovo/x201/

 Nico
 --
 coreboot mailing list: coreboot(a)coreboot.org
 https://mail.coreboot.org/mailman/listinfo/coreboot


-- 
With best regards, Peter Lemenkov.