On 27/06/13 14:48, Nick Dyer wrote:
> The existing implementation which encodes the configuration as a binary blob
> in platform data is unsatisfactory since it requires a kernel recompile for
> the configuration to be changed, and it doesn't deal well with firmware
> changes that move values around on the chip.
>
> Atmel define an ASCII format for the configuration which can be exported from
> their tools. This patch implements a parser for that format which loads the
> configuration via the firmware loader and sends it to the MXT chip.
I am using the mxt-app tool (v1.13) from
git://github.com/atmel-maxtouch/obp-utils.git
However when an existing config is dumped using that tool's --save
option the
config CRC is always zero.
That means that with this patch the config is loaded every time unless the
dumped file is manually tweaked to set the correct CRC.
>
> ---
> - dev_dbg(dev, "No cfg data defined, skipping reg init\n");
> + ret = request_firmware(&cfg, MXT_CFG_NAME, dev);
>
When building the driver into the kernel this just hangs for 60 seconds
then fails.
Wouldn't it be better to use request_firmware_nowait() ?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
> The existing implementation which encodes the configuration as a binary blob
> in platform data is unsatisfactory since it requires a kernel recompile for
> the configuration to be changed, and it doesn't deal well with firmware
> changes that move values around on the chip.
>
> Atmel define an ASCII format for the configuration which can be exported from
> their tools. This patch implements a parser for that format which loads the
> configuration via the firmware loader and sends it to the MXT chip.
I am using the mxt-app tool (v1.13) from
git://github.com/atmel-maxtouch/obp-utils.git
However when an existing config is dumped using that tool's --save
option the
config CRC is always zero.
That means that with this patch the config is loaded every time unless the
dumped file is manually tweaked to set the correct CRC.
>
> ---
> - dev_dbg(dev, "No cfg data defined, skipping reg init\n");
> + ret = request_firmware(&cfg, MXT_CFG_NAME, dev);
>
When building the driver into the kernel this just hangs for 60 seconds
then fails.
Wouldn't it be better to use request_firmware_nowait() ?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/