Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1634073 > unrolled thread
| Started by | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| First post | 2017-05-02 03:40 +0200 |
| Last post | 2017-05-02 05:00 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
Large new drm driver Stephen Rothwell <sfr@canb.auug.org.au> - 2017-05-02 03:40 +0200
Re: Large new drm driver Dave Airlie <airlied@gmail.com> - 2017-05-02 04:40 +0200
Re: Large new drm driver Stephen Rothwell <sfr@canb.auug.org.au> - 2017-05-02 05:00 +0200
| From | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| Date | 2017-05-02 03:40 +0200 |
| Subject | Large new drm driver |
| Message-ID | <tCuXf-8a7-1@gated-at.bofh.it> |
Hi Linus, Just so you don't get a large suprise: $ git diff --dirstat v4.11..next-20170501 3.9% arch/ 13.5% drivers/gpu/drm/amd/include/asic_reg/vega10/DC/ 6.0% drivers/gpu/drm/amd/include/asic_reg/vega10/GC/ 23.5% drivers/gpu/drm/amd/include/asic_reg/vega10/NBIO/ 7.5% drivers/gpu/drm/amd/include/asic_reg/vega10/ 4.9% drivers/gpu/drm/ 4.6% drivers/net/ 5.8% drivers/staging/media/atomisp/pci/atomisp2/css2400/ 3.0% drivers/staging/media/ 4.5% drivers/staging/rtl8723bs/ 12.6% drivers/ There are some very large include files in the new drm driver. wc -l over all the files in drivers/gpu/drm/amd/include/asic_reg/vega10/: 361003 total -- Cheers, Stephen Rothwell
[toc] | [next] | [standalone]
| From | Dave Airlie <airlied@gmail.com> |
|---|---|
| Date | 2017-05-02 04:40 +0200 |
| Message-ID | <tCvTj-o4-1@gated-at.bofh.it> |
| In reply to | #1634073 |
On 2 May 2017 at 11:31, Stephen Rothwell <sfr@canb.auug.org.au> wrote: > Hi Linus, > > Just so you don't get a large suprise: > > $ git diff --dirstat v4.11..next-20170501 > 3.9% arch/ > 13.5% drivers/gpu/drm/amd/include/asic_reg/vega10/DC/ > 6.0% drivers/gpu/drm/amd/include/asic_reg/vega10/GC/ > 23.5% drivers/gpu/drm/amd/include/asic_reg/vega10/NBIO/ > 7.5% drivers/gpu/drm/amd/include/asic_reg/vega10/ > 4.9% drivers/gpu/drm/ > 4.6% drivers/net/ > 5.8% drivers/staging/media/atomisp/pci/atomisp2/css2400/ > 3.0% drivers/staging/media/ > 4.5% drivers/staging/rtl8723bs/ > 12.6% drivers/ > > There are some very large include files in the new drm driver. > > wc -l over all the files in > drivers/gpu/drm/amd/include/asic_reg/vega10/: We've had register header files like this on a number of occasions before, not sure why it warrants a mention now. It's just the public header files for the new Vega AMD GPUs, as opposed to a whole new driver. AMD release the validated headers so open source developers don't have to go hunting for registers. Dave.
[toc] | [prev] | [next] | [standalone]
| From | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| Date | 2017-05-02 05:00 +0200 |
| Message-ID | <tCwcF-us-1@gated-at.bofh.it> |
| In reply to | #1634086 |
Hi Dave, On Tue, 2 May 2017 12:36:35 +1000 Dave Airlie <airlied@gmail.com> wrote: > > We've had register header files like this on a number of occasions > before, not sure why it warrants a mention now. Linus expressed suprise once before when we had a large number of new lines appear in another subsystem ... > It's just the public header files for the new Vega AMD GPUs, > as opposed to a whole new driver. > > AMD release the validated headers so open source developers > don't have to go hunting for registers. It's all fine, just noting. -- Cheers, Stephen Rothwell
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web