[PATCH 1/5] ARM: Samsung: PWM: Allow to differentiate SoCs based on platform device name.

Tomasz Figa tomasz.figa at gmail.com
Tue Sep 6 07:35:58 EDT 2011


On Tuesday 06 of September 2011 at 12:03:11, Russell King - ARM Linux wrote:
> On Tue, Sep 06, 2011 at 12:41:07PM +0200, Tomasz Figa wrote:
> > On Thursday 01 of September 2011 at 13:18:32, Tomasz Figa wrote:
> > > Well, now we have three alternatives:
> > > - to leave it as is (i.e. changing platform device names)
> > > - to use cpu_is_* (does it ever exist for S3C64xx or soc_is_s3c64xx is its equivalent?)
> > > - to use soc_is_* 
> > > 
> > > Waiting for more comments then.
> > > 
> > Ping.
> > 
> > Should I keep it as is or rather change it to use soc_is_s3c64xx instead?
> 
> I don't know what question you're asking (the questions don't make sense
> in terms of the quoted context.)
> 
> If you're asking whether you should use a cpu_ prefix instead of a soc_
> prefix for identifying s3c64xx, then consider the question.
> 
> What is the CPU?
> 
> What are S3C64xx / OMAP / PXA310 / SA-11x0 / AT91RM9200?  CPU or SoC ?
> What are Cortex A8/A9 / Xscale / StrongARM / ARM920?  CPU or SoC ?

Well, I believe that the context I quoted explains everything, but let me
explain that again:

The patch is about a driver being able to check to what SoC a device belongs,
but there are at least two ways to achieve that.

I can see many drivers using the way I used, i.e. distinct platform device names
for each SoC that has to be differentiated plus a generic one. However, I was
suggested that using cpu_is_*/soc_is_* might be a better approach, while at the
same time I received an opinion that the original method is the preferred one.

Best regards,
Tom




More information about the linux-arm-kernel mailing list