Re: [OpenWrt-Devel] [RFC 4/6] x86: geode: shrink amount of default packages

2019-01-16 Thread Tomasz Maciej Nowak
W dniu 16.01.2019 o 02:25, Philip Prindeville pisze:
> Inline
> 
>> On Jan 15, 2019, at 12:40 AM, Tomasz Maciej Nowak  wrote:
>>
>> Hi Philip
>>
>> W dniu 14.01.2019 o 21:10, Philip Prindeville pisze:
>>
>>> Why is the hwclock being dropped everywhere?
>>
>> It's provided by busybox and installed by default, also greping for its 
>> usage in target/linux/x86/base-files returned nothing. Am I missing 
>> something, is the full fledged one used somewhere or by something?
> 
> 
> It’s used at least two places.
> 
> When you first get the board and bring it up and set the clock, since it’s 
> not usually set correctly by the factory, and it’s used by the ntpd script to 
> save the synchronized time when the system is taken down.

But it uses options available in busybox hwclock so I still don't see a reason 
to add it.

> 
>>
>>> And who says the geos2, net5501, or alix2 are obsolete?
>>
>> They are not obsolete by any means, only the way to assign packages for 
>> specific device changes. Please take a look at patch nr 2 in this RFC and 
>> section adding target/linux/x86/image/geode.mk. There You have almost all 
>> packages replicated with additional geos device profile. Now You'll get two 
>> images, one generic, and one for geos. This distinction is here because with 
>> old way of adding packages, the generic image would end up with lot of 
>> useless, geos/dsl specific packages.
> 
> 
> So why is there a Device/geos but none of the other SBC’s?

None other SBC is so specific (I followed what packages were specified in 
profiles), and for other most drivers are enabled in kernel config.

> And why doesn’t the geos DEVICE_PACKAGES include -kmod-via-rhine, since it 
> uses the kmod-8139cp instead?

I can remove the call for generic profile and add all relevant packages if that 
is preferred. Anyway both of these drivers are enabled in kernel config as 
built-in and those kmod packages are empty. In next version of this series I'll 
remove kmod packages which are enabled in kernel.

> 
> -Philip

Please give this series a test.
Regards, Tomasz.

> 
> 
> 
>>
>>> I have some running right here…
>>
>> I have WRAP which runs perfectly fine with these changes.
>>

-- 
TMN

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] [RFC 4/6] x86: geode: shrink amount of default packages

2019-01-15 Thread Philip Prindeville
Inline

> On Jan 15, 2019, at 12:40 AM, Tomasz Maciej Nowak  wrote:
> 
> Hi Philip
> 
> W dniu 14.01.2019 o 21:10, Philip Prindeville pisze:
> 
>> Why is the hwclock being dropped everywhere?
> 
> It's provided by busybox and installed by default, also greping for its usage 
> in target/linux/x86/base-files returned nothing. Am I missing something, is 
> the full fledged one used somewhere or by something?


It’s used at least two places.

When you first get the board and bring it up and set the clock, since it’s not 
usually set correctly by the factory, and it’s used by the ntpd script to save 
the synchronized time when the system is taken down.


> 
>> And who says the geos2, net5501, or alix2 are obsolete?
> 
> They are not obsolete by any means, only the way to assign packages for 
> specific device changes. Please take a look at patch nr 2 in this RFC and 
> section adding target/linux/x86/image/geode.mk. There You have almost all 
> packages replicated with additional geos device profile. Now You'll get two 
> images, one generic, and one for geos. This distinction is here because with 
> old way of adding packages, the generic image would end up with lot of 
> useless, geos/dsl specific packages.


So why is there a Device/geos but none of the other SBC’s?

And why doesn’t the geos DEVICE_PACKAGES include -kmod-via-rhine, since it uses 
the kmod-8139cp instead?

-Philip



> 
>> I have some running right here…
> 
> I have WRAP which runs perfectly fine with these changes.
> 
> Regards, Tomasz
> 
> -- 
> TMN


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] [RFC 4/6] x86: geode: shrink amount of default packages

2019-01-14 Thread Tomasz Maciej Nowak
Hi Philip

W dniu 14.01.2019 o 21:10, Philip Prindeville pisze:

> Why is the hwclock being dropped everywhere?

