Re: [PATCH v4 11/16] drm: bridge: analogix/dp: add platform device type support

[Yakir Yang]

Hi Krzysztof,

å 09/07/2015 07:55 AM, Krzysztof Kozlowski åé:
> On 06.09.2015 13:07, Yakir Yang wrote:
> > Hi Krzysztof,
> >
> > å 09/04/2015 08:36 AM, Krzysztof Kozlowski åé:
> > > On 01.09.2015 15:07, Yakir Yang wrote:
> > >
> > > Empty commit message. Please explain here why you want to add platform
> > > device type support.
> > >
> > > Actually the title is confusing. You are not adding support for platform
> > > device types but rather adding a field containing type of device.
> > >
> > >
> > > > Signed-off-by: Yakir Yang <ykk@xxxxxxxxxxxxxx>
> > > > ---
> > > > Changes in v4: None
> > > > Changes in v3: None
> > > > Changes in v2:
> > > > - Add GNU license v2 declared and samsung copyright
> > > >
> > > >    drivers/gpu/drm/exynos/exynos_dp.c              |  1 +
> > > >    drivers/gpu/drm/rockchip/analogix_dp-rockchip.c |  1 +
> > > >    include/drm/bridge/analogix_dp.h                | 16 ++++++++++++++++
> > > >    3 files changed, 18 insertions(+)
> > > >
> > > > diff --git a/drivers/gpu/drm/exynos/exynos_dp.c
> > > > b/drivers/gpu/drm/exynos/exynos_dp.c
> > > > index 6060d2c..40ef727 100644
> > > > --- a/drivers/gpu/drm/exynos/exynos_dp.c
> > > > +++ b/drivers/gpu/drm/exynos/exynos_dp.c
> > > > @@ -224,6 +224,7 @@ static int exynos_dp_bind(struct device *dev,
> > > > struct device *master, void *data)
> > > >        dp->dev = dev;
> > > >        dp->drm_dev = drm_dev;
> > > >    +    dp->plat_data.dev_type = EXYNOS_DP;
> > > >        dp->plat_data.power_on = exynos_dp_poweron;
> > > >        dp->plat_data.power_off = exynos_dp_poweroff;
> > > >        dp->plat_data.get_modes = exynos_dp_get_modes;
> > > > diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
> > > > b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
> > > > index efea045..4934271 100644
> > > > --- a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
> > > > +++ b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
> > > > @@ -293,6 +293,7 @@ static int rockchip_dp_bind(struct device *dev,
> > > > struct device *master,
> > > >            return ret;
> > > >        }
> > > >    +    dp->plat_data.dev_type = RK3288_DP;
> > > >        dp->plat_data.attach = NULL;
> > > >        dp->plat_data.get_modes = NULL;
> > > >        dp->plat_data.power_on = rockchip_dp_poweron;
> > > > diff --git a/include/drm/bridge/analogix_dp.h
> > > > b/include/drm/bridge/analogix_dp.h
> > > > index 8b4ffad..7209a64 100644
> > > > --- a/include/drm/bridge/analogix_dp.h
> > > > +++ b/include/drm/bridge/analogix_dp.h
> > > > @@ -1,9 +1,25 @@
> > > > +/*
> > > > + * Analogix Core DP (Display Port) interface driver.
> > > > + *
> > > > + * Copyright (C) 2012 Samsung Electronics Co., Ltd.
> > > > + *
> > > > + * This program is free software; you can redistribute it and/or
> > > > modify it
> > > > + * under the terms of the GNU General Public License as published by
> > > > the
> > > > + * Free Software Foundation; either version 2 of the License, or (at
> > > > your
> > > > + * option) any later version.
> > > > + */
> > > Two questions:
> > > 1. Why this change is here? I would rather expect it at patch 3/16 when
> > > you created this file... That is the usual time for adding copyrights.
> > Yep, I should move this to 03/16.
> >
> > > 2. Does this file contains previous Samsung work?
> > Hmm... I think this haven't contains the previous Samsung works,
> > but I have the cover the head message from analogix_dp_core driver
> > (include the copyright, but without ant author).
> If it does not contain Samsung's work then don't mention its copyrights.

Okay,

- Yakir

> Best regards,
> Krzysztof
>
> > Thanks
> > - Yakir
> >
> > > Best regards,
> > > Krzysztof
> > >
> > > >    #ifndef _ANALOGIX_DP_H_
> > > >    #define _ANALOGIX_DP_H_
> > > >      #include <drm/drm_crtc.h>
> > > >    +enum analogix_dp_devtype {
> > > > +    EXYNOS_DP,
> > > > +    RK3288_DP,
> > > > +};
> > > > +
> > > >    struct analogix_dp_plat_data {
> > > > +    enum analogix_dp_devtype dev_type;
> > > >        struct drm_panel *panel;
> > > >          int (*power_on)(struct analogix_dp_plat_data *);


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/