LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* Re: Sata-MV, Intergated Sata Device Support
       [not found] <1176673884.5109.30.camel@Desktop>
@ 2008-02-25 23:40 ` Jeff Garzik
  2008-02-26  0:07   ` Mark Lord
  0 siblings, 1 reply; 9+ messages in thread
From: Jeff Garzik @ 2008-02-25 23:40 UTC (permalink / raw)
  To: linux-ide; +Cc: jon, LKML, Mark Lord, Saeed Bishara

Jon Li wrote:
> Hello,
> 
> I am curious as to whether there are plans to add support for integrated
> sata devices.  I personally want to add support for a 60x1C0 based
> device (pci:id = 0x5182).  I think adding support should be relatively
> simple, except for a few issues outlined below.
> 
> In the original mvSata.c (ver3.4) that has 0x5182 support, the config
> space is as such:
> 
> case MV_SATA_DEVICE_ID_5182:
>         pAdapter->numberOfChannels = MV_SATA_5182_PORT_NUM;
>         pAdapter->numberOfUnits = 1;
>         pAdapter->portsPerUnit = 2;
>         pAdapter->sataAdapterGeneration = MV_SATA_GEN_IIE;
>         /*The integrated sata core chip based on 60x1 C0*/
>         pAdapter->chipIs60X1C0 = MV_TRUE;
>         pAdapter->hostInterface = MV_HOST_IF_INTEGRATED;
>         pAdapter->mainMaskOffset = 0x20024; /*the iobaseaddress is
> 0x60000*/
>         pAdapter->mainCauseOffset = 0x20020;
>         break;
> 
> I have not yet figured out how all these values are defined in sata-mv.c
> (ver 0.8).  Specifically, where do I define "numberOfChannels" which
> should equal 2, and "numberOfUnits" which obviously equals 1?
> 
> I have a current config space (not completed) for sata-mv.c which is:
> 
> 	{  /* chip_5182 */
> 		.sht		= &mv_sht,
> 		.flags		= (MV_COMMON_FLAGS | MV_6XXX_FLAGS |
> 				   MV_FLAG_DUAL_HC),
> 		.pio_mask	= 0x1f,	/* pio0-4 */
> 		.udma_mask	= 0x7f,	/* udma0-6 */
> 		.port_ops	= &mv6_ops,
> 	},
> 
> I believe according to the new structure in sata-mv.c,
> HC_MAIN_IRQ_CAUSE_OFS should equal 0x20020 and HC_MAIN_IRQ_MASK_OFS
> should equal 0x20024 for the 0x5182 device.
> 
> My final question is how to implement the MV_HOST_IF_INTEGRATED flag?
> Is this already implemented and renamed in sata-mv.c?  Or do I need to
> also add the routines?

I just wanted to point the current sata_mv hackers to this message from 
April 2007...

	Jeff




^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-02-25 23:40 ` Sata-MV, Intergated Sata Device Support Jeff Garzik
@ 2008-02-26  0:07   ` Mark Lord
  2008-02-26  0:17     ` Jeff Garzik
  0 siblings, 1 reply; 9+ messages in thread
From: Mark Lord @ 2008-02-26  0:07 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: linux-ide, jon, LKML, Saeed Bishara

Jeff Garzik wrote:
> Jon Li wrote:
>> Hello,
>>
>> I am curious as to whether there are plans to add support for integrated
>> sata devices.  I personally want to add support for a 60x1C0 based
>> device (pci:id = 0x5182).  I think adding support should be relatively
>> simple, except for a few issues outlined below.
>>
>> In the original mvSata.c (ver3.4) that has 0x5182 support, the config
>> space is as such:
>>
>> case MV_SATA_DEVICE_ID_5182:
>>         pAdapter->numberOfChannels = MV_SATA_5182_PORT_NUM;
>>         pAdapter->numberOfUnits = 1;
>>         pAdapter->portsPerUnit = 2;
>>         pAdapter->sataAdapterGeneration = MV_SATA_GEN_IIE;
>>         /*The integrated sata core chip based on 60x1 C0*/
>>         pAdapter->chipIs60X1C0 = MV_TRUE;
>>         pAdapter->hostInterface = MV_HOST_IF_INTEGRATED;
>>         pAdapter->mainMaskOffset = 0x20024; /*the iobaseaddress is
>> 0x60000*/
>>         pAdapter->mainCauseOffset = 0x20020;
>>         break;
>>
>> I have not yet figured out how all these values are defined in sata-mv.c
>> (ver 0.8).  Specifically, where do I define "numberOfChannels" which
>> should equal 2, and "numberOfUnits" which obviously equals 1?
>>
>> I have a current config space (not completed) for sata-mv.c which is:
>>
>>     {  /* chip_5182 */
>>         .sht        = &mv_sht,
>>         .flags        = (MV_COMMON_FLAGS | MV_6XXX_FLAGS |
>>                    MV_FLAG_DUAL_HC),
>>         .pio_mask    = 0x1f,    /* pio0-4 */
>>         .udma_mask    = 0x7f,    /* udma0-6 */
>>         .port_ops    = &mv6_ops,
>>     },
...

Saeed:  isn't this what your SOC patches already implemented for us?
As near as I can tell, sata_mv now already has support for the 60x1C0.

-ml

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-02-26  0:07   ` Mark Lord
@ 2008-02-26  0:17     ` Jeff Garzik
  2008-02-26 11:39       ` saeed
  0 siblings, 1 reply; 9+ messages in thread
