Re: Samsung: Anyone using s3c2410/s3c2412 cpufreq drivers?

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

 



Hi,

On 02/22/2018 09:38 AM, Krzysztof Kozlowski wrote:
> On Thu, Feb 22, 2018 at 5:19 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:

>> I was doing some regular cleanups in the core and had a look at
>> samsung drivers for that. I am quite sure I am not reading the code
>> correctly, and need your help in that.
>>
>> The s3c24xx-cpufreq.c driver is used by s3c2410-cpufreq.c,
>> s3c2412-cpufreq.c and s3c2440-cpufreq.c files but only the
>> last one is actually setting the calc_freqtable() callback. That makes
>> me wonder if the cpufreq stuff is working on 2410/2412 at all, or if
>> anyone is actually using it there.
>>
>> If calc_freqtable() callback isn't present, we wouldn't have a frequency
>> table (ftab) in the 24xx driver and so ->s3c_cpufreq_init() should just
>> fail.
>>
>> Can you please help me understanding this piece of code ?

> Unfortunately I do not have these boards. Few months ago I was trying
> to get a development board from one of manufacturers (FriendlyARM) but
> they said that all boards are sold out.
> 
> I know that guys in Poland have one working and I think Sylwester
> brought it up to live.
> 
> Sylwester, could you help here?

The odds are good cpufreq on S3C2410 is indeed broken in current mainline
kernel, probably no one is doing test on that platform any more. I wouldn't
be surprised if there were regressions since several kernel releases on that
platform.

Unfortunately, I have only an S3C2440 based Micro2440 ev. board, so I can't 
test this.

-- 
Regards,
Sylwester
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux