[PATCH 0/3] Reorder drivers/video directory

2014-03-05 Thread Tomi Valkeinen 
On 04/03/14 21:21, Randy Dunlap wrote:

>> I have pushed this to my for-next branch. Let's see what happens... At
>> least I'm able to merge the current linux-next without any conflicts.
> 
> Thanks, I'm looking at this change in linux-next now.
> 
> EXYNOS_VIDEO seems to be a little bit odd.  Can you clarify that for me?
> (This is not a change that you introduced.)
> 
> 
> In particular, under Graphics support, select Framebuffer Devices.
> This lists:
>   Support for frame buffer devices -->
>   Exynos Video driver support
> 
> It appears to me that Exynos either is a Framebuffer Device and should depend
> on FB like the other drivers here do OR (actually XOR) it is not a frame 
> buffer
> device and it should not be listed here.
> 
> Then once that is cleared up :), we don't need 2 levels of menu to get to the
> list of FB drivers -- i.e., one of those levels can be removed.

There are others. For my config, I have:

{*} Support for frame buffer devices  --->
 OMAP2+ Display Subsystem support  --->
[ ] Exynos Video driver support  
< > Solomon SSD1307 framebuffer support

I didn't want to start fixing those at the moment, as I have no idea
about exynos or solomon, and I wanted to just try to do the reorder,
without any other changes.

I agree that there's something wrong with the items. For the OMAP DSS,
there are non-fbdev related items under that menu, used also by omapdrm.
So it should probably be split into different components.

> Oh, and if you keep the new menu item "Framebuffer Devices", please spell it
> like the other entry (Frame Buffer).

Ok, fixed.

> Other than those nits, I like this change very much.  Thanks.

Thanks. After pushing this to for-next, I'm getting compile error
reports from Fengguang and Stephen. Let's see if I manage to avoid
those... This is not the easiest change to manage.

 Tomi


-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL:

[PATCH 0/3] Reorder drivers/video directory

2014-03-04 Thread Randy Dunlap 
On 03/03/2014 12:45 AM, Tomi Valkeinen wrote:
> On 27/02/14 13:54, Tomi Valkeinen wrote:
>> Hi,
>>
>> This is a re-send of the series, with RFC removed from the subject, and a 
>> bunch
>> of acks added.
>>
>> I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
>> make sure this is not a bad idea, doomed to fail horribly.
>>
>> So this series creates a new directory, drivers/video/fbdev/, to which all
>> fbdev related files are moved. Also, a new directory, 
>> drivers/video/fbdev/core/
>> is created, to which the core fbdev framework files are moved. This makes the
>> drivers/video hierarchy much more clear.
>>
>> Presuming no one has objections to this as such, I wonder what's the least
>> painful way to merge this? Normally, like any other fbdev change? As a 
>> separate
>> pull request, maybe at -rc2 time frame, based on -rc1? Something else?
>>
>>  Tomi
>>
>> Tomi Valkeinen (3):
>>   video: move fbdev to drivers/video/fbdev
>>   fbdev: move fbdev core files to separate directory
>>   video: Kconfig: move drm and fb into separate menus
> 
> I have pushed this to my for-next branch. Let's see what happens... At
> least I'm able to merge the current linux-next without any conflicts.

Thanks, I'm looking at this change in linux-next now.

EXYNOS_VIDEO seems to be a little bit odd.  Can you clarify that for me?
(This is not a change that you introduced.)


In particular, under Graphics support, select Framebuffer Devices.
This lists:
  Support for frame buffer devices -->
  Exynos Video driver support

It appears to me that Exynos either is a Framebuffer Device and should depend
on FB like the other drivers here do OR (actually XOR) it is not a frame buffer
device and it should not be listed here.

Then once that is cleared up :), we don't need 2 levels of menu to get to the
list of FB drivers -- i.e., one of those levels can be removed.

Oh, and if you keep the new menu item "Framebuffer Devices", please spell it
like the other entry (Frame Buffer).


Other than those nits, I like this change very much.  Thanks.

-- 
~Randy

[PATCH 0/3] Reorder drivers/video directory

2014-03-04 Thread Daniel Vetter 
On Thu, Feb 27, 2014 at 01:54:43PM +0200, Tomi Valkeinen wrote:
> Hi,
> 
> This is a re-send of the series, with RFC removed from the subject, and a 
> bunch
> of acks added.
> 
> I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
> make sure this is not a bad idea, doomed to fail horribly.
> 
> So this series creates a new directory, drivers/video/fbdev/, to which all
> fbdev related files are moved. Also, a new directory, 
> drivers/video/fbdev/core/
> is created, to which the core fbdev framework files are moved. This makes the
> drivers/video hierarchy much more clear.
> 
> Presuming no one has objections to this as such, I wonder what's the least
> painful way to merge this? Normally, like any other fbdev change? As a 
> separate
> pull request, maybe at -rc2 time frame, based on -rc1? Something else?

I like this. Acked-by: Daniel Vetter 

> 
>  Tomi
> 
> Tomi Valkeinen (3):
>   video: move fbdev to drivers/video/fbdev
>   fbdev: move fbdev core files to separate directory
>   video: Kconfig: move drm and fb into separate menus
> 
>  drivers/Makefile   |4 +-
>  drivers/video/Kconfig  | 2487 
> +---
>  drivers/video/Makefile |  168 +-
>  drivers/video/{ => fbdev}/68328fb.c|0
>  drivers/video/fbdev/Kconfig| 2481 +++
>  drivers/video/fbdev/Makefile   |  153 ++
>  drivers/video/{ => fbdev}/acornfb.c|0
>  drivers/video/{ => fbdev}/acornfb.h|0
>  drivers/video/{ => fbdev}/amba-clcd.c  |0
>  drivers/video/{ => fbdev}/amifb.c  |0
>  drivers/video/{ => fbdev}/arcfb.c  |0
>  drivers/video/{ => fbdev}/arkfb.c  |0
>  drivers/video/{ => fbdev}/asiliantfb.c |0
>  drivers/video/{ => fbdev}/atafb.c  |0
>  drivers/video/{ => fbdev}/atafb.h  |0
>  drivers/video/{ => fbdev}/atafb_iplan2p2.c |0
>  drivers/video/{ => fbdev}/atafb_iplan2p4.c |0
>  drivers/video/{ => fbdev}/atafb_iplan2p8.c |0
>  drivers/video/{ => fbdev}/atafb_mfb.c  |0
>  drivers/video/{ => fbdev}/atafb_utils.h|0
>  drivers/video/{ => fbdev}/atmel_lcdfb.c|0
>  drivers/video/{ => fbdev}/aty/Makefile |0
>  drivers/video/{ => fbdev}/aty/ati_ids.h|0
>  drivers/video/{ => fbdev}/aty/aty128fb.c   |0
>  drivers/video/{ => fbdev}/aty/atyfb.h  |0
>  drivers/video/{ => fbdev}/aty/atyfb_base.c |0
>  drivers/video/{ => fbdev}/aty/mach64_accel.c   |0
>  drivers/video/{ => fbdev}/aty/mach64_ct.c  |0
>  drivers/video/{ => fbdev}/aty/mach64_cursor.c  |0
>  drivers/video/{ => fbdev}/aty/mach64_gx.c  |0
>  drivers/video/{ => fbdev}/aty/radeon_accel.c   |0
>  drivers/video/{ => fbdev}/aty/radeon_backlight.c   |0
>  drivers/video/{ => fbdev}/aty/radeon_base.c|0
>  drivers/video/{ => fbdev}/aty/radeon_i2c.c |0
>  drivers/video/{ => fbdev}/aty/radeon_monitor.c |0
>  drivers/video/{ => fbdev}/aty/radeon_pm.c  |0
>  drivers/video/{ => fbdev}/aty/radeonfb.h   |0
>  drivers/video/{ => fbdev}/au1100fb.c   |0
>  drivers/video/{ => fbdev}/au1100fb.h   |0
>  drivers/video/{ => fbdev}/au1200fb.c   |0
>  drivers/video/{ => fbdev}/au1200fb.h   |0
>  drivers/video/{ => fbdev}/auo_k1900fb.c|0
>  drivers/video/{ => fbdev}/auo_k1901fb.c|0
>  drivers/video/{ => fbdev}/auo_k190x.c  |0
>  drivers/video/{ => fbdev}/auo_k190x.h  |0
>  drivers/video/{ => fbdev}/bf537-lq035.c|0
>  drivers/video/{ => fbdev}/bf54x-lq043fb.c  |0
>  drivers/video/{ => fbdev}/bfin-lq035q1-fb.c|0
>  drivers/video/{ => fbdev}/bfin-t350mcqb-fb.c   |0
>  drivers/video/{ => fbdev}/bfin_adv7393fb.c |0
>  drivers/video/{ => fbdev}/bfin_adv7393fb.h |0
>  drivers/video/{ => fbdev}/broadsheetfb.c   |0
>  drivers/video/{ => fbdev}/bt431.h  |0
>  drivers/video/{ => fbdev}/bt455.h  |0
>  drivers/video/{ => fbdev}/bw2.c|0
>  drivers/video/{ => fbdev}/c2p.h|0
>  drivers/video/{ => fbdev}/c2p_core.h   |0
>  drivers/video/{ => fbdev}/c2p_iplan2.c |0
>  drivers/video/{ => fbdev}/c2p_planar.c |0
>  drivers/video/{ => fbdev}/carminefb.c  |0
>  drivers/video/{ => fbdev}/carminefb.h  |0
>  drivers/video/{ => fbdev}/carminefb_regs.h |0
>  drivers/video/{ => fbdev}/cg14.c   |0
>  drivers/video/{ =>

[PATCH 0/3] Reorder drivers/video directory

2014-03-03 Thread Tomi Valkeinen 
On 27/02/14 13:54, Tomi Valkeinen wrote:
> Hi,
> 
> This is a re-send of the series, with RFC removed from the subject, and a 
> bunch
> of acks added.
> 
> I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
> make sure this is not a bad idea, doomed to fail horribly.
> 
> So this series creates a new directory, drivers/video/fbdev/, to which all
> fbdev related files are moved. Also, a new directory, 
> drivers/video/fbdev/core/
> is created, to which the core fbdev framework files are moved. This makes the
> drivers/video hierarchy much more clear.
> 
> Presuming no one has objections to this as such, I wonder what's the least
> painful way to merge this? Normally, like any other fbdev change? As a 
> separate
> pull request, maybe at -rc2 time frame, based on -rc1? Something else?
> 
>  Tomi
> 
> Tomi Valkeinen (3):
>   video: move fbdev to drivers/video/fbdev
>   fbdev: move fbdev core files to separate directory
>   video: Kconfig: move drm and fb into separate menus

I have pushed this to my for-next branch. Let's see what happens... At
least I'm able to merge the current linux-next without any conflicts.

 Tomi


-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL:

[PATCH 0/3] Reorder drivers/video directory

2014-02-28 Thread Tomi Valkeinen 
On 27/02/14 20:16, Randy Dunlap wrote:
> On 02/27/2014 03:54 AM, Tomi Valkeinen wrote:
>> Hi,
>>
>> This is a re-send of the series, with RFC removed from the subject, and a 
>> bunch
>> of acks added.
>>
>> I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
>> make sure this is not a bad idea, doomed to fail horribly.
> 
> What does this patch apply to?  I can't get it to apply cleanly to
> -rc4 or Linus's current git tree.
> 
> I like what I see so far.

My branch was based on -rc2. Some minor fbdev fixes went in after that,
and they seem to break the apply. I think the drivers/video/Kconfig and
drivers/video/Makefile are the most problematic ones, any change there
will conflict badly.

I pushed my branch (still -rc2 based) to:

git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git
work/fb-reorder

 Tomi


-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL:

[PATCH 0/3] Reorder drivers/video directory

2014-02-28 Thread Geert Uytterhoeven 
On Fri, Feb 28, 2014 at 8:07 AM, Tomi Valkeinen  
wrote:
> On 27/02/14 20:16, Randy Dunlap wrote:
>> On 02/27/2014 03:54 AM, Tomi Valkeinen wrote:
>>> Hi,
>>>
>>> This is a re-send of the series, with RFC removed from the subject, and a 
>>> bunch
>>> of acks added.
>>>
>>> I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
>>> make sure this is not a bad idea, doomed to fail horribly.
>>
>> What does this patch apply to?  I can't get it to apply cleanly to
>> -rc4 or Linus's current git tree.
>>
>> I like what I see so far.
>
> My branch was based on -rc2. Some minor fbdev fixes went in after that,
> and they seem to break the apply. I think the drivers/video/Kconfig and
> drivers/video/Makefile are the most problematic ones, any change there
> will conflict badly.

So you want to do the move based on one of the later -rcs, to avoid conflicits.
-rc5 may be too late, but it's up to Linus to decide.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 
linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

[PATCH 0/3] Reorder drivers/video directory

2014-02-27 Thread Tomi Valkeinen 
Hi,

This is a re-send of the series, with RFC removed from the subject, and a bunch
of acks added.

I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
make sure this is not a bad idea, doomed to fail horribly.

So this series creates a new directory, drivers/video/fbdev/, to which all
fbdev related files are moved. Also, a new directory, drivers/video/fbdev/core/
is created, to which the core fbdev framework files are moved. This makes the
drivers/video hierarchy much more clear.

Presuming no one has objections to this as such, I wonder what's the least
painful way to merge this? Normally, like any other fbdev change? As a separate
pull request, maybe at -rc2 time frame, based on -rc1? Something else?

 Tomi

Tomi Valkeinen (3):
  video: move fbdev to drivers/video/fbdev
  fbdev: move fbdev core files to separate directory
  video: Kconfig: move drm and fb into separate menus

 drivers/Makefile   |4 +-
 drivers/video/Kconfig  | 2487 +---
 drivers/video/Makefile |  168 +-
 drivers/video/{ => fbdev}/68328fb.c|0
 drivers/video/fbdev/Kconfig| 2481 +++
 drivers/video/fbdev/Makefile   |  153 ++
 drivers/video/{ => fbdev}/acornfb.c|0
 drivers/video/{ => fbdev}/acornfb.h|0
 drivers/video/{ => fbdev}/amba-clcd.c  |0
 drivers/video/{ => fbdev}/amifb.c  |0
 drivers/video/{ => fbdev}/arcfb.c  |0
 drivers/video/{ => fbdev}/arkfb.c  |0
 drivers/video/{ => fbdev}/asiliantfb.c |0
 drivers/video/{ => fbdev}/atafb.c  |0
 drivers/video/{ => fbdev}/atafb.h  |0
 drivers/video/{ => fbdev}/atafb_iplan2p2.c |0
 drivers/video/{ => fbdev}/atafb_iplan2p4.c |0
 drivers/video/{ => fbdev}/atafb_iplan2p8.c |0
 drivers/video/{ => fbdev}/atafb_mfb.c  |0
 drivers/video/{ => fbdev}/atafb_utils.h|0
 drivers/video/{ => fbdev}/atmel_lcdfb.c|0
 drivers/video/{ => fbdev}/aty/Makefile |0
 drivers/video/{ => fbdev}/aty/ati_ids.h|0
 drivers/video/{ => fbdev}/aty/aty128fb.c   |0
 drivers/video/{ => fbdev}/aty/atyfb.h  |0
 drivers/video/{ => fbdev}/aty/atyfb_base.c |0
 drivers/video/{ => fbdev}/aty/mach64_accel.c   |0
 drivers/video/{ => fbdev}/aty/mach64_ct.c  |0
 drivers/video/{ => fbdev}/aty/mach64_cursor.c  |0
 drivers/video/{ => fbdev}/aty/mach64_gx.c  |0
 drivers/video/{ => fbdev}/aty/radeon_accel.c   |0
 drivers/video/{ => fbdev}/aty/radeon_backlight.c   |0
 drivers/video/{ => fbdev}/aty/radeon_base.c|0
 drivers/video/{ => fbdev}/aty/radeon_i2c.c |0
 drivers/video/{ => fbdev}/aty/radeon_monitor.c |0
 drivers/video/{ => fbdev}/aty/radeon_pm.c  |0
 drivers/video/{ => fbdev}/aty/radeonfb.h   |0
 drivers/video/{ => fbdev}/au1100fb.c   |0
 drivers/video/{ => fbdev}/au1100fb.h   |0
 drivers/video/{ => fbdev}/au1200fb.c   |0
 drivers/video/{ => fbdev}/au1200fb.h   |0
 drivers/video/{ => fbdev}/auo_k1900fb.c|0
 drivers/video/{ => fbdev}/auo_k1901fb.c|0
 drivers/video/{ => fbdev}/auo_k190x.c  |0
 drivers/video/{ => fbdev}/auo_k190x.h  |0
 drivers/video/{ => fbdev}/bf537-lq035.c|0
 drivers/video/{ => fbdev}/bf54x-lq043fb.c  |0
 drivers/video/{ => fbdev}/bfin-lq035q1-fb.c|0
 drivers/video/{ => fbdev}/bfin-t350mcqb-fb.c   |0
 drivers/video/{ => fbdev}/bfin_adv7393fb.c |0
 drivers/video/{ => fbdev}/bfin_adv7393fb.h |0
 drivers/video/{ => fbdev}/broadsheetfb.c   |0
 drivers/video/{ => fbdev}/bt431.h  |0
 drivers/video/{ => fbdev}/bt455.h  |0
 drivers/video/{ => fbdev}/bw2.c|0
 drivers/video/{ => fbdev}/c2p.h|0
 drivers/video/{ => fbdev}/c2p_core.h   |0
 drivers/video/{ => fbdev}/c2p_iplan2.c |0
 drivers/video/{ => fbdev}/c2p_planar.c |0
 drivers/video/{ => fbdev}/carminefb.c  |0
 drivers/video/{ => fbdev}/carminefb.h  |0
 drivers/video/{ => fbdev}/carminefb_regs.h |0
 drivers/video/{ => fbdev}/cg14.c   |0
 drivers/video/{ => fbdev}/cg3.c|0
 drivers/video/{ => fbdev}/cg6.c|0
 drivers/video/{ => fbdev}/chipsfb.c|0
 drivers/video/{ => fbdev}/cirrusfb.c   |0
 drivers/video/{ => fbdev}/clps711xfb.c |0
 drivers/video/{

[PATCH 0/3] Reorder drivers/video directory

2014-02-27 Thread Randy Dunlap 
On 02/27/2014 03:54 AM, Tomi Valkeinen wrote:
> Hi,
> 
> This is a re-send of the series, with RFC removed from the subject, and a 
> bunch
> of acks added.
> 
> I'm cc'ing more people, to make sure this doesn't come as a surprise, and to
> make sure this is not a bad idea, doomed to fail horribly.

What does this patch apply to?  I can't get it to apply cleanly to
-rc4 or Linus's current git tree.

I like what I see so far.

Thanks.

> So this series creates a new directory, drivers/video/fbdev/, to which all
> fbdev related files are moved. Also, a new directory, 
> drivers/video/fbdev/core/
> is created, to which the core fbdev framework files are moved. This makes the
> drivers/video hierarchy much more clear.
> 
> Presuming no one has objections to this as such, I wonder what's the least
> painful way to merge this? Normally, like any other fbdev change? As a 
> separate
> pull request, maybe at -rc2 time frame, based on -rc1? Something else?


-- 
~Randy