[Buildroot] [PATCH v3 1/2] pcsc-lite: new package

[Buildroot] [PATCH v3 1/2] pcsc-lite: new package

[Waldemar Rymarkiewicz]

Middleware to access a smart card using SCard API (PC/SC).

http://pcsclite.alioth.debian.org/

Signed-off-by: Waldemar Rymarkiewicz <waldemar.rymarkiewicz@tieto.com>
---
 package/Config.in              |    1 +
 package/pcsc-lite/Config.in    |   29 +++++++++++++++++++++++++++++
 package/pcsc-lite/pcsc-lite.mk |   36 ++++++++++++++++++++++++++++++++++++
 3 files changed, 66 insertions(+)
 create mode 100644 package/pcsc-lite/Config.in
 create mode 100644 package/pcsc-lite/pcsc-lite.mk

diff --git a/package/Config.in b/package/Config.in
index a4e0b2c..dcc84b6 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -395,6 +395,7 @@ source "package/libusb/Config.in"
 source "package/libusb-compat/Config.in"
 source "package/libv4l/Config.in"
 source "package/mtdev/Config.in"
+source "package/pcsc-lite/Config.in"
 endmenu
 
 menu "Javascript"
diff --git a/package/pcsc-lite/Config.in b/package/pcsc-lite/Config.in
new file mode 100644
index 0000000..cfbcb69
--- /dev/null
+++ b/package/pcsc-lite/Config.in
@@ -0,0 +1,29 @@
+config BR2_PACKAGE_PCSC_LITE
+	bool "pcsc-lite"
+	depends on BR2_TOOLCHAIN_HAS_THREADS
+	help
+	  Middleware to access smart card using SCard API (PC/SC).
+
+	  http://pcsclite.alioth.debian.org/
+
+if BR2_PACKAGE_PCSC_LITE
+
+config BR2_PACKAGE_PCSC_LITE_FORCE_LIBUSB
+	bool "use libusb"
+	depends on !BR2_PACKAGE_UDEV
+	select BR2_PACKAGE_LIBUSB
+
+config BR2_PACKAGE_PCSC_LITE_DEBUGATR
+	bool "enable ATR debug messages"
+	help
+	  enable ATR debug messages from pcscd
+
+config BR2_PACKAGE_PCSC_LITE_EMBEDDED
+	bool "enable embedded mode"
+	help
+	  limit RAM and CPU resources by disabling features (log)
+
+endif
+
+comment "pcsc-lite needs a toolchain with thread support"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
diff --git a/package/pcsc-lite/pcsc-lite.mk b/package/pcsc-lite/pcsc-lite.mk
new file mode 100644
index 0000000..35d0cf3
--- /dev/null
+++ b/package/pcsc-lite/pcsc-lite.mk
@@ -0,0 +1,36 @@
+##########################################################
+#
+# PCSC-Lite
+#
+# ########################################################
+PCSC_LITE_VERSION = 1.8.6
+PCSC_LITE_SOURCE = pcsc-lite-$(PCSC_LITE_VERSION).tar.bz2
+PCSC_LITE_SITE = https://alioth.debian.org/frs/download.php/3757
+PCSC_LITE_INSTALL_STAGING = YES
+PCSC_LITE_DEPENDENCIES = host-pkg-config
+
+# - libudev and libusb are optional
+# - libudev and libusb can't be used together
+# - libudev has a priority over libusb
+
+ifeq ($(BR2_PACKAGE_UDEV),y)
+PCSC_LITE_CONF_OPT += --enable-libudev --disable-libusb
+PCSC_LITE_DEPENDENCIES += udev
+else
+ifeq ($(BR2_PACKAGE_LIBUSB),y)
+PCSC_LITE_CONF_OPT += --enable-libusb --disable-libudev
+PCSC_LITE_DEPENDENCIES += libusb
+else
+PCSC_LITE_CONF_OPT += --disable-libusb --disable-libudev
+endif
+endif
+
+ifeq ($(PACKAGE_PCSC_LITE_DEBUGATR),y)
+PCSC_LITE_CONF_OPT += --enable-debugatr
+endif
+
+ifeq ($(PACKAGE_PCSC_LITE_EMBEDDED),y)
+PCSC_LITE_CONF_OPT += --enable-embedded
+endif
+
+$(eval $(call autotools-package))
-- 
1.7.10

[Buildroot] [PATCH v3 2/2] ccid: new package

[Waldemar Rymarkiewicz]

PC/SC driver for USB CCID smart card readers.

http://pcsclite.alioth.debian.org/ccid.html

Signed-off-by: Waldemar Rymarkiewicz <waldemar.rymarkiewicz@tieto.com>
---
 package/Config.in      |    1 +
 package/ccid/Config.in |   12 ++++++++++++
 package/ccid/ccid.mk   |   12 ++++++++++++
 3 files changed, 25 insertions(+)
 create mode 100644 package/ccid/Config.in
 create mode 100644 package/ccid/ccid.mk

diff --git a/package/Config.in b/package/Config.in
index dcc84b6..a0fb593 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -382,6 +382,7 @@ source "package/zxing/Config.in"
 endmenu
 
 menu "Hardware handling"
+source "package/ccid/Config.in"
 source "package/libaio/Config.in"
 source "package/libraw1394/Config.in"
 source "package/tslib/Config.in"
diff --git a/package/ccid/Config.in b/package/ccid/Config.in
new file mode 100644
index 0000000..c9b77d1
--- /dev/null
+++ b/package/ccid/Config.in
@@ -0,0 +1,12 @@
+config BR2_PACKAGE_CCID
+	bool "ccid"
+	select BR2_PACKAGE_PCSC_LITE
+	select BR2_PACKAGE_LIBUSB
+	depends on BR2_TOOLCHAIN_HAS_THREADS # libusb
+	help
+	  PC/SC driver for USB CCID smart card readers
+
+	  http://pcsclite.alioth.debian.org/ccid.html
+
+comment "ccid needs a toolchain with thread support"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
diff --git a/package/ccid/ccid.mk b/package/ccid/ccid.mk
new file mode 100644
index 0000000..d24f75d
--- /dev/null
+++ b/package/ccid/ccid.mk
@@ -0,0 +1,12 @@
+##########################################################
+#
+# CCID
+#
+# ########################################################
+CCID_VERSION = 1.4.7
+CCID_SOURCE = ccid-$(CCID_VERSION).tar.bz2
+CCID_SITE = https://alioth.debian.org/frs/download.php/3730
+CCID_INSTALL_STAGING = YES
+CCID_DEPENDENCIES = pcsc-lite host-pkg-config libusb
+
+$(eval $(call autotools-package))
-- 
1.7.10

[Buildroot] [PATCH v3 2/2] ccid: new package

[Waldemar.Rymarkiewicz at tieto.com]

>PC/SC driver for USB CCID smart card readers.
>
>http://pcsclite.alioth.debian.org/ccid.html
>
>Signed-off-by: Waldemar Rymarkiewicz <waldemar.rymarkiewicz@tieto.com>
>---
> package/Config.in      |    1 +
> package/ccid/Config.in |   12 ++++++++++++
> package/ccid/ccid.mk   |   12 ++++++++++++
> 3 files changed, 25 insertions(+)
> create mode 100644 package/ccid/Config.in
> create mode 100644 package/ccid/ccid.mk


No more comments on this series?

Thanks,
/Waldek

[Buildroot] [PATCH v3 2/2] ccid: new package

[Waldemar.Rymarkiewicz at tieto.com]

Hi,

As I don't see more comments on patches (ccid and pcsc) I sent some time ago would appreciate to get a clear answer if you consider to upstream them or not.

Thanks,
/Waldek

[Buildroot] [PATCH v3 2/2] ccid: new package

[Peter Korsgaard]

>>>>>   <Waldemar.Rymarkiewicz@tieto.com> writes:

 > Hi,

 > As I don't see more comments on patches (ccid and pcsc) I sent some
 > time ago would appreciate to get a clear answer if you consider to
 > upstream them or not.

Sorry for the delay. I'll try to find time to review them this week.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH v3 2/2] ccid: new package

[Waldemar.Rymarkiewicz at tieto.com]

Hi Peter.

>Sorry for the delay. I'll try to find time to review them this week.
>

No worries. Just wasn't sure if anyone plan to do more with them or not. 

Then, I will wait for your feedback.

Thanks,
/Waldek


 

[Buildroot] [PATCH v3 1/2] pcsc-lite: new package

[Arnout Vandecappelle]

On 21/09/12 09:32, Waldemar Rymarkiewicz wrote:
> Middleware to access a smart card using SCard API (PC/SC).
>
> http://pcsclite.alioth.debian.org/
>
> Signed-off-by: Waldemar Rymarkiewicz<waldemar.rymarkiewicz@tieto.com>
[snip]
> +config BR2_PACKAGE_PCSC_LITE_FORCE_LIBUSB
> +	bool "use libusb"
> +	depends on !BR2_PACKAGE_UDEV
> +	select BR2_PACKAGE_LIBUSB

  This one is missing a help text.

[snip]
> +# - libudev and libusb are optional
> +# - libudev and libusb can't be used together
> +# - libudev has a priority over libusb

  Well put!

> +
> +ifeq ($(BR2_PACKAGE_UDEV),y)
> +PCSC_LITE_CONF_OPT += --enable-libudev --disable-libusb
> +PCSC_LITE_DEPENDENCIES += udev
> +else
> +ifeq ($(BR2_PACKAGE_LIBUSB),y)
> +PCSC_LITE_CONF_OPT += --enable-libusb --disable-libudev
> +PCSC_LITE_DEPENDENCIES += libusb
> +else
> +PCSC_LITE_CONF_OPT += --disable-libusb --disable-libudev
> +endif
> +endif
> +
> +ifeq ($(PACKAGE_PCSC_LITE_DEBUGATR),y)
> +PCSC_LITE_CONF_OPT += --enable-debugatr
> +endif
> +
> +ifeq ($(PACKAGE_PCSC_LITE_EMBEDDED),y)
> +PCSC_LITE_CONF_OPT += --enable-embedded
> +endif

  There is also a --enable-usbdropdir which defaults to
/usr/local/lib/pcsc/drivers.  We avoid using /usr/local, so could
you make that point to /usr/lib/pcsc/drivers?  Or is it anyway
not used?


  Regards,
  Arnout

> +
> +$(eval $(call autotools-package))

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

[Buildroot] [PATCH v3 2/2] ccid: new package

[Arnout Vandecappelle]

On 21/09/12 09:32, Waldemar Rymarkiewicz wrote:
> PC/SC driver for USB CCID smart card readers.
>
> http://pcsclite.alioth.debian.org/ccid.html
>
> Signed-off-by: Waldemar Rymarkiewicz<waldemar.rymarkiewicz@tieto.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

[snip]
> diff --git a/package/ccid/ccid.mk b/package/ccid/ccid.mk
> new file mode 100644
> index 0000000..d24f75d
> --- /dev/null
> +++ b/package/ccid/ccid.mk
> @@ -0,0 +1,12 @@
> +##########################################################
> +#
> +# CCID
> +#
> +# ########################################################
> +CCID_VERSION = 1.4.7
> +CCID_SOURCE = ccid-$(CCID_VERSION).tar.bz2
> +CCID_SITE = https://alioth.debian.org/frs/download.php/3730
> +CCID_INSTALL_STAGING = YES
> +CCID_DEPENDENCIES = pcsc-lite host-pkg-config libusb
> +
> +$(eval $(call autotools-package))

  When installing, ccid says:
copy the src/92_pcscd_ccid.rules file in udev directory (/etc/udev/rules.d/)
Perhaps it would be a good idea to do that?  (If BR2_PACKAGE_UDEV, of course)

  Also, ccid has the same --enable-embedded option as pcsc-lite.  I'm not
sure if it is functional, though.

  Regards,
  Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

[Buildroot] [PATCH v3 1/2] pcsc-lite: new package

[Waldemar.Rymarkiewicz at tieto.com]

Hi,

>  There is also a --enable-usbdropdir which defaults to
>/usr/local/lib/pcsc/drivers.  We avoid using /usr/local, so could
>you make that point to /usr/lib/pcsc/drivers?  Or is it anyway
>not used?

Then we have to set --prefix=/usr as  it defaults to /usr/local. If we do so usbdropdir will be correct as well.

Thanks,
/Waldek

[Buildroot] [PATCH v3 1/2] pcsc-lite: new package

[Arnout Vandecappelle]

On 01/10/12 07:44, Waldemar.Rymarkiewicz at tieto.com wrote:
> Hi,
>
>>   There is also a --enable-usbdropdir which defaults to
>> /usr/local/lib/pcsc/drivers.  We avoid using /usr/local, so could
>> you make that point to /usr/lib/pcsc/drivers?  Or is it anyway
>> not used?
>
> Then we have to set --prefix=/usr as  it defaults to /usr/local. If we do so usbdropdir will be correct as well.

  My bad, --prefix=/usr is already passed to configure.  It's just the help text
that was confusing me.

  Regards,
  Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

[Buildroot] [PATCH v3 2/2] ccid: new package

[Waldemar.Rymarkiewicz at tieto.com]

Hi,

>  When installing, ccid says:
>copy the src/92_pcscd_ccid.rules file in udev directory (/etc/udev/rules.d/)
>Perhaps it would be a good idea to do that?  (If BR2_PACKAGE_UDEV, of course)

or maybe "if BR2_ROTFS_DEVICE_CREATION_DYNAMIC_UDEV" would be better ?
Should  POST_UNINSTALL_TARGET_HOOKS be added as well ?

>  Also, ccid has the same --enable-embedded option as pcsc-lite.  I'm not
>sure if it is functional, though.

I doubt as well. Will leave it as it's.


Thanks,
/Waldek