Stephen Rothwell wrote:
>
> Hi all,
>
Hi...
Thank you for your information :-)
> Today's linux-next merge of the sound-asoc tree got a conflict in
> arch/arm/mach-s3c64xx/dev-audio.c between commit
> f08269d850e4ee0cd028cf5279cc08259d0469df ("ARM: S3C64XX: Change to using
> s3c_gpio_cfgpin_range()") from the s5p tree and commit
> e4b6b74ffd32d114227a5dcfe814e2cecd80554b ("ARM: S3C64XX: Fix fallthrough
> bug in i2sv3 gpio configuration, improve logging") from the sound-asoc
> tree.
>
Hmm...how can/should I handle for it?
> Both patches added the needed "break". I fixed it up (see below) and can
> carry the fix as necessary.
> --
> Cheers,
> Stephen Rothwell sfr@xxxxxxxxxxxxxxxx
>
> diff --cc arch/arm/mach-s3c64xx/dev-audio.c
> index e4bb023,3838335..0000000
> --- a/arch/arm/mach-s3c64xx/dev-audio.c
> +++ b/arch/arm/mach-s3c64xx/dev-audio.c
> @@@ -22,19 -22,31 +22,20 @@@
> #include <plat/audio.h>
> #include <plat/gpio-cfg.h>
>
> -#include <mach/gpio-bank-c.h>
> -#include <mach/gpio-bank-d.h>
> -#include <mach/gpio-bank-e.h>
> -#include <mach/gpio-bank-h.h>
> -
> static int s3c64xx_i2sv3_cfg_gpio(struct platform_device *pdev)
> {
> + unsigned int base;
> +
> switch (pdev->id) {
> case 0:
> - s3c_gpio_cfgpin(S3C64XX_GPD(0),
> S3C64XX_GPD0_I2S0_CLK);
> - s3c_gpio_cfgpin(S3C64XX_GPD(1),
> S3C64XX_GPD1_I2S0_CDCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPD(2),
> S3C64XX_GPD2_I2S0_LRCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPD(3), S3C64XX_GPD3_I2S0_DI);
> - s3c_gpio_cfgpin(S3C64XX_GPD(4), S3C64XX_GPD4_I2S0_D0);
> + base = S3C64XX_GPD(0);
> break;
> case 1:
> - s3c_gpio_cfgpin(S3C64XX_GPE(0), S3C64XX_GPE0_I2S1_CLK);
> - s3c_gpio_cfgpin(S3C64XX_GPE(1),
> S3C64XX_GPE1_I2S1_CDCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPE(2),
> S3C64XX_GPE2_I2S1_LRCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPE(3), S3C64XX_GPE3_I2S1_DI);
> - s3c_gpio_cfgpin(S3C64XX_GPE(4), S3C64XX_GPE4_I2S1_D0);
> + base = S3C64XX_GPE(0);
> break;
> default:
> - printk(KERN_DEBUG "Invalid I2S Controller number!");
> + printk(KERN_DEBUG "Invalid I2S Controller number: %d\n",
> + pdev->id);
> return -EINVAL;
> }
>
> @@@ -157,17 -170,24 +158,18 @@@ EXPORT_SYMBOL(s3c64xx_device_iisv4)
>
> static int s3c64xx_pcm_cfg_gpio(struct platform_device *pdev)
> {
> + unsigned int base;
> +
> switch (pdev->id) {
> case 0:
> - s3c_gpio_cfgpin(S3C64XX_GPD(0),
> S3C64XX_GPD0_PCM0_SCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPD(1),
> S3C64XX_GPD1_PCM0_EXTCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPD(2),
> S3C64XX_GPD2_PCM0_FSYNC);
> - s3c_gpio_cfgpin(S3C64XX_GPD(3),
> S3C64XX_GPD3_PCM0_SIN);
> - s3c_gpio_cfgpin(S3C64XX_GPD(4),
> S3C64XX_GPD4_PCM0_SOUT);
> + base = S3C64XX_GPD(0);
> break;
> case 1:
> - s3c_gpio_cfgpin(S3C64XX_GPE(0),
> S3C64XX_GPE0_PCM1_SCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPE(1),
> S3C64XX_GPE1_PCM1_EXTCLK);
> - s3c_gpio_cfgpin(S3C64XX_GPE(2),
> S3C64XX_GPE2_PCM1_FSYNC);
> - s3c_gpio_cfgpin(S3C64XX_GPE(3),
> S3C64XX_GPE3_PCM1_SIN);
> - s3c_gpio_cfgpin(S3C64XX_GPE(4),
> S3C64XX_GPE4_PCM1_SOUT);
> + base = S3C64XX_GPE(0);
> break;
> default:
> - printk(KERN_DEBUG "Invalid PCM Controller number!");
> + printk(KERN_DEBUG "Invalid PCM Controller number: %d\n",
> + pdev->id);
> return -EINVAL;
> }
>
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-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
