Re: [PATCH] video: drm: exynos: mie bypass enable for fimd
Hello Inki Dae,
On Thu, Dec 27, 2012 at 11:47 AM, Inki Dae inki@samsung.com wrote:
Hi,
DISP1BLK_CFG register is related to GScaler, HDCP and MIXER as well. So
it's not good that this register is controlled in fimd module. And I think
the function to control the register should be placed in SoC common file .
In other words, other drivers should be able to control the register through
common thing also.
Thanks for reviewing the patch.
You mean to say that this functionality should be implemented at arch side
and called by drivers using call back functions ?
If so, then if we moved the driver to full DT version, all the call
backs will be removed.
Then how to make a call to this function?
So I thought other drivers (apart from Fimd) also parses the
appropriate nodes and
program the register as per the need.
Please correct me if my understanding is wrong.
Best Wishes,
Leela Krishna Amudala.
Thanks,
Inki Dae
2012/12/26 Leela Krishna Amudala l.kris...@samsung.com
Bypasses the mie for fimd by parsing the register and bit offset values
from mie-bypass node, if mie-bypass node is present in the dts file.
Signed-off-by: Leela Krishna Amudala l.kris...@samsung.com
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 55
1 file changed, 55 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index bf0d9ba..f8ad259 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
@@ -118,6 +118,12 @@ static const struct of_device_id
fimd_driver_dt_match[] = {
MODULE_DEVICE_TABLE(of, fimd_driver_dt_match);
#endif
+struct mie_bypass {
+ u32 enable_bypass;
+ void __iomem*bypass_reg;
+ u32 bypass_bit_offset;
+};
+
static inline struct fimd_driver_data *drm_fimd_get_driver_data(
struct platform_device *pdev)
{
@@ -133,6 +139,41 @@ static inline struct fimd_driver_data
*drm_fimd_get_driver_data(
platform_get_device_id(pdev)-driver_data;
}
+static struct mie_bypass *parse_mie_bypass_for_fimd(struct device *dev,
+ struct device_node
*mie_bypass_node)
+{
+ struct mie_bypass *bypass_data;
+ u32 phy_address;
+
+ bypass_data = devm_kzalloc(dev, sizeof(*bypass_data),
GFP_KERNEL);
+ if (!bypass_data) {
+ dev_err(dev, memory allocation for bypass data
failed\n);
+ return ERR_PTR(-ENOMEM);
+ }
+ of_property_read_u32(mie_bypass_node,
samsung,mie-bypass-enable,
+ bypass_data-enable_bypass);
+ of_property_read_u32(mie_bypass_node, samsung,disp1blk-cfg-reg,
+ phy_address);
+ of_property_read_u32(mie_bypass_node,
samsung,bypass-bit-offset,
+ bypass_data-bypass_bit_offset);
+
+ bypass_data-bypass_reg = ioremap(phy_address, SZ_4);
+ if (!bypass_data-bypass_reg) {
+ dev_err(dev, failed to ioremap phy_address\n);
+ return ERR_PTR(-ENOMEM);
+ }
+ return bypass_data;
+}
+
+static void mie_bypass_for_fimd(struct mie_bypass *bypass_data)
+{
+ u32 reg;
+
+ reg = __raw_readl(bypass_data-bypass_reg);
+ reg |= (1 bypass_data-bypass_bit_offset);
+ __raw_writel(reg, bypass_data-bypass_reg);
+}
+
static bool fimd_display_is_connected(struct device *dev)
{
DRM_DEBUG_KMS(%s\n, __FILE__);
@@ -906,12 +947,26 @@ static int __devinit fimd_probe(struct
platform_device *pdev)
struct exynos_drm_fimd_pdata *pdata;
struct exynos_drm_panel_info *panel;
struct resource *res;
+ struct device_node *mie_bypass_node;
+ struct mie_bypass *bypass_data = NULL;
int win;
int ret = -EINVAL;
DRM_DEBUG_KMS(%s\n, __FILE__);
pdata = pdev-dev.platform_data;
+ if (pdev-dev.of_node) {
+ mie_bypass_node = of_find_node_by_name(pdev-dev.of_node,
+ mie-bypass);
+ if (mie_bypass_node) {
+ bypass_data =
parse_mie_bypass_for_fimd(pdev-dev,
+ mie_bypass_node);
+ if (IS_ERR(bypass_data))
+ return PTR_ERR(bypass_data);
+ if (bypass_data-enable_bypass)
+ mie_bypass_for_fimd(bypass_data);
+ }
+ }
if (!pdata) {
dev_err(dev, no platform data specified\n);
return -EINVAL;
--
1.8.0
___
dri-devel mailing list
dri-de...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH] video: drm: exynos: mie bypass enable for fimd
2012/12/28 Leela Krishna Amudala l.kris...@samsung.com:
Hello Inki Dae,
On Thu, Dec 27, 2012 at 11:47 AM, Inki Dae inki@samsung.com wrote:
Hi,
DISP1BLK_CFG register is related to GScaler, HDCP and MIXER as well. So
it's not good that this register is controlled in fimd module. And I think
the function to control the register should be placed in SoC common file .
In other words, other drivers should be able to control the register through
common thing also.
Thanks for reviewing the patch.
You mean to say that this functionality should be implemented at arch side
and called by drivers using call back functions ?
If so, then if we moved the driver to full DT version, all the call
backs will be removed.
Then how to make a call to this function?
So I thought other drivers (apart from Fimd) also parses the
appropriate nodes and
program the register as per the need.
Please correct me if my understanding is wrong.
The base address of DISP1BLK_CFG already was iorempped by
iotable_init() at machine init. So you can control DISP1BLK_CFG
register using only offset. But your patch does ioremap again and also
even not iounmap. This is ugly. Please see
arch/arm/plat-samsung/setup-mipiphy.c how the common register is
controlled and this is a good example. And please abuse dt.
Best Wishes,
Leela Krishna Amudala.
Thanks,
Inki Dae
2012/12/26 Leela Krishna Amudala l.kris...@samsung.com
Bypasses the mie for fimd by parsing the register and bit offset values
from mie-bypass node, if mie-bypass node is present in the dts file.
Signed-off-by: Leela Krishna Amudala l.kris...@samsung.com
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 55
1 file changed, 55 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index bf0d9ba..f8ad259 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
@@ -118,6 +118,12 @@ static const struct of_device_id
fimd_driver_dt_match[] = {
MODULE_DEVICE_TABLE(of, fimd_driver_dt_match);
#endif
+struct mie_bypass {
+ u32 enable_bypass;
+ void __iomem*bypass_reg;
+ u32 bypass_bit_offset;
+};
+
static inline struct fimd_driver_data *drm_fimd_get_driver_data(
struct platform_device *pdev)
{
@@ -133,6 +139,41 @@ static inline struct fimd_driver_data
*drm_fimd_get_driver_data(
platform_get_device_id(pdev)-driver_data;
}
+static struct mie_bypass *parse_mie_bypass_for_fimd(struct device *dev,
+ struct device_node
*mie_bypass_node)
+{
+ struct mie_bypass *bypass_data;
+ u32 phy_address;
+
+ bypass_data = devm_kzalloc(dev, sizeof(*bypass_data),
GFP_KERNEL);
+ if (!bypass_data) {
+ dev_err(dev, memory allocation for bypass data
failed\n);
+ return ERR_PTR(-ENOMEM);
+ }
+ of_property_read_u32(mie_bypass_node,
samsung,mie-bypass-enable,
+ bypass_data-enable_bypass);
+ of_property_read_u32(mie_bypass_node, samsung,disp1blk-cfg-reg,
+ phy_address);
+ of_property_read_u32(mie_bypass_node,
samsung,bypass-bit-offset,
+ bypass_data-bypass_bit_offset);
+
+ bypass_data-bypass_reg = ioremap(phy_address, SZ_4);
+ if (!bypass_data-bypass_reg) {
+ dev_err(dev, failed to ioremap phy_address\n);
+ return ERR_PTR(-ENOMEM);
+ }
+ return bypass_data;
+}
+
+static void mie_bypass_for_fimd(struct mie_bypass *bypass_data)
+{
+ u32 reg;
+
+ reg = __raw_readl(bypass_data-bypass_reg);
+ reg |= (1 bypass_data-bypass_bit_offset);
+ __raw_writel(reg, bypass_data-bypass_reg);
+}
+
static bool fimd_display_is_connected(struct device *dev)
{
DRM_DEBUG_KMS(%s\n, __FILE__);
@@ -906,12 +947,26 @@ static int __devinit fimd_probe(struct
platform_device *pdev)
struct exynos_drm_fimd_pdata *pdata;
struct exynos_drm_panel_info *panel;
struct resource *res;
+ struct device_node *mie_bypass_node;
+ struct mie_bypass *bypass_data = NULL;
int win;
int ret = -EINVAL;
DRM_DEBUG_KMS(%s\n, __FILE__);
pdata = pdev-dev.platform_data;
+ if (pdev-dev.of_node) {
+ mie_bypass_node = of_find_node_by_name(pdev-dev.of_node,
+ mie-bypass);
+ if (mie_bypass_node) {
+ bypass_data =
parse_mie_bypass_for_fimd(pdev-dev,
+ mie_bypass_node);
+ if (IS_ERR(bypass_data))
+ return PTR_ERR(bypass_data);
+ if
[PATCH] video: drm: exynos: mie bypass enable for fimd
Bypasses the mie for fimd by parsing the register and bit offset values
from mie-bypass node, if mie-bypass node is present in the dts file.
Signed-off-by: Leela Krishna Amudala l.kris...@samsung.com
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 55
1 file changed, 55 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index bf0d9ba..f8ad259 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
@@ -118,6 +118,12 @@ static const struct of_device_id fimd_driver_dt_match[] = {
MODULE_DEVICE_TABLE(of, fimd_driver_dt_match);
#endif
+struct mie_bypass {
+ u32 enable_bypass;
+ void __iomem*bypass_reg;
+ u32 bypass_bit_offset;
+};
+
static inline struct fimd_driver_data *drm_fimd_get_driver_data(
struct platform_device *pdev)
{
@@ -133,6 +139,41 @@ static inline struct fimd_driver_data
*drm_fimd_get_driver_data(
platform_get_device_id(pdev)-driver_data;
}
+static struct mie_bypass *parse_mie_bypass_for_fimd(struct device *dev,
+ struct device_node *mie_bypass_node)
+{
+ struct mie_bypass *bypass_data;
+ u32 phy_address;
+
+ bypass_data = devm_kzalloc(dev, sizeof(*bypass_data), GFP_KERNEL);
+ if (!bypass_data) {
+ dev_err(dev, memory allocation for bypass data failed\n);
+ return ERR_PTR(-ENOMEM);
+ }
+ of_property_read_u32(mie_bypass_node, samsung,mie-bypass-enable,
+ bypass_data-enable_bypass);
+ of_property_read_u32(mie_bypass_node, samsung,disp1blk-cfg-reg,
+ phy_address);
+ of_property_read_u32(mie_bypass_node, samsung,bypass-bit-offset,
+ bypass_data-bypass_bit_offset);
+
+ bypass_data-bypass_reg = ioremap(phy_address, SZ_4);
+ if (!bypass_data-bypass_reg) {
+ dev_err(dev, failed to ioremap phy_address\n);
+ return ERR_PTR(-ENOMEM);
+ }
+ return bypass_data;
+}
+
+static void mie_bypass_for_fimd(struct mie_bypass *bypass_data)
+{
+ u32 reg;
+
+ reg = __raw_readl(bypass_data-bypass_reg);
+ reg |= (1 bypass_data-bypass_bit_offset);
+ __raw_writel(reg, bypass_data-bypass_reg);
+}
+
static bool fimd_display_is_connected(struct device *dev)
{
DRM_DEBUG_KMS(%s\n, __FILE__);
@@ -906,12 +947,26 @@ static int __devinit fimd_probe(struct platform_device
*pdev)
struct exynos_drm_fimd_pdata *pdata;
struct exynos_drm_panel_info *panel;
struct resource *res;
+ struct device_node *mie_bypass_node;
+ struct mie_bypass *bypass_data = NULL;
int win;
int ret = -EINVAL;
DRM_DEBUG_KMS(%s\n, __FILE__);
pdata = pdev-dev.platform_data;
+ if (pdev-dev.of_node) {
+ mie_bypass_node = of_find_node_by_name(pdev-dev.of_node,
+ mie-bypass);
+ if (mie_bypass_node) {
+ bypass_data = parse_mie_bypass_for_fimd(pdev-dev,
+ mie_bypass_node);
+ if (IS_ERR(bypass_data))
+ return PTR_ERR(bypass_data);
+ if (bypass_data-enable_bypass)
+ mie_bypass_for_fimd(bypass_data);
+ }
+ }
if (!pdata) {
dev_err(dev, no platform data specified\n);
return -EINVAL;
--
1.8.0
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