It's provided by busybox and installed by default, also greping for its usage 
in target/linux/x86/base-files returned nothing. Am I missing something, is the 
full fledged one used somewhere or by something?

> And who says the geos2, net5501, or alix2 are obsolete?

They are not obsolete by any means, only the way to assign packages for 
specific device changes. Please take a look at patch nr 2 in this RFC and 
section adding target/linux/x86/image/geode.mk. There You have almost all 
packages replicated with additional geos device profile. Now You'll get two 
images, one generic, and one for geos. This distinction is here because with 
old way of adding packages, the generic image would end up with lot of useless, 
geos/dsl specific packages.

> I have some running right here…

I have WRAP which runs perfectly fine with these changes.

Regards, Tomasz

-- 
TMN

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] [RFC 4/6] x86: geode: shrink amount of default packages

2019-01-14 Thread Philip Prindeville
Why is the hwclock being dropped everywhere?

And who says the geos2, net5501, or alix2 are obsolete?

I have some running right here…


> On Jan 13, 2019, at 2:49 PM, Tomasz Maciej Nowak  wrote:
> 
> Drop excesive amount of default packages, instead, rely on packages
> specified in device definitions.
> 
> Signed-off-by: Tomasz Maciej Nowak 
> ---
> target/linux/x86/geode/target.mk | 12 +---
> 1 file changed, 1 insertion(+), 11 deletions(-)
> 
> diff --git a/target/linux/x86/geode/target.mk 
> b/target/linux/x86/geode/target.mk
> index 5473d2a32b..cd7afef944 100644
> --- a/target/linux/x86/geode/target.mk
> +++ b/target/linux/x86/geode/target.mk
> @@ -1,16 +1,6 @@
> BOARDNAME:=AMD Geode based systems
> FEATURES+=pci usb gpio
> -DEFAULT_PACKAGES += \
> - kmod-crypto-hw-geode kmod-crypto-cbc \
> - kmod-ath5k kmod-ath9k \
> - kmod-button-hotplug \
> - kmod-ledtrig-heartbeat kmod-ledtrig-gpio \
> - kmod-ledtrig-netdev hwclock wpad-basic
> -# Geos
> -DEFAULT_PACKAGES += \
> - soloscli linux-atm br2684ctl ppp-mod-pppoa pppdump pppstats \
> - hwclock flashrom tc kmod-pppoa kmod-8139cp kmod-mppe \
> - kmod-usb-ohci-pci kmod-hwmon-lm90
> +DEFAULT_PACKAGES+=kmod-button-hotplug
> 
> define Target/Description
>   Build firmware images for AMD Geode GX/LX based systems (net5501, alix, 
> geos)
> -- 
> 2.20.1
> 
> 
> ___
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[OpenWrt-Devel] [RFC 4/6] x86: geode: shrink amount of default packages

2019-01-13 Thread Tomasz Maciej Nowak
Drop excesive amount of default packages, instead, rely on packages
specified in device definitions.

Signed-off-by: Tomasz Maciej Nowak 
---
 target/linux/x86/geode/target.mk | 12 +---
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/target/linux/x86/geode/target.mk b/target/linux/x86/geode/target.mk
index 5473d2a32b..cd7afef944 100644
--- a/target/linux/x86/geode/target.mk
+++ b/target/linux/x86/geode/target.mk
@@ -1,16 +1,6 @@
 BOARDNAME:=AMD Geode based systems
 FEATURES+=pci usb gpio
-DEFAULT_PACKAGES += \
-   kmod-crypto-hw-geode kmod-crypto-cbc \
-   kmod-ath5k kmod-ath9k \
-   kmod-button-hotplug \
-   kmod-ledtrig-heartbeat kmod-ledtrig-gpio \
-   kmod-ledtrig-netdev hwclock wpad-basic
-# Geos
-DEFAULT_PACKAGES += \
-   soloscli linux-atm br2684ctl ppp-mod-pppoa pppdump pppstats \
-   hwclock flashrom tc kmod-pppoa kmod-8139cp kmod-mppe \
-   kmod-usb-ohci-pci kmod-hwmon-lm90
+DEFAULT_PACKAGES+=kmod-button-hotplug
 
 define Target/Description
Build firmware images for AMD Geode GX/LX based systems (net5501, alix, 
geos)
-- 
2.20.1


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel