public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Minkyu Kang <mk7.kang@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 00/25] exynos: video: Convert exynos LCD driver to use driver model
Date: Tue, 17 May 2016 13:38:09 +0900	[thread overview]
Message-ID: <573AA031.7010503@samsung.com> (raw)
In-Reply-To: <573A9C6F.4090508@samsung.com>

Hi,

On 17/05/16 13:22, Jaehoon Chung wrote:
> Hi Simon.
> 
> On 05/15/2016 06:22 AM, Simon Glass wrote:
>> Hi Jaehoon,
>>
>> On 13 May 2016 at 07:09, Jaehoon Chung <jh80.chung@samsung.com> wrote:
>>> Hi Simon,
>>>
>>> On 05/09/2016 08:00 PM, Minkyu Kang wrote:
>>>> Hi Jaehoon,
>>>>
>>>> On 08/05/16 06:25, Simon Glass wrote:
>>>>> Hi,
>>>>>
>>>>> On 1 March 2016 at 18:51, Minkyu Kang <mk7.kang@samsung.com> wrote:
>>>>>> Hi,
>>>>>>
>>>>>> On 29/02/16 16:18, Jaehoon Chung wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> On 02/29/2016 01:23 PM, Simon Glass wrote:
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> On 23 February 2016 at 05:04, Minkyu Kang <mk7.kang@samsung.com> wrote:
>>>>>>>>> Dear Simon Glass,
>>>>>>>>>
>>>>>>>>> On 23/02/16 15:42, Simon Glass wrote:
>>>>>>>>>> Hi Minkyu,
>>>>>>>>>>
>>>>>>>>>> On 21 February 2016 at 21:08, Simon Glass <sjg@chromium.org> wrote:
>>>>>>>>>>>
>>>>>>>>>>> This series converts the exynos LCD driver to work with driver model. Only
>>>>>>>>>>> the eDP display is converted - the MIPI display is left mostly alone as I
>>>>>>>>>>> do not have a device to test with.
>>>>>>>>>>>
>>>>>>>>>>> The conversion involves some refactoring to simplify the code, reduce the
>>>>>>>>>>> amount of global/static data and allow boards to be built without LCD
>>>>>>>>>>> support.
>>>>>>>>>>>
>>>>>>>>>>> The series is tested with snow, spring, pit and pi. It disables LCD on other
>>>>>>>>>>> exynos boards so that they will still build with these changes. Further work
>>>>>>>>>>> will be needed (by someone with a board) to convert the others. I am not
>>>>>>>>>>> sure when this series can be applied.
>>>>>>>>>>>
>>>>>>>>>>> The series also brings in device tree features from Linux v4.4. It would be
>>>>>>>>>>> useful to do a full sync with the kernel on these files, but this is really
>>>>>>>>>>> a job for the Samsung maintainer :-)
>>>>>>>>>>>
>>>>>>>>>>> The series is available at u-boot-dm/rkg-working.
>>>>>>>>>>>
>>>>>>>>>>> Changes in v2:
>>>>>>>>>>> - Rebase to master
>>>>>>>>>>> - Rebase to master
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> What do you think we should do about applying this series?
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Although we have to do remaining work, I think applying is possible without next work.
>>>>>>>>>
>>>>>>>>> Board maintainers, what do you think?
>>>>>>>>>
>>>>>>>>> Remaining Work:
>>>>>>>>> - MIPI display driver
>>>>>>>>> - s5pc210_universal
>>>>>>>>> - smdk5420
>>>>>>>>> - smdk5250
>>>>>>>>> - trats
>>>>>>>>> - trats2
>>>>>>>>>
>>>>>>>>> Thanks,
>>>>>>>>> Minkyu Kang.
>>>>>>>>
>>>>>>>> Well, no replies as yet. The main work requires is MIPI, but as I
>>>>>>>> don't have the hardware, I can't help with that. At least I have made
>>>>>>>> sure that everything builds OK.
>>>>>>>
>>>>>>> Sorry for late..I will check these patches on this week..Tomorrow is holiday..
>>>>>>> So I can check from Wed.
>>>>>>
>>>>>> Thank you, Jaehoon.
>>>>>
>>>>> It has been a few months now - if you like I can apply these via the -dm tree.
>>>
>>> Yep, I checked these..It needs to remain the future work.
>>> But it seems that it's not problem to apply at -dm repository.
>>> (I have checked with your patch on other boadrs.)
>>>
>>> I needs to check one more things..Could you wait until next Tuesday?
>>>
>>
>> Sure, it's been 3 months so another week won't matter :-)
> 
> Sorry. I have checked more with these patches..We have to work too many things in future for remaining work.
> But it seems those task are our role..If you and Minkyu are ok, you can apply these at dm repository.

I'm OK.
If you want, those can be merged to samsung tree.

> 
> I will make a plan to work. (Based on your repository.)
> 
> Actually, i didn't follow a few month, so i'm looking for u-boot history. Sorry. :(
> If you want to add my Reviewed-by tag, then you can add.
> 
> In future work,
> Change to driver model for above targets. (trats2/trats...)
> MIPI display driver..(I will assign to my colleague for this task.)
> 

Thank you!

Minkyu Kang.

  reply	other threads:[~2016-05-17  4:38 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-22  4:08 [U-Boot] [PATCH v2 00/25] exynos: video: Convert exynos LCD driver to use driver model Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 01/25] exynos: video: Move driver files into their own directory Simon Glass
2016-02-22 13:59   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 02/25] exynos: video: Drop dead code Simon Glass
2016-02-22 14:05   ` Anatolij Gustschin
2016-02-22 14:08   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 03/25] exynos: video: Remove use of vidinfo_t typedef Simon Glass
2016-02-22 14:09   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 04/25] exynos: video: Drop the static lcd_base_addr variable Simon Glass
2016-02-22 14:14   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 05/25] exynos: video: Drop static variables in exynos_fimd.c Simon Glass
2016-02-22 14:23   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 06/25] exynos: video: Drop static variables in exynos_fb.c Simon Glass
2016-02-22 14:24   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 07/25] exynos: video: Drop static variables in exynos_dp_lowlevel.c Simon Glass
2016-02-22 15:58   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 08/25] exynos: video: Move dsim_config_dt into a function Simon Glass
2016-02-22 16:02   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 09/25] exynos: video: Move struct exynos_platform_mipi_dsim into vidinfo Simon Glass
2016-02-22 16:05   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 10/25] exynos: video: Move mipi_lcd_device_dt into a function Simon Glass
2016-02-22 16:08   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 11/25] exynos: video: Combine LCD driver into one file Simon Glass
2016-02-22 16:12   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 12/25] exynos: pwm: Add a driver for the exynos5 PWM Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 13/25] video: Add an enum for active low/high Simon Glass
2016-02-22 16:13   ` Anatolij Gustschin
2016-02-22  4:08 ` [U-Boot] [PATCH v2 14/25] exynos: dts: Add pwm device tree node Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 15/25] exynos: Allow tizen to be built without an LCD Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 16/25] exynos: Allow CONFIG_MISC_COMMON to be build " Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 17/25] exynos: Disable LCD display for boards we can't convert Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 18/25] dts: Add clock and regulator binding files for max77802 Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 19/25] exynos: Allow PWM0 pinmux to be set up Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 20/25] exynos: Simplify calling of exynos_dp_phy_ctrl() Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 21/25] exynos: dts: Add display-related device tree fragments Simon Glass
2016-02-22  4:08 ` [U-Boot] [PATCH v2 22/25] exynos: video: Rename edp_device_info to exynos_dp_priv Simon Glass
2016-02-22 16:15   ` Anatolij Gustschin
2016-02-22  4:09 ` [U-Boot] [PATCH v2 23/25] exynos: video: Rename variables for driver model Simon Glass
2016-02-22 16:45   ` Anatolij Gustschin
2016-02-22  4:09 ` [U-Boot] [PATCH v2 24/25] exynos: video: Convert several boards to driver model for video Simon Glass
2016-02-22 16:58   ` Anatolij Gustschin
2016-02-22  4:09 ` [U-Boot] [PATCH v2 25/25] exynos: video: Drop old unused code Simon Glass
2016-02-22 17:00   ` Anatolij Gustschin
2016-02-23  6:42 ` [U-Boot] [PATCH v2 00/25] exynos: video: Convert exynos LCD driver to use driver model Simon Glass
2016-02-23 12:04   ` Minkyu Kang
2016-02-29  4:23     ` Simon Glass
2016-02-29  7:18       ` Jaehoon Chung
2016-03-02  1:51         ` Minkyu Kang
2016-05-07 21:25           ` Simon Glass
2016-05-09 11:00             ` Minkyu Kang
2016-05-10  1:23               ` Jaehoon Chung
2016-05-13 13:09               ` Jaehoon Chung
2016-05-14 21:22                 ` Simon Glass
2016-05-17  4:22                   ` Jaehoon Chung
2016-05-17  4:38                     ` Minkyu Kang [this message]
2016-05-23 20:18                       ` Simon Glass
2016-05-24  1:35                         ` Jaehoon Chung
2016-05-26  1:30                           ` Minkyu Kang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=573AA031.7010503@samsung.com \
    --to=mk7.kang@samsung.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox