Hi Sylwester,
On Mon, Mar 11, 2013 at 2:24 AM, Sylwester Nawrocki
<sylvester.nawrocki@xxxxxxxxx> wrote:
> On 03/06/2013 12:53 PM, Shaik Ameer Basha wrote:
>>
>> This patch adds necessary source definations needed for mipi-csis
>> driver and adds devicetree node for exynos5250.
>>
>> Signed-off-by: Shaik Ameer Basha<shaik.ameer@xxxxxxxxxxx>
>> ---
>> arch/arm/boot/dts/exynos5250.dtsi | 18 ++++++++++++++++++
>> arch/arm/mach-exynos/clock-exynos5.c | 16 ++++++++++++++--
>> arch/arm/mach-exynos/include/mach/map.h | 3 +++
>> arch/arm/mach-exynos/mach-exynos5-dt.c | 4 ++++
>> 4 files changed, 39 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos5250.dtsi
>> b/arch/arm/boot/dts/exynos5250.dtsi
>> index 3a2cd9a..4fff98b 100644
>> --- a/arch/arm/boot/dts/exynos5250.dtsi
>> +++ b/arch/arm/boot/dts/exynos5250.dtsi
>> @@ -47,6 +47,8 @@
>> i2c6 =&i2c_6;
>> i2c7 =&i2c_7;
>> i2c8 =&i2c_8;
>> + csis0 =&csis_0;
>> + csis1 =&csis_1;
>> };
>>
>> gic:interrupt-controller@10481000 {
>> @@ -357,4 +359,20 @@
>> reg =<0x14450000 0x10000>;
>> interrupts =<0 94 0>;
>> };
>> +
>> + csis_0: csis@13C20000 {
>> + compatible = "samsung,exynos5250-csis";
>> + reg =<0x13C20000 0x4000>;
>> + interrupts =<0 79 0>;
>> + bus-width =<4>;
>> + status = "disabled";
>> + };
>> +
>> + csis_1: csis@13C30000 {
>> + compatible = "samsung,exynos5250-csis";
>> + reg =<0x13C30000 0x4000>;
>> + interrupts =<0 80 0>;
>> + bus-width =<4>;
>
>
> Shouldn't this be 2 ? Anyway what's the point of adding this node here
Yes, It has to be 2. Seems some Ctrl+c/v issues. :)
> only to move it in a subsequent patch ? I guess you should first add
> 'camera' node and then have further patches adding relevant device nodes
Ok.. I even got the same comments just before postings. As it was a RFC patch,
I just posted this as it is.
Definitely, i will modify this in the upcoming version of patches.
Regards,
Shaik Ameer Basha
>
>
>> + status = "disabled";
>> + };
>> };
>
>
>> diff --git a/arch/arm/mach-exynos/clock-exynos5.c
>> b/arch/arm/mach-exynos/clock-exynos5.c
>> index e9d7b80..34a22ff 100644
>> --- a/arch/arm/mach-exynos/clock-exynos5.c
>> +++ b/arch/arm/mach-exynos/clock-exynos5.c
>
>
> This file is already removed in Kukjin's for-next tree.
> And for dts changes I would start the patch summary line with "ARM: dts:".
>
>
>> @@ -859,6 +859,16 @@ static struct clk exynos5_init_clocks_off[] = {
>> .enable = exynos5_clk_ip_gscl_ctrl,
>> .ctrlbit = (1<< 3),
>> }, {
>> + .name = "csis",
>> + .devname = "s5p-mipi-csis.0",
>> + .enable = exynos5_clk_ip_gscl_ctrl,
>> + .ctrlbit = (1<< 5),
>> + }, {
>> + .name = "csis",
>> + .devname = "s5p-mipi-csis.1",
>> + .enable = exynos5_clk_ip_gscl_ctrl,
>> + .ctrlbit = (1<< 6),
>> + }, {
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
