[Buildroot] Size of svn

[Buildroot] Size of svn

[Ulf Samuelsson]

I looked through the svn.

The avr32 boards (target/device/Atmel/atngw100_<xxx>)
recently added by John Voltz, each have their own patchset, 
and the large patches are duplications of the patches in
(target/device/Atmel/arch-avr32/kernel-patches-<version>)
That adds 2-3 MB and should be cleaned up.

The major remaining part in terms of size in target/device/Atmel 
are kernel patches for at91 and avr32 for different kernel versions.
It would be possible to download these patches as well
from an ftpserver, but the infrastructure in "target/linux" is
not there yet. I think this is 5-6 MB.

==========================
The toolchain patches for avr32 added by John,
is IMHO unneccessary and can be removed. 
IIRC they add 7-8 MB to the svn size.

The avr32 toolchain can (and should)
be built using the prepatched source.
If patches are required, then they can be located in the
(target/device/Atmel/toolchain) directory.



Best Regards
Ulf Samuelsson                ulf at atmel.com
Atmel Nordic AB
Mail:  Box 2033, 174 02 Sundbyberg, Sweden
Visit:  Kavalleriv?gen 24, 174 58 Sundbyberg, Sweden
Phone +46 (8) 441 54 22     Fax +46 (8) 441 54 29
GSM    +46 (706) 22 44 57

Join the Atmel 2008 Microcontroller Seminar April 7-10.
Introducing X-Mega, AVR32 and ARM9. STK600 Kit included.
Register at Acte, Arrow or EBV webpage.

Technical support when I am not available:
AT90 AVR Applications Group: mailto:avr at atmel.com
AT91 ARM Applications Group: mailto:at91support at atmel.com
AVR32 Applications Group        mailto:avr32 at atmel.com
http://www.avrfreaks.net/;            http://avr32linux.org/
http://www.at91.com/ ;                 http://www.linux4sam.org/

[Buildroot] Size of svn

[Peter Korsgaard]

>>>>> "Ulf" == Ulf Samuelsson <ulf.samuelsson@atmel.com> writes:

 Ulf> I looked through the svn.

Thanks.

 Ulf> The avr32 boards (target/device/Atmel/atngw100_<xxx>)
 Ulf> recently added by John Voltz, each have their own patchset, 
 Ulf> and the large patches are duplications of the patches in
 Ulf> (target/device/Atmel/arch-avr32/kernel-patches-<version>)
 Ulf> That adds 2-3 MB and should be cleaned up.

Who will do that? John? You?

 Ulf> The major remaining part in terms of size in target/device/Atmel 
 Ulf> are kernel patches for at91 and avr32 for different kernel versions.
 Ulf> It would be possible to download these patches as well
 Ulf> from an ftpserver, but the infrastructure in "target/linux" is
 Ulf> not there yet. I think this is 5-6 MB.

 Ulf> ==========================
 Ulf> The toolchain patches for avr32 added by John,
 Ulf> is IMHO unneccessary and can be removed. 
 Ulf> IIRC they add 7-8 MB to the svn size.

It would be nice to get rid of those as well.

 Ulf> The avr32 toolchain can (and should)
 Ulf> be built using the prepatched source.
 Ulf> If patches are required, then they can be located in the
 Ulf> (target/device/Atmel/toolchain) directory.

-- 
Bye, Peter Korsgaard

[Buildroot] Size of svn

[Ulf Samuelsson]

>>>>>> "Ulf" == Ulf Samuelsson <ulf.samuelsson@atmel.com> writes:
> 
> Ulf> I looked through the svn.
> 
> Thanks.
> 
> Ulf> The avr32 boards (target/device/Atmel/atngw100_<xxx>)
> Ulf> recently added by John Voltz, each have their own patchset, 
> Ulf> and the large patches are duplications of the patches in
> Ulf> (target/device/Atmel/arch-avr32/kernel-patches-<version>)
> Ulf> That adds 2-3 MB and should be cleaned up.
> 
> Who will do that? John? You?

Since John introduced them, it is probably best
if John removes them after testing on his boards.

The result should be something which would use
"target/device/Atmel/arch-avr32/kernel-patches-<version>"
for general purpose AVR32 patches, and if some
board requires something special, they can reside in 
"target/device/Atmel/<board>/kernel-patches"
though I still think that it is better to have a single
repository for AVR32 kernel patches
and any patch which is board specific, should
#ifdef in the resulting source.

> 
> Ulf> The major remaining part in terms of size in target/device/Atmel 
> Ulf> are kernel patches for at91 and avr32 for different kernel versions.
> Ulf> It would be possible to download these patches as well
> Ulf> from an ftpserver, but the infrastructure in "target/linux" is
> Ulf> not there yet. I think this is 5-6 MB.
> 
> Ulf> ==========================
> Ulf> The toolchain patches for avr32 added by John,
> Ulf> is IMHO unneccessary and can be removed. 
> Ulf> IIRC they add 7-8 MB to the svn size.
> 
> It would be nice to get rid of those as well.

Again I think it is best that John gets rid of this.
Were they not moved to a subdirectory so they
are hanging loose at the moment?

> Ulf> The avr32 toolchain can (and should)
> Ulf> be built using the prepatched source.
> Ulf> If patches are required, then they can be located in the
> Ulf> (target/device/Atmel/toolchain) directory.
> 
> -- 
> Bye, Peter Korsgaard
>

[Buildroot] Size of svn

[Thiago A. Corrêa]

Howdy,

>  The avr32 boards (target/device/Atmel/atngw100_<xxx>)
>  recently added by John Voltz, each have their own patchset,
>  and the large patches are duplications of the patches in
>  (target/device/Atmel/arch-avr32/kernel-patches-<version>)
>  That adds 2-3 MB and should be cleaned up.

I agree, unfortunately, they are never applied if left there. There is
no code applying it anywhere. Where should we add this?
target/linux/Makefile.advanced.in doesn't seams right place to do it.
Where do you apply the arm counterparts? If they are applied at all.

>  The major remaining part in terms of size in target/device/Atmel
>  are kernel patches for at91 and avr32 for different kernel versions.
>  It would be possible to download these patches as well
>  from an ftpserver, but the infrastructure in "target/linux" is
>  not there yet. I think this is 5-6 MB.

At least Atmel is pushing those up into mainstream, so they tend to
shrink over time. But sometimes, we can't wait for the kernel
maintainers to release for us to get important fixes or add support to
existing hardware, such as is the case for the ISI. Not to mention
that they get more visibility... you know, the more eye balls thing ;)

>  ==========================
>  The toolchain patches for avr32 added by John,
>  is IMHO unneccessary and can be removed.
>  IIRC they add 7-8 MB to the svn size.
>
>  The avr32 toolchain can (and should)
>  be built using the prepatched source.
>  If patches are required, then they can be located in the
>  (target/device/Atmel/toolchain) directory.
>

I'm sorry to disagree. Having the choice is always welcome. The
external toolchain has, more than once I must add, felt short on us.
Also, for our despair, fix didn't came fast on those occasions,
despite reports. Not to mention that there is the issue of who gets to
update the prepatched sources if there is a critical patch? We don't
have access to the FTP to add patches or fix things.

We could instead download patches from avr32linux.org, at least being
a wiki things are bound to be more open and a solution to problems
might present itself faster, updates made easier, etc.

I completely agree with you that we should strive for a cleaner
solution to the patch proliferation. But it's not because of it's
size. You already need about 1 or 2 gigabytes to build a target.

Kind Regards,
    Thiago A. Correa