diff mbox

[1/1] ARM: exynos_defconfig: Enable HS-I2C

Message ID 1393935776-22657-1-git-send-email-sachin.kamat@linaro.org
State Accepted
Headers show

Commit Message

Sachin Kamat March 4, 2014, 12:22 p.m. UTC
High speed I2C is used on Exynos5 based SoCs. Enable it.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/configs/exynos_defconfig |    1 +
 1 file changed, 1 insertion(+)

Comments

Tushar Behera April 24, 2014, 8:02 a.m. UTC | #1
On 03/04/2014 05:52 PM, Sachin Kamat wrote:
> High speed I2C is used on Exynos5 based SoCs. Enable it.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  arch/arm/configs/exynos_defconfig |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
> index 4ce7b70ea901..e07a227ec0db 100644
> --- a/arch/arm/configs/exynos_defconfig
> +++ b/arch/arm/configs/exynos_defconfig
> @@ -65,6 +65,7 @@ CONFIG_TCG_TIS_I2C_INFINEON=y
>  CONFIG_I2C=y
>  CONFIG_I2C_MUX=y
>  CONFIG_I2C_ARB_GPIO_CHALLENGE=y
> +CONFIG_I2C_EXYNOS5=y
>  CONFIG_I2C_S3C2410=y
>  CONFIG_DEBUG_GPIO=y
>  # CONFIG_HWMON is not set
> 
Kukjin,

Please consider picking this patch when you plan to update
exynos_defconfig. With this change on upstream kernel, we are able to
get Arndale_Octa board to boot and mount an MMC partition successfully.
Kukjin Kim April 26, 2014, 11:14 a.m. UTC | #2
Tushar Behera wrote:
> 
> On 03/04/2014 05:52 PM, Sachin Kamat wrote:
> > High speed I2C is used on Exynos5 based SoCs. Enable it.
> >
> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> > ---
> >  arch/arm/configs/exynos_defconfig |    1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm/configs/exynos_defconfig
> b/arch/arm/configs/exynos_defconfig
> > index 4ce7b70ea901..e07a227ec0db 100644
> > --- a/arch/arm/configs/exynos_defconfig
> > +++ b/arch/arm/configs/exynos_defconfig
> > @@ -65,6 +65,7 @@ CONFIG_TCG_TIS_I2C_INFINEON=y
> >  CONFIG_I2C=y
> >  CONFIG_I2C_MUX=y
> >  CONFIG_I2C_ARB_GPIO_CHALLENGE=y
> > +CONFIG_I2C_EXYNOS5=y
> >  CONFIG_I2C_S3C2410=y
> >  CONFIG_DEBUG_GPIO=y
> >  # CONFIG_HWMON is not set
> >
> Kukjin,
> 
> Please consider picking this patch when you plan to update
> exynos_defconfig. With this change on upstream kernel, we are able to
> get Arndale_Octa board to boot and mount an MMC partition successfully.
> 
Oh, OK.

BTW, basically the boot should be fine with/without selecting
I2C_EXYNOS5...or if the config is required always for arndale_octa board, it
should be enabled without any manual selecting. I think, it means this is
just temporal solution not right approach to fix the problem..

I need to look at the codes...

Thanks,
Kukjin

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Tushar Behera April 28, 2014, 3:56 a.m. UTC | #3
On 04/26/2014 04:44 PM, Kukjin Kim wrote:
> Tushar Behera wrote:
>>
>> On 03/04/2014 05:52 PM, Sachin Kamat wrote:
>>> High speed I2C is used on Exynos5 based SoCs. Enable it.
>>>
>>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>>> ---
>>>  arch/arm/configs/exynos_defconfig |    1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/arch/arm/configs/exynos_defconfig
>> b/arch/arm/configs/exynos_defconfig
>>> index 4ce7b70ea901..e07a227ec0db 100644
>>> --- a/arch/arm/configs/exynos_defconfig
>>> +++ b/arch/arm/configs/exynos_defconfig
>>> @@ -65,6 +65,7 @@ CONFIG_TCG_TIS_I2C_INFINEON=y
>>>  CONFIG_I2C=y
>>>  CONFIG_I2C_MUX=y
>>>  CONFIG_I2C_ARB_GPIO_CHALLENGE=y
>>> +CONFIG_I2C_EXYNOS5=y
>>>  CONFIG_I2C_S3C2410=y
>>>  CONFIG_DEBUG_GPIO=y
>>>  # CONFIG_HWMON is not set
>>>
>> Kukjin,
>>
>> Please consider picking this patch when you plan to update
>> exynos_defconfig. With this change on upstream kernel, we are able to
>> get Arndale_Octa board to boot and mount an MMC partition successfully.
>>
> Oh, OK.
> 
> BTW, basically the boot should be fine with/without selecting

The boot is just fine. Just that, we can't use the MMC partition to
mount rootfs.

> I2C_EXYNOS5...or if the config is required always for arndale_octa board, it
> should be enabled without any manual selecting. I think, it means this is
> just temporal solution not right approach to fix the problem..
> 

I don't see any reason why HS-I2C support not be enabled by default on
exynos_defconfig. On many boards,the regulators are connected HS-I2C bus
and without this option enabled the regulators don't come by default.

If you have any reason why this is not the right approach to fix the
problem, please let us know.

> I need to look at the codes...
> 
> Thanks,
> Kukjin
>
diff mbox

Patch

diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
index 4ce7b70ea901..e07a227ec0db 100644
--- a/arch/arm/configs/exynos_defconfig
+++ b/arch/arm/configs/exynos_defconfig
@@ -65,6 +65,7 @@  CONFIG_TCG_TIS_I2C_INFINEON=y
 CONFIG_I2C=y
 CONFIG_I2C_MUX=y
 CONFIG_I2C_ARB_GPIO_CHALLENGE=y
+CONFIG_I2C_EXYNOS5=y
 CONFIG_I2C_S3C2410=y
 CONFIG_DEBUG_GPIO=y
 # CONFIG_HWMON is not set