* [PATCH] mfd: Add timb-radio to the timberdale MFD
@ 2010-02-14 18:50 Richard Röjfors
2010-02-19 11:52 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 6+ messages in thread
From: Richard Röjfors @ 2010-02-14 18:50 UTC (permalink / raw)
To: Linux Media Mailing List, Mauro Carvalho Chehab; +Cc: sameo
This patch addes timb-radio to all configurations of the timberdale MFD.
Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
board info of these devices is passed via the timb-radio platform data.
Signed-off-by: Richard Röjfors <richard.rojfors@pelagicore.com>
---
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 603cf06..1ed44d2 100644
--- a/drivers/mfd/timberdale.c
+++ b/drivers/mfd/timberdale.c
@@ -37,6 +37,8 @@
#include <linux/spi/max7301.h>
#include <linux/spi/mc33880.h>
+#include <media/timb_radio.h>
+
#include "timberdale.h"
#define DRIVER_NAME "timberdale"
@@ -213,6 +215,40 @@ const static __devinitconst struct resource timberdale_uartlite_resources[] = {
},
};
+const static __devinitconst struct resource timberdale_radio_resources[] = {
+ {
+ .start = RDSOFFSET,
+ .end = RDSEND,
+ .flags = IORESOURCE_MEM,
+ },
+ {
+ .start = IRQ_TIMBERDALE_RDS,
+ .end = IRQ_TIMBERDALE_RDS,
+ .flags = IORESOURCE_IRQ,
+ },
+};
+
+static __devinitdata struct i2c_board_info timberdale_tef6868_i2c_board_info = {
+ I2C_BOARD_INFO("tef6862", 0x60)
+};
+
+static __devinitdata struct i2c_board_info timberdale_saa7706_i2c_board_info = {
+ I2C_BOARD_INFO("saa7706h", 0x1C)
+};
+
+static __devinitdata struct timb_radio_platform_data
+ timberdale_radio_platform_data = {
+ .i2c_adapter = 0,
+ .tuner = {
+ .module_name = "tef6862",
+ .info = &timberdale_tef6868_i2c_board_info
+ },
+ .dsp = {
+ .module_name = "saa7706h",
+ .info = &timberdale_saa7706_i2c_board_info
+ }
+};
+
const static __devinitconst struct resource timberdale_dma_resources[] = {
{
.start = DMAOFFSET,
@@ -240,6 +276,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg0[] = {
.data_size = sizeof(timberdale_gpio_platform_data),
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
@@ -282,6 +325,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg1[] = {
.resources = timberdale_mlogicore_resources,
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
@@ -314,6 +364,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg2[] = {
.data_size = sizeof(timberdale_gpio_platform_data),
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
@@ -348,6 +405,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg3[] = {
.data_size = sizeof(timberdale_gpio_platform_data),
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] mfd: Add timb-radio to the timberdale MFD
2010-02-14 18:50 [PATCH] mfd: Add timb-radio to the timberdale MFD Richard Röjfors
@ 2010-02-19 11:52 ` Mauro Carvalho Chehab
2010-02-19 16:39 ` Richard Röjfors
2010-02-19 22:41 ` Richard Röjfors
0 siblings, 2 replies; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2010-02-19 11:52 UTC (permalink / raw)
To: Richard Röjfors; +Cc: Linux Media Mailing List, sameo
Richard Röjfors wrote:
> This patch addes timb-radio to all configurations of the timberdale MFD.
>
> Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
> board info of these devices is passed via the timb-radio platform data.
Hi Richard,
I'm trying to apply it to my git tree (http://git.linuxtv.org/v4l-dvb.git),
but it is failing:
patching file drivers/mfd/timberdale.c
Hunk #1 FAILED at 37.
Hunk #2 FAILED at 215.
Hunk #3 FAILED at 276.
Hunk #4 FAILED at 325.
Hunk #5 FAILED at 364.
Hunk #6 FAILED at 405.
6 out of 6 hunks FAILED -- saving rejects to file drivers/mfd/timberdale.c.rej
Patch doesn't apply
Could you please verify what's going wrong?
>
> Signed-off-by: Richard Röjfors <richard.rojfors@pelagicore.com>
> ---
> diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
> index 603cf06..1ed44d2 100644
> --- a/drivers/mfd/timberdale.c
> +++ b/drivers/mfd/timberdale.c
> @@ -37,6 +37,8 @@
> #include <linux/spi/max7301.h>
> #include <linux/spi/mc33880.h>
>
> +#include <media/timb_radio.h>
> +
> #include "timberdale.h"
>
> #define DRIVER_NAME "timberdale"
> @@ -213,6 +215,40 @@ const static __devinitconst struct resource
> timberdale_uartlite_resources[] = {
> },
> };
>
> +const static __devinitconst struct resource
> timberdale_radio_resources[] = {
> + {
> + .start = RDSOFFSET,
> + .end = RDSEND,
> + .flags = IORESOURCE_MEM,
> + },
> + {
> + .start = IRQ_TIMBERDALE_RDS,
> + .end = IRQ_TIMBERDALE_RDS,
> + .flags = IORESOURCE_IRQ,
> + },
> +};
> +
> +static __devinitdata struct i2c_board_info
> timberdale_tef6868_i2c_board_info = {
> + I2C_BOARD_INFO("tef6862", 0x60)
> +};
> +
> +static __devinitdata struct i2c_board_info
> timberdale_saa7706_i2c_board_info = {
> + I2C_BOARD_INFO("saa7706h", 0x1C)
> +};
> +
> +static __devinitdata struct timb_radio_platform_data
> + timberdale_radio_platform_data = {
> + .i2c_adapter = 0,
> + .tuner = {
> + .module_name = "tef6862",
> + .info = &timberdale_tef6868_i2c_board_info
> + },
> + .dsp = {
> + .module_name = "saa7706h",
> + .info = &timberdale_saa7706_i2c_board_info
> + }
> +};
> +
> const static __devinitconst struct resource timberdale_dma_resources[] = {
> {
> .start = DMAOFFSET,
> @@ -240,6 +276,13 @@ static __devinitdata struct mfd_cell
> timberdale_cells_bar0_cfg0[] = {
> .data_size = sizeof(timberdale_gpio_platform_data),
> },
> {
> + .name = "timb-radio",
> + .num_resources = ARRAY_SIZE(timberdale_radio_resources),
> + .resources = timberdale_radio_resources,
> + .platform_data = &timberdale_radio_platform_data,
> + .data_size = sizeof(timberdale_radio_platform_data),
> + },
> + {
> .name = "xilinx_spi",
> .num_resources = ARRAY_SIZE(timberdale_spi_resources),
> .resources = timberdale_spi_resources,
> @@ -282,6 +325,13 @@ static __devinitdata struct mfd_cell
> timberdale_cells_bar0_cfg1[] = {
> .resources = timberdale_mlogicore_resources,
> },
> {
> + .name = "timb-radio",
> + .num_resources = ARRAY_SIZE(timberdale_radio_resources),
> + .resources = timberdale_radio_resources,
> + .platform_data = &timberdale_radio_platform_data,
> + .data_size = sizeof(timberdale_radio_platform_data),
> + },
> + {
> .name = "xilinx_spi",
> .num_resources = ARRAY_SIZE(timberdale_spi_resources),
> .resources = timberdale_spi_resources,
> @@ -314,6 +364,13 @@ static __devinitdata struct mfd_cell
> timberdale_cells_bar0_cfg2[] = {
> .data_size = sizeof(timberdale_gpio_platform_data),
> },
> {
> + .name = "timb-radio",
> + .num_resources = ARRAY_SIZE(timberdale_radio_resources),
> + .resources = timberdale_radio_resources,
> + .platform_data = &timberdale_radio_platform_data,
> + .data_size = sizeof(timberdale_radio_platform_data),
> + },
> + {
> .name = "xilinx_spi",
> .num_resources = ARRAY_SIZE(timberdale_spi_resources),
> .resources = timberdale_spi_resources,
> @@ -348,6 +405,13 @@ static __devinitdata struct mfd_cell
> timberdale_cells_bar0_cfg3[] = {
> .data_size = sizeof(timberdale_gpio_platform_data),
> },
> {
> + .name = "timb-radio",
> + .num_resources = ARRAY_SIZE(timberdale_radio_resources),
> + .resources = timberdale_radio_resources,
> + .platform_data = &timberdale_radio_platform_data,
> + .data_size = sizeof(timberdale_radio_platform_data),
> + },
> + {
> .name = "xilinx_spi",
> .num_resources = ARRAY_SIZE(timberdale_spi_resources),
> .resources = timberdale_spi_resources,
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Cheers,
Mauro
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mfd: Add timb-radio to the timberdale MFD
2010-02-19 11:52 ` Mauro Carvalho Chehab
@ 2010-02-19 16:39 ` Richard Röjfors
2010-02-19 22:41 ` Richard Röjfors
1 sibling, 0 replies; 6+ messages in thread
From: Richard Röjfors @ 2010-02-19 16:39 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Linux Media Mailing List, sameo
On 02/19/2010 04:52 AM, Mauro Carvalho Chehab wrote:
> Richard Röjfors wrote:
>> This patch addes timb-radio to all configurations of the timberdale MFD.
>>
>> Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
>> board info of these devices is passed via the timb-radio platform data.
>
> Hi Richard,
>
> I'm trying to apply it to my git tree (http://git.linuxtv.org/v4l-dvb.git),
> but it is failing:
Ah, this patch was against your linux-next.git at kernel.org.
I will generate a new patch against the proper git.
--Richard
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mfd: Add timb-radio to the timberdale MFD
2010-02-19 11:52 ` Mauro Carvalho Chehab
2010-02-19 16:39 ` Richard Röjfors
@ 2010-02-19 22:41 ` Richard Röjfors
2010-02-19 22:56 ` Randy Dunlap
1 sibling, 1 reply; 6+ messages in thread
From: Richard Röjfors @ 2010-02-19 22:41 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Linux Media Mailing List, sameo
[-- Attachment #1: Type: text/plain, Size: 650 bytes --]
On 02/19/2010 04:52 AM, Mauro Carvalho Chehab wrote:
> Richard Röjfors wrote:
>> This patch addes timb-radio to all configurations of the timberdale MFD.
>>
>> Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
>> board info of these devices is passed via the timb-radio platform data.
>
> Hi Richard,
>
> I'm trying to apply it to my git tree (http://git.linuxtv.org/v4l-dvb.git),
> but it is failing:
Hi Mauro,
Right now my mail client, icedove, confuses me. Just upgraded to ver 3.
It seem to add in an extra space to lines not starting with a plus in the patch.
I attached the patch.
Sorry for the inconvenience.
--Richard
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: timberdale-add-timb-radio.patch --]
[-- Type: text/x-patch; name="timberdale-add-timb-radio.patch", Size: 3744 bytes --]
This patch addes timb-radio to all configurations of the timberdale MFD.
Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
board info of these devices is passed via the timb-radio platform data.
Signed-off-by: Richard Röjfors <richard.rojfors@pelagicore.com>
---
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 603cf06..1ed44d2 100644
--- a/drivers/mfd/timberdale.c
+++ b/drivers/mfd/timberdale.c
@@ -37,6 +37,8 @@
#include <linux/spi/max7301.h>
#include <linux/spi/mc33880.h>
+#include <media/timb_radio.h>
+
#include "timberdale.h"
#define DRIVER_NAME "timberdale"
@@ -213,6 +215,40 @@ const static __devinitconst struct resource timberdale_uartlite_resources[] = {
},
};
+const static __devinitconst struct resource timberdale_radio_resources[] = {
+ {
+ .start = RDSOFFSET,
+ .end = RDSEND,
+ .flags = IORESOURCE_MEM,
+ },
+ {
+ .start = IRQ_TIMBERDALE_RDS,
+ .end = IRQ_TIMBERDALE_RDS,
+ .flags = IORESOURCE_IRQ,
+ },
+};
+
+static __devinitdata struct i2c_board_info timberdale_tef6868_i2c_board_info = {
+ I2C_BOARD_INFO("tef6862", 0x60)
+};
+
+static __devinitdata struct i2c_board_info timberdale_saa7706_i2c_board_info = {
+ I2C_BOARD_INFO("saa7706h", 0x1C)
+};
+
+static __devinitdata struct timb_radio_platform_data
+ timberdale_radio_platform_data = {
+ .i2c_adapter = 0,
+ .tuner = {
+ .module_name = "tef6862",
+ .info = &timberdale_tef6868_i2c_board_info
+ },
+ .dsp = {
+ .module_name = "saa7706h",
+ .info = &timberdale_saa7706_i2c_board_info
+ }
+};
+
const static __devinitconst struct resource timberdale_dma_resources[] = {
{
.start = DMAOFFSET,
@@ -240,6 +276,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg0[] = {
.data_size = sizeof(timberdale_gpio_platform_data),
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
@@ -282,6 +325,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg1[] = {
.resources = timberdale_mlogicore_resources,
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
@@ -314,6 +364,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg2[] = {
.data_size = sizeof(timberdale_gpio_platform_data),
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
@@ -348,6 +405,13 @@ static __devinitdata struct mfd_cell timberdale_cells_bar0_cfg3[] = {
.data_size = sizeof(timberdale_gpio_platform_data),
},
{
+ .name = "timb-radio",
+ .num_resources = ARRAY_SIZE(timberdale_radio_resources),
+ .resources = timberdale_radio_resources,
+ .platform_data = &timberdale_radio_platform_data,
+ .data_size = sizeof(timberdale_radio_platform_data),
+ },
+ {
.name = "xilinx_spi",
.num_resources = ARRAY_SIZE(timberdale_spi_resources),
.resources = timberdale_spi_resources,
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] mfd: Add timb-radio to the timberdale MFD
2010-02-19 22:41 ` Richard Röjfors
@ 2010-02-19 22:56 ` Randy Dunlap
2010-02-20 0:10 ` Richard Röjfors
0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2010-02-19 22:56 UTC (permalink / raw)
To: Richard Röjfors
Cc: Mauro Carvalho Chehab, Linux Media Mailing List, sameo
On 02/19/10 14:41, Richard Röjfors wrote:
> On 02/19/2010 04:52 AM, Mauro Carvalho Chehab wrote:
>> Richard Röjfors wrote:
>>> This patch addes timb-radio to all configurations of the timberdale MFD.
>>>
>>> Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
>>> board info of these devices is passed via the timb-radio platform data.
>>
>> Hi Richard,
>>
>> I'm trying to apply it to my git tree
>> (http://git.linuxtv.org/v4l-dvb.git),
>> but it is failing:
>
> Hi Mauro,
>
> Right now my mail client, icedove, confuses me. Just upgraded to ver 3.
> It seem to add in an extra space to lines not starting with a plus in
> the patch.
I had that problem with something called Thunderbird. Perhaps you could
use the hints in Documentation/email-clients.txt but change Thunderbird
to icedove. (?)
> I attached the patch.
>
> Sorry for the inconvenience.
>
> --Richard
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mfd: Add timb-radio to the timberdale MFD
2010-02-19 22:56 ` Randy Dunlap
@ 2010-02-20 0:10 ` Richard Röjfors
0 siblings, 0 replies; 6+ messages in thread
From: Richard Röjfors @ 2010-02-20 0:10 UTC (permalink / raw)
To: Randy Dunlap; +Cc: Mauro Carvalho Chehab, Linux Media Mailing List, sameo
On 02/19/2010 03:56 PM, Randy Dunlap wrote:
> On 02/19/10 14:41, Richard Röjfors wrote:
>> On 02/19/2010 04:52 AM, Mauro Carvalho Chehab wrote:
>>> Richard Röjfors wrote:
>>>> This patch addes timb-radio to all configurations of the timberdale MFD.
>>>>
>>>> Connected to the FPGA is a TEF6862 tuner and a SAA7706H DSP, the I2C
>>>> board info of these devices is passed via the timb-radio platform data.
>>>
>>> Hi Richard,
>>>
>>> I'm trying to apply it to my git tree
>>> (http://git.linuxtv.org/v4l-dvb.git),
>>> but it is failing:
>>
>> Hi Mauro,
>>
>> Right now my mail client, icedove, confuses me. Just upgraded to ver 3.
>> It seem to add in an extra space to lines not starting with a plus in
>> the patch.
>
> I had that problem with something called Thunderbird. Perhaps you could
> use the hints in Documentation/email-clients.txt but change Thunderbird
> to icedove. (?)
icedove is debians fork of thunderbird, so I think it's basically the
same client. So if people using it isn't aware of it I think we should
change it to Thunderbird/Icedove in the documentation
--Richard
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-02-20 0:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-02-14 18:50 [PATCH] mfd: Add timb-radio to the timberdale MFD Richard Röjfors
2010-02-19 11:52 ` Mauro Carvalho Chehab
2010-02-19 16:39 ` Richard Röjfors
2010-02-19 22:41 ` Richard Röjfors
2010-02-19 22:56 ` Randy Dunlap
2010-02-20 0:10 ` Richard Röjfors
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox