FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 03-25-2009, 08:27 AM
Stefan Bader
 
Default UBUNTU: Copy header files for various kernel media driver

Ike Panhc wrote:
> Bug: #322732
>
> Copy header files because various kernel media driver need them.
> Header files under drivers/media/dvb/frontends/, drivers/media/video/ and
> drivers/media/dvb/dvb-core/ copied.
>
> Signed-off-by: Ike Panhc <ike.pan@canonical.com>
> ---
> debian/rules.d/3-binary-indep.mk | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/debian/rules.d/3-binary-indep.mk b/debian/rules.d/3-binary-indep.mk
> index a68b791..1ddafe3 100644
> --- a/debian/rules.d/3-binary-indep.mk
> +++ b/debian/rules.d/3-binary-indep.mk
> @@ -35,6 +35,9 @@ install-headers:
> ( -name 'Makefile*' -o -name 'Kconfig*' -o -name 'Kbuild*' -o
> -name '*.sh' -o -name '*.pl' -o -name '*.lds' )
> -print | cpio -pd --preserve-modification-time $(indep_hdrdir)
> + cp -a drivers/media/dvb/dvb-core/*.h $(indep_hdrdir)/drivers/media/dvb/dvb-core
> + cp -a drivers/media/video/*.h $(indep_hdrdir)/drivers/media/video
> + cp -a drivers/media/dvb/frontends/*.h $(indep_hdrdir)/drivers/media/dvb/frontends
> cp -a scripts include $(indep_hdrdir)
>
> srcpkg = linux-source-$(release)
Looks good to me, ACK

--

When all other means of communication fail, try words!



--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-25-2009, 12:39 PM
Andy Whitcroft
 
Default UBUNTU: Copy header files for various kernel media driver

On Wed, Mar 25, 2009 at 04:40:18PM +0800, Ike Panhc wrote:
> Bug: #322732
>
> Copy header files because various kernel media driver need them.
> Header files under drivers/media/dvb/frontends/, drivers/media/video/ and
> drivers/media/dvb/dvb-core/ copied.
>
> Signed-off-by: Ike Panhc <ike.pan@canonical.com>
> ---
> debian/rules.d/3-binary-indep.mk | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/debian/rules.d/3-binary-indep.mk b/debian/rules.d/3-binary-indep.mk
> index a68b791..1ddafe3 100644
> --- a/debian/rules.d/3-binary-indep.mk
> +++ b/debian/rules.d/3-binary-indep.mk
> @@ -35,6 +35,9 @@ install-headers:
> ( -name 'Makefile*' -o -name 'Kconfig*' -o -name 'Kbuild*' -o
> -name '*.sh' -o -name '*.pl' -o -name '*.lds' )
> -print | cpio -pd --preserve-modification-time $(indep_hdrdir)
> + cp -a drivers/media/dvb/dvb-core/*.h $(indep_hdrdir)/drivers/media/dvb/dvb-core
> + cp -a drivers/media/video/*.h $(indep_hdrdir)/drivers/media/video
> + cp -a drivers/media/dvb/frontends/*.h $(indep_hdrdir)/drivers/media/dvb/frontends
> cp -a scripts include $(indep_hdrdir)
>
> srcpkg = linux-source-$(release)

ACK

I did have a look at the ramifications of including all .h files in the
headers packages. Basically it added about 60MB to the image, ie near to
doubling it. If that isn't a mental ammount we might consider including
all of them.

Thoughts?

-apw

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-25-2009, 01:41 PM
Tim Gardner
 
Default UBUNTU: Copy header files for various kernel media driver

Andy Whitcroft wrote:
> On Wed, Mar 25, 2009 at 04:40:18PM +0800, Ike Panhc wrote:
>> Bug: #322732
>>
>> Copy header files because various kernel media driver need them.
>> Header files under drivers/media/dvb/frontends/, drivers/media/video/ and
>> drivers/media/dvb/dvb-core/ copied.
>>
>> Signed-off-by: Ike Panhc <ike.pan@canonical.com>
>> ---
>> debian/rules.d/3-binary-indep.mk | 3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/debian/rules.d/3-binary-indep.mk b/debian/rules.d/3-binary-indep.mk
>> index a68b791..1ddafe3 100644
>> --- a/debian/rules.d/3-binary-indep.mk
>> +++ b/debian/rules.d/3-binary-indep.mk
>> @@ -35,6 +35,9 @@ install-headers:
>> ( -name 'Makefile*' -o -name 'Kconfig*' -o -name 'Kbuild*' -o
>> -name '*.sh' -o -name '*.pl' -o -name '*.lds' )
>> -print | cpio -pd --preserve-modification-time $(indep_hdrdir)
>> + cp -a drivers/media/dvb/dvb-core/*.h $(indep_hdrdir)/drivers/media/dvb/dvb-core
>> + cp -a drivers/media/video/*.h $(indep_hdrdir)/drivers/media/video
>> + cp -a drivers/media/dvb/frontends/*.h $(indep_hdrdir)/drivers/media/dvb/frontends
>> cp -a scripts include $(indep_hdrdir)
>>
>> srcpkg = linux-source-$(release)
>
> ACK
>
> I did have a look at the ramifications of including all .h files in the
> headers packages. Basically it added about 60MB to the image, ie near to
> doubling it. If that isn't a mental ammount we might consider including
> all of them.
>
> Thoughts?
>
> -apw
>

I'm not particularly keen on the idea of adding a zillion unnecessary
header files to the package. Nearly all of the .h files that are _not_
under include tend to be for internal private use by their specific
device drivers, media header files notwithstanding (but that's a
different issue).

Anyway, I'm still waiting for the pull request from Ike.

rtg
--
Tim Gardner tim.gardner@canonical.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-25-2009, 01:56 PM
Ike Panhc
 
Default UBUNTU: Copy header files for various kernel media driver

Tim Gardner wrote:
> Andy Whitcroft wrote:
>> I did have a look at the ramifications of including all .h files in the
>> headers packages. Basically it added about 60MB to the image, ie near to
>> doubling it. If that isn't a mental ammount we might consider including
>> all of them.
>>
>> Thoughts?
>>
>> -apw
>>
>
> I'm not particularly keen on the idea of adding a zillion unnecessary
> header files to the package. Nearly all of the .h files that are _not_
> under include tend to be for internal private use by their specific
> device drivers, media header files notwithstanding (but that's a
> different issue).
>
> Anyway, I'm still waiting for the pull request from Ike.
>
> rtg

# du -s /usr/src/linux-headers-2.6.27-11/*
5476 /usr/src/linux-headers-2.6.27-11/arch
16 /usr/src/linux-headers-2.6.27-11/block
40 /usr/src/linux-headers-2.6.27-11/crypto
196 /usr/src/linux-headers-2.6.27-11/Documentation
4544 /usr/src/linux-headers-2.6.27-11/drivers
16 /usr/src/linux-headers-2.6.27-11/firmware
680 /usr/src/linux-headers-2.6.27-11/fs
28348 /usr/src/linux-headers-2.6.27-11/include
40 /usr/src/linux-headers-2.6.27-11/init
8 /usr/src/linux-headers-2.6.27-11/ipc
4 /usr/src/linux-headers-2.6.27-11/Kbuild
80 /usr/src/linux-headers-2.6.27-11/kernel
76 /usr/src/linux-headers-2.6.27-11/lib
60 /usr/src/linux-headers-2.6.27-11/Makefile
16 /usr/src/linux-headers-2.6.27-11/mm
768 /usr/src/linux-headers-2.6.27-11/net
36 /usr/src/linux-headers-2.6.27-11/samples
1488 /usr/src/linux-headers-2.6.27-11/scripts
72 /usr/src/linux-headers-2.6.27-11/security
816 /usr/src/linux-headers-2.6.27-11/sound
496 /usr/src/linux-headers-2.6.27-11/ubuntu
12 /usr/src/linux-headers-2.6.27-11/usr

Do we need the Kbuild and Makefile in linux-headers?
I think there is something we can remove.
I will check for that.

-- Ike Panhc

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-25-2009, 02:05 PM
Tim Gardner
 
Default UBUNTU: Copy header files for various kernel media driver

Ike Panhc wrote:
> Tim Gardner wrote:
>> Andy Whitcroft wrote:
>>> I did have a look at the ramifications of including all .h files in the
>>> headers packages. Basically it added about 60MB to the image, ie near to
>>> doubling it. If that isn't a mental ammount we might consider including
>>> all of them.
>>>
>>> Thoughts?
>>>
>>> -apw
>>>
>> I'm not particularly keen on the idea of adding a zillion unnecessary
>> header files to the package. Nearly all of the .h files that are _not_
>> under include tend to be for internal private use by their specific
>> device drivers, media header files notwithstanding (but that's a
>> different issue).
>>
>> Anyway, I'm still waiting for the pull request from Ike.
>>
>> rtg
>
> # du -s /usr/src/linux-headers-2.6.27-11/*
> 5476 /usr/src/linux-headers-2.6.27-11/arch
> 16 /usr/src/linux-headers-2.6.27-11/block
> 40 /usr/src/linux-headers-2.6.27-11/crypto
> 196 /usr/src/linux-headers-2.6.27-11/Documentation
> 4544 /usr/src/linux-headers-2.6.27-11/drivers
> 16 /usr/src/linux-headers-2.6.27-11/firmware
> 680 /usr/src/linux-headers-2.6.27-11/fs
> 28348 /usr/src/linux-headers-2.6.27-11/include
> 40 /usr/src/linux-headers-2.6.27-11/init
> 8 /usr/src/linux-headers-2.6.27-11/ipc
> 4 /usr/src/linux-headers-2.6.27-11/Kbuild
> 80 /usr/src/linux-headers-2.6.27-11/kernel
> 76 /usr/src/linux-headers-2.6.27-11/lib
> 60 /usr/src/linux-headers-2.6.27-11/Makefile
> 16 /usr/src/linux-headers-2.6.27-11/mm
> 768 /usr/src/linux-headers-2.6.27-11/net
> 36 /usr/src/linux-headers-2.6.27-11/samples
> 1488 /usr/src/linux-headers-2.6.27-11/scripts
> 72 /usr/src/linux-headers-2.6.27-11/security
> 816 /usr/src/linux-headers-2.6.27-11/sound
> 496 /usr/src/linux-headers-2.6.27-11/ubuntu
> 12 /usr/src/linux-headers-2.6.27-11/usr
>
> Do we need the Kbuild and Makefile in linux-headers?
> I think there is something we can remove.
> I will check for that.
>
> -- Ike Panhc
>

They are needed in some cases for doing external module builds.

--
Tim Gardner tim.gardner@canonical.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 10:08 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org