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

From: [email-masked] (Tomasz Figa)
Date: Tue, 06 Sep 2011 12:41:07 +0200

On Thursday 01 of September 2011 at 13:18:32, Tomasz Figa wrote:
> On Thursday 01 of September 2011 at 12:33:10, Kyungmin Park wrote:
> > On Thu, Sep 1, 2011 at 11:26 AM, Kukjin Kim <[email-masked]> wrote:
> > > Tomasz Figa wrote:
> > >>
> > >> 2011/8/31 Mark Brown <[email-masked]>:
> > >> > On Wed, Aug 31, 2011 at 02:34:15PM +0200, Tomasz Figa wrote:
> > >> >
> > >> >> This patch is a prerequisite to adding generic time support for
> > > S3C64xx.
> > >> >> It makes possible to differentiate SoCs, required to exclude timers 3
> > > and 4
> > >> >> from PWM driver only on S3C64xx.
> > >> >
> > >> > Now we have the cpu_is_foo() support for Samsung CPUs can we use that
> > >> > instead?
> > >>
> > >> [snip]
> > >
> > > Hi Tomasz,
> > >
> > > You can use "soc_is_s3c64xx()" based on Samsung -next tree.
> > > git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> > > for-next
> > 
> > +static struct platform_device_id s3c_pwm_driver_ids[] = {
> > +       {
> > +               .name           = "s3c24xx-pwm",
> > +               .driver_data    = TYPE_GENERIC,
> > +       }, {
> > +               .name           = "s3c64xx-pwm",
> > +               .driver_data    = TYPE_S3C64XX,
> > +       },
> > +};
> > +MODULE_DEVICE_TABLE(platform, s3c_pwm_driver_ids);
> > 
> > doesn't it right way?
> > do you want to use the soc_is_* at device?
> > 
> 
> 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?

Best regards,
Tom