From: Jeff Garzik @ 2008-02-26  0:17 UTC (permalink / raw)
  To: Mark Lord; +Cc: linux-ide, jon, LKML, Saeed Bishara

Mark Lord wrote:
> Jeff Garzik wrote:
>> Jon Li wrote:
>>> Hello,
>>>
>>> I am curious as to whether there are plans to add support for integrated
>>> sata devices.  I personally want to add support for a 60x1C0 based
>>> device (pci:id = 0x5182).  I think adding support should be relatively
>>> simple, except for a few issues outlined below.
>>>
>>> In the original mvSata.c (ver3.4) that has 0x5182 support, the config
>>> space is as such:
>>>
>>> case MV_SATA_DEVICE_ID_5182:
>>>         pAdapter->numberOfChannels = MV_SATA_5182_PORT_NUM;
>>>         pAdapter->numberOfUnits = 1;
>>>         pAdapter->portsPerUnit = 2;
>>>         pAdapter->sataAdapterGeneration = MV_SATA_GEN_IIE;
>>>         /*The integrated sata core chip based on 60x1 C0*/
>>>         pAdapter->chipIs60X1C0 = MV_TRUE;
>>>         pAdapter->hostInterface = MV_HOST_IF_INTEGRATED;
>>>         pAdapter->mainMaskOffset = 0x20024; /*the iobaseaddress is
>>> 0x60000*/
>>>         pAdapter->mainCauseOffset = 0x20020;
>>>         break;
>>>
>>> I have not yet figured out how all these values are defined in sata-mv.c
>>> (ver 0.8).  Specifically, where do I define "numberOfChannels" which
>>> should equal 2, and "numberOfUnits" which obviously equals 1?
>>>
>>> I have a current config space (not completed) for sata-mv.c which is:
>>>
>>>     {  /* chip_5182 */
>>>         .sht        = &mv_sht,
>>>         .flags        = (MV_COMMON_FLAGS | MV_6XXX_FLAGS |
>>>                    MV_FLAG_DUAL_HC),
>>>         .pio_mask    = 0x1f,    /* pio0-4 */
>>>         .udma_mask    = 0x7f,    /* udma0-6 */
>>>         .port_ops    = &mv6_ops,
>>>     },
> ...
> 
> Saeed:  isn't this what your SOC patches already implemented for us?
> As near as I can tell, sata_mv now already has support for the 60x1C0.

Saeed's stuff didn't support PCI though, and Jon Li is definitely 
talking about PCI...

	Jeff





^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-02-26  0:17     ` Jeff Garzik
@ 2008-02-26 11:39       ` saeed
  2008-02-26 15:55         ` Mark Lord
  0 siblings, 1 reply; 9+ messages in thread
From: saeed @ 2008-02-26 11:39 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: Mark Lord, linux-ide, jon, LKML



On Mon, 25 Feb 2008, Jeff Garzik wrote:

> > ...
> > 
> > Saeed:  isn't this what your SOC patches already implemented for us?
> > As near as I can tell, sata_mv now already has support for the 60x1C0.
> 
> Saeed's stuff didn't support PCI though, and Jon Li is definitely talking
> about PCI...
yes, my patch added support for the SoC sata like in the 5182, and this 
is what Jon Li was concerned about. he mentioneded the 60x1C0 pci device 
just to suggest to use it's code for the SoC sata as it is very similar.

saeed

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-02-26 11:39       ` saeed
@ 2008-02-26 15:55         ` Mark Lord
  2008-03-04 18:26           ` Mark Lord
  2008-03-04 18:42           ` Byron Bradley
  0 siblings, 2 replies; 9+ messages in thread
From: Mark Lord @ 2008-02-26 15:55 UTC (permalink / raw)
  To: saeed; +Cc: Jeff Garzik, linux-ide, jon, LKML

saeed wrote:
> 
> On Mon, 25 Feb 2008, Jeff Garzik wrote:
> 
>>> ...
>>>
>>> Saeed:  isn't this what your SOC patches already implemented for us?
>>> As near as I can tell, sata_mv now already has support for the 60x1C0.
>> Saeed's stuff didn't support PCI though, and Jon Li is definitely talking
>> about PCI...
> yes, my patch added support for the SoC sata like in the 5182, and this 
> is what Jon Li was concerned about. he mentioneded the 60x1C0 pci device 
> just to suggest to use it's code for the SoC sata as it is very similar.
..

I don't think I understand your english there.

Does the current sata_mv driver work as-is with the chipset this person wants?
If not, then exactly what has to change to make it work?

Thanks

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-02-26 15:55         ` Mark Lord
@ 2008-03-04 18:26           ` Mark Lord
  2008-03-04 18:42           ` Byron Bradley
  1 sibling, 0 replies; 9+ messages in thread
From: Mark Lord @ 2008-03-04 18:26 UTC (permalink / raw)
  To: saeed; +Cc: Jeff Garzik, linux-ide, jon, LKML

Mark Lord wrote:
> saeed wrote:
>>
>> On Mon, 25 Feb 2008, Jeff Garzik wrote:
>>
>>>> ...
>>>>
>>>> Saeed:  isn't this what your SOC patches already implemented for us?
>>>> As near as I can tell, sata_mv now already has support for the 60x1C0.
>>> Saeed's stuff didn't support PCI though, and Jon Li is definitely 
>>> talking
>>> about PCI...
>> yes, my patch added support for the SoC sata like in the 5182, and 
>> this is what Jon Li was concerned about. he mentioneded the 60x1C0 pci 
>> device just to suggest to use it's code for the SoC sata as it is very 
>> similar.
> ..
> 
> I don't think I understand your english there.
> 
> Does the current sata_mv driver work as-is with the chipset this person wants?
> If not, then exactly what has to change to make it work?
..

Saeed ??

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-02-26 15:55         ` Mark Lord
  2008-03-04 18:26           ` Mark Lord
@ 2008-03-04 18:42           ` Byron Bradley
  2008-03-04 21:18             ` Mark Lord
  1 sibling, 1 reply; 9+ messages in thread
From: Byron Bradley @ 2008-03-04 18:42 UTC (permalink / raw)
  To: Mark Lord; +Cc: saeed, Jeff Garzik, linux-ide, jon, LKML

On Tue, Feb 26, 2008 at 3:55 PM, Mark Lord <liml@rtr.ca> wrote:
> saeed wrote:
>  >
>  > On Mon, 25 Feb 2008, Jeff Garzik wrote:
>  >
>  >>> ...
>  >>>
>  >>> Saeed:  isn't this what your SOC patches already implemented for us?
>  >>> As near as I can tell, sata_mv now already has support for the 60x1C0.
>  >> Saeed's stuff didn't support PCI though, and Jon Li is definitely talking
>  >> about PCI...
>  > yes, my patch added support for the SoC sata like in the 5182, and this
>  > is what Jon Li was concerned about. he mentioneded the 60x1C0 pci device
>  > just to suggest to use it's code for the SoC sata as it is very similar.
>  ..
>
>  I don't think I understand your english there.
>
>  Does the current sata_mv driver work as-is with the chipset this person wants?
>  If not, then exactly what has to change to make it work?

Jon Li was one of the people who started porting Linux to the Marvell
Orion 88F5182 before Marvell did it. sata_mv does now work on the
device Jon was talking about. When this was sent I think we were
unsure whether it could work over PCI or just the system bus which
probably confused things a little.

Cheers,

