Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1232071

Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging

From Tomi Valkeinen <tomi.valkeinen@ti.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging
Date 2015-09-24 14:00 +0200
Message-ID <qcdlU-27W-25@gated-at.bofh.it> (permalink)
References <pNrBL-1ny-5@gated-at.bofh.it> <q3TNp-7Nw-11@gated-at.bofh.it> <q3Ugq-8lN-15@gated-at.bofh.it> <q4eRQ-4a6-7@gated-at.bofh.it> <q4fEe-5kp-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On 02/09/15 15:48, Sudip Mukherjee wrote:

> Now I am getting confused. :(
> Since this has already been merged I guess we need to maintain it now.

Oh, ok. I thought it was still in staging. I haven't been able to follow
the list properly lately...

Well, in theory we could still revert it, as it's only -rc2.

> So then should I fix the things you pointed out or should i instead
> give more priority to the DRM driver and fix these things later?

Is there a public datasheet for the HW?

The driver as it is now is quite horrible. Huge tables of raw data,
written directly to the registers. Lots of magic numbers all around.
Even with a datasheet, presuming it's public and it contains all those
values, the driver is rather unmaintainable.

All those need to be sorted out for the DRM driver anyway, so maybe it's
better to clean up the fb driver first.

> And, just to inform you, there are two more fbdev drivers in staging,
> staging/sm750fb and staging/fbtft. And the ultimate goal of any driver

Thanks for pointing this out, I need to check those and think how to
proceed.

 Tomi

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-24 14:00 +0200
  Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-25 15:50 +0200
    Re: [PATCH v2 1/3] staging: sm7xxfb: move sm712fb out of staging Mike Rapoport <mike.rapoport@gmail.com> - 2015-09-27 06:50 +0200

csiph-web