2012/8/20 InKi Dae inki.dae@samsung.com:
2012/8/20 Joonyoung Shim jy0922.shim@samsung.com:
On 08/20/2012 03:17 PM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.shim@samsung.com:
On 08/20/2012 11:29 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.shim@samsung.com:
On 08/17/2012 06:50 PM, Inki Dae wrote: > > this patch changes ctx variable name in exynos_drm_hdmi_context > structure to client because the use of ctx variable makes it confused.
I don't prefer "client" name. This is not client and server relationship.
Okay, give me your opinion. which one do you prefer?
Just "data".
It's not clear. "data" is so comprehensive just use "child_ctx". we already use "parent_ctx" as the context of exynos_drm_hdmi module.
Actually, i prefer "data" term than "ctx" term. Anyway i want to focus why this problem is occurred because i think that codes are some ugly. I think we first need to consider to solve this situation using code redesign, structure change and etc.
calm down and focus on the use of term. this is so minor and this is not related to design issue.
and it's not important whether you prefer this term or not. if the use of "data" term is logical enough then we will use it otherwise "child_ctx" so give me your comment why you don't prefer "child_ctx" term. actually, now hdmi codes use "hdata" term as context's instance but other use "ctx" term. as I mentioned before, I gave you my opinion that "data" term is so comprehensive so I DON'T PREFER "data" term.
Thanks.
Thanks.
> Signed-off-by: Inki Dae inki.dae@samsung.com > Signed-off-by: Kyungmin Park kyungmin.park@samsung.com > --- > drivers/gpu/drm/exynos/exynos_drm_hdmi.c | 38 > +++++++++++++++--------------- > drivers/gpu/drm/exynos/exynos_drm_hdmi.h | 4 +- > drivers/gpu/drm/exynos/exynos_hdmi.c | 12 ++++---- > drivers/gpu/drm/exynos/exynos_mixer.c | 8 +++--- > 4 files changed, 31 insertions(+), 31 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_hdmi.c > b/drivers/gpu/drm/exynos/exynos_drm_hdmi.c > index 3fdf0b6..bced38e 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_hdmi.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_hdmi.c > @@ -64,7 +64,7 @@ static bool drm_hdmi_is_connected(struct device > *dev) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->is_connected) > - return hdmi_ops->is_connected(ctx->hdmi_ctx->ctx); > + return hdmi_ops->is_connected(ctx->hdmi_ctx->client); > return false; > } > @@ -77,8 +77,8 @@ static int drm_hdmi_get_edid(struct device *dev, > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->get_edid) > - return hdmi_ops->get_edid(ctx->hdmi_ctx->ctx, > connector, > edid, > - len); > + return hdmi_ops->get_edid(ctx->hdmi_ctx->client, > connector, > + edid, len); > return 0; > } > @@ -90,7 +90,7 @@ static int drm_hdmi_check_timing(struct device *dev, > void *timing) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->check_timing) > - return hdmi_ops->check_timing(ctx->hdmi_ctx->ctx, > timing); > + return hdmi_ops->check_timing(ctx->hdmi_ctx->client, > timing); > return 0; > } > @@ -102,7 +102,7 @@ static int drm_hdmi_power_on(struct device *dev, > int > mode) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->power_on) > - return hdmi_ops->power_on(ctx->hdmi_ctx->ctx, mode); > + return hdmi_ops->power_on(ctx->hdmi_ctx->client, > mode); > return 0; > } > @@ -124,7 +124,7 @@ static int drm_hdmi_enable_vblank(struct device > *subdrv_dev) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (mixer_ops && mixer_ops->enable_vblank) > - return mixer_ops->enable_vblank(ctx->mixer_ctx->ctx, > + return > mixer_ops->enable_vblank(ctx->mixer_ctx->client, > manager->pipe); > return 0; > @@ -137,12 +137,12 @@ static void drm_hdmi_disable_vblank(struct > device > *subdrv_dev) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (mixer_ops && mixer_ops->disable_vblank) > - return mixer_ops->disable_vblank(ctx->mixer_ctx->ctx); > + return > mixer_ops->disable_vblank(ctx->mixer_ctx->client); > } > static void drm_hdmi_mode_fixup(struct device *subdrv_dev, > struct drm_connector *connector, > - const struct drm_display_mode *mode, > + struct drm_display_mode *mode, > struct drm_display_mode > *adjusted_mode) > { > struct drm_hdmi_context *ctx = to_context(subdrv_dev); > @@ -150,7 +150,7 @@ static void drm_hdmi_mode_fixup(struct device > *subdrv_dev, > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->mode_fixup) > - hdmi_ops->mode_fixup(ctx->hdmi_ctx->ctx, connector, > mode, > + hdmi_ops->mode_fixup(ctx->hdmi_ctx->client, connector, > mode, > adjusted_mode); > } > @@ -161,7 +161,7 @@ static void drm_hdmi_mode_set(struct device > *subdrv_dev, void *mode) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->mode_set) > - hdmi_ops->mode_set(ctx->hdmi_ctx->ctx, mode); > + hdmi_ops->mode_set(ctx->hdmi_ctx->client, mode); > } > static void drm_hdmi_get_max_resol(struct device *subdrv_dev, > @@ -172,7 +172,7 @@ static void drm_hdmi_get_max_resol(struct device > *subdrv_dev, > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->get_max_resol) > - hdmi_ops->get_max_resol(ctx->hdmi_ctx->ctx, width, > height); > + hdmi_ops->get_max_resol(ctx->hdmi_ctx->client, width, > height); > } > static void drm_hdmi_commit(struct device *subdrv_dev) > @@ -182,7 +182,7 @@ static void drm_hdmi_commit(struct device > *subdrv_dev) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (hdmi_ops && hdmi_ops->commit) > - hdmi_ops->commit(ctx->hdmi_ctx->ctx); > + hdmi_ops->commit(ctx->hdmi_ctx->client); > } > static void drm_hdmi_dpms(struct device *subdrv_dev, int mode) > @@ -192,10 +192,10 @@ static void drm_hdmi_dpms(struct device > *subdrv_dev, > int mode) > DRM_DEBUG_KMS("%s\n", __FILE__); > if (mixer_ops && mixer_ops->dpms) > - mixer_ops->dpms(ctx->mixer_ctx->ctx, mode); > + mixer_ops->dpms(ctx->mixer_ctx->client, mode); > if (hdmi_ops && hdmi_ops->dpms) > - hdmi_ops->dpms(ctx->hdmi_ctx->ctx, mode); > + hdmi_ops->dpms(ctx->hdmi_ctx->client, mode); > } > static void drm_hdmi_apply(struct device *subdrv_dev) > @@ -209,11 +209,11 @@ static void drm_hdmi_apply(struct device > *subdrv_dev) > if (!ctx->enabled[i]) > continue; > if (mixer_ops && mixer_ops->win_commit) > - mixer_ops->win_commit(ctx->mixer_ctx->ctx, i); > + mixer_ops->win_commit(ctx->mixer_ctx->client, > i); > } > if (hdmi_ops && hdmi_ops->commit) > - hdmi_ops->commit(ctx->hdmi_ctx->ctx); > + hdmi_ops->commit(ctx->hdmi_ctx->client); > } > static struct exynos_drm_manager_ops drm_hdmi_manager_ops = { > @@ -235,7 +235,7 @@ static void drm_mixer_mode_set(struct device > *subdrv_dev, > DRM_DEBUG_KMS("%s\n", __FILE__); > if (mixer_ops && mixer_ops->win_mode_set) > - mixer_ops->win_mode_set(ctx->mixer_ctx->ctx, overlay); > + mixer_ops->win_mode_set(ctx->mixer_ctx->client, > overlay); > } > static void drm_mixer_commit(struct device *subdrv_dev, int > zpos) > @@ -251,7 +251,7 @@ static void drm_mixer_commit(struct device > *subdrv_dev, int zpos) > } > if (mixer_ops && mixer_ops->win_commit) > - mixer_ops->win_commit(ctx->mixer_ctx->ctx, win); > + mixer_ops->win_commit(ctx->mixer_ctx->client, win); > ctx->enabled[win] = true; > } > @@ -269,7 +269,7 @@ static void drm_mixer_disable(struct device > *subdrv_dev, int zpos) > } > if (mixer_ops && mixer_ops->win_disable) > - mixer_ops->win_disable(ctx->mixer_ctx->ctx, win); > + mixer_ops->win_disable(ctx->mixer_ctx->client, win); > ctx->enabled[win] = false; > } > diff --git a/drivers/gpu/drm/exynos/exynos_drm_hdmi.h > b/drivers/gpu/drm/exynos/exynos_drm_hdmi.h > index a91c420..237434d 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_hdmi.h > +++ b/drivers/gpu/drm/exynos/exynos_drm_hdmi.h > @@ -33,12 +33,12 @@ > * exynos hdmi common context structure. > * > * @drm_dev: pointer to drm_device. > - * @ctx: pointer to the context of specific device driver. > + * @client: pointer to the context of specific device driver. > * this context should be hdmi_context or mixer_context. > */ > struct exynos_drm_hdmi_context { > struct drm_device *drm_dev; > - void *ctx; > + void *client; > }; > struct exynos_hdmi_ops { > diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c > b/drivers/gpu/drm/exynos/exynos_hdmi.c > index bb504cb..7538489 100644 > --- a/drivers/gpu/drm/exynos/exynos_hdmi.c > +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c > @@ -2108,7 +2108,7 @@ static struct exynos_hdmi_ops hdmi_ops = { > static irqreturn_t hdmi_external_irq_thread(int irq, void *arg) > { > struct exynos_drm_hdmi_context *ctx = arg; > - struct hdmi_context *hdata = ctx->ctx; > + struct hdmi_context *hdata = ctx->client; > if (!hdata->get_hpd) > goto out; > @@ -2127,7 +2127,7 @@ out: > static irqreturn_t hdmi_internal_irq_thread(int irq, void *arg) > { > struct exynos_drm_hdmi_context *ctx = arg; > - struct hdmi_context *hdata = ctx->ctx; > + struct hdmi_context *hdata = ctx->client; > u32 intc_flag; > intc_flag = hdmi_reg_read(hdata, HDMI_INTC_FLAG); > @@ -2295,7 +2295,7 @@ static int __devinit hdmi_probe(struct > platform_device *pdev) > mutex_init(&hdata->hdmi_mutex); > - drm_hdmi_ctx->ctx = (void *)hdata; > + drm_hdmi_ctx->client = (void *)hdata; > hdata->parent_ctx = (void *)drm_hdmi_ctx; > platform_set_drvdata(pdev, drm_hdmi_ctx); > @@ -2395,7 +2395,7 @@ static int __devexit hdmi_remove(struct > platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct exynos_drm_hdmi_context *ctx = > platform_get_drvdata(pdev); > - struct hdmi_context *hdata = ctx->ctx; > + struct hdmi_context *hdata = ctx->client; > DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__); > @@ -2417,7 +2417,7 @@ static int __devexit hdmi_remove(struct > platform_device *pdev) > static int hdmi_suspend(struct device *dev) > { > struct exynos_drm_hdmi_context *ctx = get_hdmi_context(dev); > - struct hdmi_context *hdata = ctx->ctx; > + struct hdmi_context *hdata = ctx->client; > disable_irq(hdata->internal_irq); > disable_irq(hdata->external_irq); > @@ -2434,7 +2434,7 @@ static int hdmi_suspend(struct device *dev) > static int hdmi_resume(struct device *dev) > { > struct exynos_drm_hdmi_context *ctx = get_hdmi_context(dev); > - struct hdmi_context *hdata = ctx->ctx; > + struct hdmi_context *hdata = ctx->client; > enable_irq(hdata->external_irq); > enable_irq(hdata->internal_irq); > diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c > b/drivers/gpu/drm/exynos/exynos_mixer.c > index 30fcc12..d3ff595 100644 > --- a/drivers/gpu/drm/exynos/exynos_mixer.c > +++ b/drivers/gpu/drm/exynos/exynos_mixer.c > @@ -864,7 +864,7 @@ static void mixer_finish_pageflip(struct > drm_device > *drm_dev, int crtc) > static irqreturn_t mixer_irq_handler(int irq, void *arg) > { > struct exynos_drm_hdmi_context *drm_hdmi_ctx = arg; > - struct mixer_context *ctx = drm_hdmi_ctx->ctx; > + struct mixer_context *ctx = drm_hdmi_ctx->client; > struct mixer_resources *res = &ctx->mixer_res; > u32 val, base, shadow; > @@ -909,7 +909,7 @@ out: > static int __devinit mixer_resources_init(struct > exynos_drm_hdmi_context > *ctx, > struct platform_device *pdev) > { > - struct mixer_context *mixer_ctx = ctx->ctx; > + struct mixer_context *mixer_ctx = ctx->client; > struct device *dev = &pdev->dev; > struct mixer_resources *mixer_res = &mixer_ctx->mixer_res; > struct resource *res; > @@ -1035,7 +1035,7 @@ static int __devinit mixer_probe(struct > platform_device *pdev) > mutex_init(&ctx->mixer_mutex); > ctx->dev = &pdev->dev; > - drm_hdmi_ctx->ctx = (void *)ctx; > + drm_hdmi_ctx->client = (void *)ctx; > platform_set_drvdata(pdev, drm_hdmi_ctx); > @@ -1070,7 +1070,7 @@ static int mixer_remove(struct > platform_device > *pdev) > static int mixer_suspend(struct device *dev) > { > struct exynos_drm_hdmi_context *drm_hdmi_ctx = > get_mixer_context(dev); > - struct mixer_context *ctx = drm_hdmi_ctx->ctx; > + struct mixer_context *ctx = drm_hdmi_ctx->client; > mixer_poweroff(ctx); > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel