[PATCH 0/3] video: clps711x: New CLPS711X FB driver

[PATCH 0/3] video: clps711x: New CLPS711X FB driver

[Alexander Shiyan]

This series adds a new framebuffer driver for Cirrus Logic CLPS711X
CPUs. Since all code rewritten from scratch, patch is designed as a
replacement of the old (not updated for a long time) for a new one.
The second part of the patch affects changes for the ARM CLPS711X
core subsystem, so ACK from the ARM maintainers would be useful.
Patch originally designed for apply to the video subsystem, but I
have no prejudice it and if it will be applied to the arm-soc branch,
this will also be good.

Alexander Shiyan (3):
  video: clps711x: Add new Cirrus Logic CLPS711X framebuffer driver
  video: clps711x: Switch CLPS711X boards to use new FB driver
  video: clps711x: Add bindings documentation for CLPS711X framebuffer

 .../bindings/video/cirrus,clps711x-fb.txt          |  48 +++
 arch/arm/mach-clps711x/board-autcpu12.c            |   2 +-
 arch/arm/mach-clps711x/board-edb7211.c             |   2 +-
 arch/arm/mach-clps711x/board-p720t.c               |   2 +-
 arch/arm/mach-clps711x/devices.c                   |  17 +
 arch/arm/mach-clps711x/devices.h                   |   1 +
 drivers/video/fbdev/Kconfig                        |  18 +-
 drivers/video/fbdev/Makefile                       |   2 +-
 drivers/video/fbdev/clps711x-fb.c                  | 456 +++++++++++++++++++++
 drivers/video/fbdev/clps711xfb.c                   | 315 --------------
 10 files changed, 537 insertions(+), 326 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/video/cirrus,clps711x-fb.txt
 create mode 100644 drivers/video/fbdev/clps711x-fb.c
 delete mode 100644 drivers/video/fbdev/clps711xfb.c

-- 
1.8.3.2

Re: [PATCH 0/3] video: clps711x: New CLPS711X FB driver

[Tomi Valkeinen]

[-- Attachment #1: Type: text/plain, Size: 399 bytes --]

Hi,

On 20/03/14 18:24, Alexander Shiyan wrote:
> This series adds a new framebuffer driver for Cirrus Logic CLPS711X
> CPUs. Since all code rewritten from scratch, patch is designed as a
> replacement of the old (not updated for a long time) for a new one.

Why was it rewritten from scratch? It's almost always better to improve
the old one, than write a new, replacing one.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 0/3] video: clps711x: New CLPS711X FB driver

[Tomi Valkeinen]

[-- Attachment #1: Type: text/plain, Size: 1158 bytes --]

On 08/04/14 13:15, Alexander Shiyan wrote:
> Tue, 8 Apr 2014 12:56:24 +0300 от Tomi Valkeinen <tomi.valkeinen@ti.com>:
>> Hi,
>>
>> On 20/03/14 18:24, Alexander Shiyan wrote:
>>> This series adds a new framebuffer driver for Cirrus Logic CLPS711X
>>> CPUs. Since all code rewritten from scratch, patch is designed as a
>>> replacement of the old (not updated for a long time) for a new one.
>>
>> Why was it rewritten from scratch? It's almost always better to improve
>> the old one, than write a new, replacing one.
> Hello.
> 
> Most of the code has been rewritten. It will be very difficult to see the changes,
> if it is presented as a patch for the old driver.

Well, now it's impossible to see the changes, except opening both files
and going through line by line. You don't even list any new features in
the description, or give any reason why a new driver is required.

And of course one huge patch changing the old driver to the new form is
bad also.

The proper way would be to gradually change the old driver with a patch
series. Then it's possible to review the patches and see what is
actually changed.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]