Vasily Khoruzhick wrote:
>
> On Wednesday 01 December 2010 09:26:31 Kukjin Kim wrote:
>
> > How about following...
> >
> > +void __init s3c2440_map_io(void)
> > +{
> > + s3c24xx_gpiocfg_default.set_pull = s3c_gpio_setpull_1up;
> > + s3c24xx_gpiocfg_default.get_pull = s3c_gpio_getpull_1up;
> > +
> > + s3c244x_map_io();
> > +}
>
> What's difference except order?
>
Just same ordering is used in other Samsung S3C SoCs.
Now actually doesn't matter it.
> > Yeah, however, in my opinion, need to add following during
gpiolib_init().
> >
> > @@ -222,6 +222,11 @@ static __init int s3c24xx_gpiolib_init(void)
> > if (!chip->config)
> > chip->config = &s3c24xx_gpiocfg_default;
> >
> > + if (!chip->config->set_pull)
> > + chip->config->set_pull = s3c_gpio_setpull_1up;
> > + if (!chip->config->get_pull)
> > + chip->config->get_pull = s3c_gpio_getpull_1up;
> > +
> > s3c_gpiolib_add(chip);
> > }
>
> Pull callbacks are soc-specific, s3c24xx_gpiolib_init is common for all
socs,
> so we can't do like this.
>
Yeah, right...it depends on each SoCs. But, if there is no setup it in
map_io(), there is no functions of set_pull() and get_pull(). As you know,
used s3c_gpio_{set,get}pull_1up as default before your patch...
Hmm...however, I will think about its necessity again.
> Btw, I've sent v6 version of patch, it still with empty line issues,
should I
> resend v7 version or maintainers can fixup it at their side? :)
>
Ok..I will do it and will apply.
Then, will be sent to Linus during 37-rc.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
--
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