-- 
Byron Bradley

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-03-04 18:42           ` Byron Bradley
@ 2008-03-04 21:18             ` Mark Lord
  2008-03-05 13:07               ` Jeff Garzik
  0 siblings, 1 reply; 9+ messages in thread
From: Mark Lord @ 2008-03-04 21:18 UTC (permalink / raw)
  To: Byron Bradley; +Cc: saeed, Jeff Garzik, linux-ide, jon, LKML

Byron Bradley wrote:
> On Tue, Feb 26, 2008 at 3:55 PM, Mark Lord <liml@rtr.ca> wrote:
>> saeed wrote:
>>  >
>>  > On Mon, 25 Feb 2008, Jeff Garzik wrote:
>>  >
>>  >>> ...
>>  >>>
>>  >>> Saeed:  isn't this what your SOC patches already implemented for us?
>>  >>> As near as I can tell, sata_mv now already has support for the 60x1C0.
>>  >> Saeed's stuff didn't support PCI though, and Jon Li is definitely talking
>>  >> about PCI...
>>  > yes, my patch added support for the SoC sata like in the 5182, and this
>>  > is what Jon Li was concerned about. he mentioneded the 60x1C0 pci device
>>  > just to suggest to use it's code for the SoC sata as it is very similar.
>>  ..
>>
>>  I don't think I understand your english there.
>>
>>  Does the current sata_mv driver work as-is with the chipset this person wants?
>>  If not, then exactly what has to change to make it work?
> 
> Jon Li was one of the people who started porting Linux to the Marvell
> Orion 88F5182 before Marvell did it. sata_mv does now work on the
> device Jon was talking about. When this was sent I think we were
> unsure whether it could work over PCI or just the system bus which
> probably confused things a little.
..

Thanks for pointing this out.

So, no further action required -- it already works now.

Got that, Jeff?

Cheers


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Sata-MV, Intergated Sata Device Support
  2008-03-04 21:18             ` Mark Lord
@ 2008-03-05 13:07               ` Jeff Garzik
  0 siblings, 0 replies; 9+ messages in thread
From: Jeff Garzik @ 2008-03-05 13:07 UTC (permalink / raw)
  To: Mark Lord; +Cc: Byron Bradley, saeed, linux-ide, jon, LKML

Mark Lord wrote:
> Byron Bradley wrote:
>> On Tue, Feb 26, 2008 at 3:55 PM, Mark Lord <liml@rtr.ca> wrote:
>>> saeed wrote:
>>>  >
>>>  > On Mon, 25 Feb 2008, Jeff Garzik wrote:
>>>  >
>>>  >>> ...
>>>  >>>
>>>  >>> Saeed:  isn't this what your SOC patches already implemented for 
>>> us?
>>>  >>> As near as I can tell, sata_mv now already has support for the 
>>> 60x1C0.
>>>  >> Saeed's stuff didn't support PCI though, and Jon Li is definitely 
>>> talking
>>>  >> about PCI...
>>>  > yes, my patch added support for the SoC sata like in the 5182, and 
>>> this
>>>  > is what Jon Li was concerned about. he mentioneded the 60x1C0 pci 
>>> device
>>>  > just to suggest to use it's code for the SoC sata as it is very 
>>> similar.
>>>  ..
>>>
>>>  I don't think I understand your english there.
>>>
>>>  Does the current sata_mv driver work as-is with the chipset this 
>>> person wants?
>>>  If not, then exactly what has to change to make it work?
>>
>> Jon Li was one of the people who started porting Linux to the Marvell
>> Orion 88F5182 before Marvell did it. sata_mv does now work on the
>> device Jon was talking about. When this was sent I think we were
>> unsure whether it could work over PCI or just the system bus which
>> probably confused things a little.
> ..
> 
> Thanks for pointing this out.
> 
> So, no further action required -- it already works now.
> 
> Got that, Jeff?

hehe, thanks :)



^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2008-03-05 13:07 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1176673884.5109.30.camel@Desktop>
2008-02-25 23:40 ` Sata-MV, Intergated Sata Device Support Jeff Garzik
2008-02-26  0:07   ` Mark Lord
2008-02-26  0:17     ` Jeff Garzik
2008-02-26 11:39       ` saeed
2008-02-26 15:55         ` Mark Lord
2008-03-04 18:26           ` Mark Lord
2008-03-04 18:42           ` Byron Bradley
2008-03-04 21:18             ` Mark Lord
2008-03-05 13:07               ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).