Re: [PATCH v2 2/7] clk: exynos5420: Add more clock IDs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Tomasz,


On Tue, Apr 15, 2014 at 10:20 PM, Tomasz Figa <tomasz.figa@xxxxxxxxx> wrote:
> Hi Shaik,
>
>
> On 27.03.2014 12:07, Shaik Ameer Basha wrote:
>>
>> From: Rahul Sharma <rahul.sharma@xxxxxxxxxxx>
>>
>> Add more clock IDs to be used in DT bindings for Exynos5420.
>>
>> Signed-off-by: Rahul Sharma <rahul.sharma@xxxxxxxxxxx>
>> Signed-off-by: Shaik Ameer Basha <shaik.ameer@xxxxxxxxxxx>
>> ---
>>   include/dt-bindings/clock/exynos5420.h |   62
>> ++++++++++++++++++++++++++++++--
>>   1 file changed, 60 insertions(+), 2 deletions(-)
>>
>> diff --git a/include/dt-bindings/clock/exynos5420.h
>> b/include/dt-bindings/clock/exynos5420.h
>> index 5eefd88..e921913 100644
>> --- a/include/dt-bindings/clock/exynos5420.h
>> +++ b/include/dt-bindings/clock/exynos5420.h
>> @@ -58,6 +58,16 @@
>>   #define CLK_SCLK_GSCL_WA      156
>>   #define CLK_SCLK_GSCL_WB      157
>>   #define CLK_SCLK_HDMIPHY      158
>> +#define CLK_SCLK_MPHY_REFCLK   159
>> +#define CLK_SCLK_SPI0_ISP      160
>> +#define CLK_SCLK_SPI1_ISP      161
>> +#define CLK_SCLK_UART_ISP      162
>> +#define CLK_SCLK_ISP_SENSOR0   163
>> +#define CLK_SCLK_ISP_SENSOR1   164
>> +#define CLK_SCLK_ISP_SENSOR2   165
>> +#define CLK_SCLK_PWM_ISP       166
>> +#define CLK_SCLK_HSIC_12M      167
>> +#define CLK_SCLK_MPHY_IXTAL24  168
>>
>>   /* gate clocks */
>>   #define CLK_ACLK66_PERIC      256
>> @@ -123,6 +133,7 @@
>>   #define CLK_USBH20            365
>>   #define CLK_USBD300           366
>>   #define CLK_USBD301           367
>> +#define CLK_PCLK200_FSYS       370
>>   #define CLK_ACLK400_MSCL      380
>>   #define CLK_MSCL0             381
>>   #define CLK_MSCL1             382
>> @@ -141,6 +152,8 @@
>>   #define CLK_ACLK300_DISP1     420
>>   #define CLK_FIMD1             421
>>   #define CLK_SMMU_FIMD1                422
>> +#define CLK_SMMU_FIMD1M1       423
>> +#define CLK_ACLK400_DISP1      424
>>   #define CLK_ACLK166           430
>>   #define CLK_MIXER             431
>>   #define CLK_ACLK266           440
>> @@ -172,12 +185,57 @@
>>   #define CLK_SMMU_FIMCL1               493
>>   #define CLK_SMMU_FIMCL3               494
>>   #define CLK_FIMC_LITE3                495
>> -#define CLK_ACLK_G3D           500
>> -#define CLK_G3D                        501
>> +#define CLK_G3D                        500
>
>
> What is the reason for this ID change? Even if CLK_ACLK_G3D is removed, as
> it wasn't even referenced by the driver, original clock IDs should remain
> fixed.

My Bad.
Anyways, I am planning to post one more patch at the end of the series
to remove the gaps b/w the numbers.
I hope there is no significance of the numbers once we move completely
to macros.

>
>
>>   #define CLK_SMMU_MIXER                502
>> +#define CLK_PCLK_TZPC10                503
>> +#define CLK_PCLK_TZPC11                504
>> +#define CLK_PCLK_MC            505
>> +#define CLK_PCLK_TOP_RTC       506
>> +#define CLK_SMMU_JPEG2         507
>> +#define CLK_PCLK_ROTATOR       508
>> +#define CLK_SMMU_RTIC          509
>> +#define CLK_PCLK_G2D           510
>> +#define CLK_ACLK_SMMU_G2D      511
>> +#define CLK_SMMU_G2D           512
>> +#define CLK_ACLK_SMMU_MDMA0    513
>> +#define CLK_SMMU_MDMA0         514
>> +#define CLK_ACLK_SMMU_SSS      515
>> +#define CLK_SMMU_SSS           516
>> +#define CLK_SMMU_SLIM_SSS      517
>> +#define CLK_ACLK_SMMU_SLIM_SSS 518
>> +#define CLK_ACLK266_ISP                519
>> +#define CLK_ACLK400_ISP                520
>> +#define CLK_ACLK333_432_ISP0   521
>> +#define CLK_ACLK333_432_ISP    522
>> +#define CLK_ACLK_SMMU_MIXER    523
>> +#define CLK_PCLK_HDMIPHY       524
>> +#define CLK_PCLK_GSCL0         525
>> +#define CLK_PCLK_GSCL1         526
>> +#define CLK_PCLK_FIMC_3AA      527
>> +#define CLK_ACLK_FIMC_LITE0    528
>> +#define CLK_ACLK_FIMC_LITE1    529
>> +#define CLK_PCLK_FIMC_LITE0    530
>> +#define CLK_PCLK_FIMC_LITE1    531
>> +#define CLK_PCLK_FIMC_LITE3    532
>> +#define CLK_PCLK_MSCL0         533
>> +#define CLK_PCLK_MSCL1         534
>> +#define CLK_PCLK_MSCL2         535
>> +#define CLK_PCLK_MFC           536
>>
>>   /* mux clocks */
>>   #define CLK_MOUT_HDMI         640
>> +#define CLK_MOUT_FIMD1                 641
>> +#define CLK_MOUT_MAUDIO0               642
>> +#define CLK_MOUT_SPI0                  643
>> +#define CLK_MOUT_SPI1                  644
>> +#define CLK_MOUT_SPI2                  645
>> +#define CLK_MOUT_SW_ACLK333            646
>> +#define CLK_MOUT_USER_ACLK333          647
>> +#define CLK_MOUT_SW_ACLK300_GSCL       648
>> +#define CLK_MOUT_USER_ACLK300_GSCL     649
>> +#define CLK_MOUT_SW_ACLK333_432_GSCL   650
>> +#define CLK_MOUT_USER_ACLK333_432_GSCL 651
>> +#define CLK_MOUT_G3D                   652
>>
>>   /* divider clocks */
>>   #define CLK_DOUT_PIXEL                768
>>
>
> Also similar comment as for patch 1/7, it's hard to tell why this change is
> needed at all.

Ok. Will take care for the next series.

Regards,
Shaik Ameer Basha

>
> Best regards,
> Tomasz
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux