Thanks for the help Pteridium. BTW, dmidecode is only available for x86, not 
for x86_64:
https://dev.openwrt.org/browser/packages/utils/dmidecode/Makefile -> 
DEPENDS:=@TARGET_x86

@Luka: If you have any other suggestions or changes just tell me.

P.S: Maybe we should completely remove dmidecode dependency, since it disables 
flashrom for any other targets (without this patch), and let the user select it 
if he wants that functionality.

Álvaro

El 09/05/2014 10:51, Luka Perkov escribió:
> On Fri, May 09, 2014 at 10:44:38AM +0200, José Vázquez wrote:
>> Sure? maybe i am a bit blind. :(
> 
> You are not - I've missed it... Thanks. I guess it was too early in the
> morning :)
> 
> Luka
> 
>> V1:
>> Eliminated:  DEPENDS:=+zlib +pciutils +dmidecode +libftdi
>> +libusb-compat @(arm||armeb||i386||i686||x86_64)
>> Added:  DEPENDS:=+zlib +pciutils +dmidecode +libftdi +libusb-compat
>> V2:
>> Eliminated: DEPENDS:=+zlib +pciutils +dmidecode +libftdi
>> +libusb-compat @(arm||armeb||i386||i686||x86_64)
>> Added: DEPENDS:=+zlib +pciutils +TARGET_x86:dmidecode +libftdi +libusb-compat
>>
>> Pepe
>>
>> 2014-05-09 10:34 GMT+02:00, Luka Perkov <l...@openwrt.org>:
>>> On Fri, May 09, 2014 at 10:28:50AM +0200, José Vázquez wrote:
>>>> 2014-05-09 8:32 GMT+02:00, Luka Perkov <l...@openwrt.org>:
>>>>> Hi Alvaro,
>>>>>
>>>>> I don't see this change. Patch v2 is same as v1.
>>>>>
>>>>> Luka
>>>>>
>>>> In v1 he eliminated cpu dependencies while in v2 dmidecode is only
>>>> selected by TARGET_x86.
>>>> I think that, in addition to x86 should be added x86_64 and any other
>>>> target that has dmi.
>>>
>>> Yes, it says that in the description but the patches are the same.
>>>
>>> Luka
>>>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
> 
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to