Sergei Shtylyov wrote:
>> It doesn't buy us anything in here, but it's conceivable that someone
>> may want to write a driver that uses a shift in the I/O accessor
>> rather than an array of port offsets,
>
>
>It wouldn't be IDE driver then, and neither it would be libata which
> also does
Scott Wood wrote:
>>> It doesn't buy us anything in here, but it's conceivable that someone
>>> may want to write a driver that uses a shift in the I/O accessor
>>> rather than an array of port offsets,
>>It wouldn't be IDE driver then, and neither it would be libata
>> which also does thi
> This doesn't mean that shift is better anyway. If everyone
> considers it
> better, I give up. But be warned that shift (stride) is not the only
> property
> characterizing register accesses -- the regs might be only accessible
> as
> 16/32-bit quantities, for example (16-bit is a real wor
Hello.
Segher Boessenkool wrote:
>> This doesn't mean that shift is better anyway. If everyone
>> considers it
>> better, I give up. But be warned that shift (stride) is not the only
>> property
>> characterizing register accesses -- the regs might be only accessible as
>> 16/32-bit quantit
>> More importantly, "reg-shift" doesn't say what part of
>> the bigger words to access. A common example is byte-wide
>> registers on a 32-bit-only bus; it's about 50%-50% between
>> connecting the registers to the low byte vs. connecting it
>> to the byte with the lowest address.
>
>We alrea
This is now very similar to pata_platform.c, they both use
same platform data structure and same resources.
To achieve that, byte_lanes_swapping platform data variable
and platform specified iops removed from that driver. It's fine,
since those were never used anyway.
pata_platform and ide_platf
> pata_platform and ide_platform are carrying same driver names,
> to easily switch between these drivers, without need to touch
> platform code.
>
> Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]>
> Signed-off-by: Vitaly Bordug <[EMAIL PROTECTED]>
Acked-by: Alan Cox <[EMAIL PROTECTED]>
_
On Jul 25, 2007, at 11:53 AM, Vitaly Bordug wrote:
>
> This is now very similar to pata_platform.c, they both use
> same platform data structure and same resources.
>
> To achieve that, byte_lanes_swapping platform data variable
> and platform specified iops removed from that driver. It's fine,
>
On Thu, 26 Jul 2007, Sergei Shtylyov wrote:
> Guennadi Liakhovetski wrote:
>
> > > Wrong list to submit sych stuff, post to linux-ide.
>
> > Not entirely. The patch (or other patches in the series) would also touch
> > ARM platforms in the mainline, currently using that driver. As I didn't
>
> driver to using platform-device. I got a reply, that it's not worth it now
> that IDE is slowly becoming obsolete, and the pata_platform serves the
> perpose perfectly well. I found this argument reasonable, I had the same
> doubt, just wanted to double-check. So, why do we now need a new lega
Guennadi Liakhovetski wrote:
>>This is now very similar to pata_platform.c, they both use
>>same platform data structure and same resources.
>>To achieve that, byte_lanes_swapping platform data variable
>>and platform specified iops removed from that driver. It's fine,
>>since those were never us
Hello.
Guennadi Liakhovetski wrote:
>>>driver to using platform-device. I got a reply, that it's not worth it now
>>>that IDE is slowly becoming obsolete, and the pata_platform serves the
>>>perpose perfectly well. I found this argument reasonable, I had the same
>>>doubt, just wanted to doubl
On Thu, 26 Jul 2007, Sergei Shtylyov wrote:
> Guennadi Liakhovetski wrote:
>
> > Ok, thanks for the explanation Alan. So, there's no technical argument,
> > just "being nice to the users", and add a new driver, which we know we'll
> > have to remove soon, thus having to persuade its users, who
On Wednesday 25 July 2007, Alan Cox wrote:
> > pata_platform and ide_platform are carrying same driver names,
> > to easily switch between these drivers, without need to touch
> > platform code.
> >
> > Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]>
> > Signed-off-by: Vitaly Bordug <[EMAIL PRO
On Wed, 25 Jul 2007, Alan Cox wrote:
> > driver to using platform-device. I got a reply, that it's not worth it now
> > that IDE is slowly becoming obsolete, and the pata_platform serves the
> > perpose perfectly well. I found this argument reasonable, I had the same
> > doubt, just wanted to d
Hi,
On Wednesday 25 July 2007, Vitaly Bordug wrote:
>
> This is now very similar to pata_platform.c, they both use
> same platform data structure and same resources.
>
> To achieve that, byte_lanes_swapping platform data variable
> and platform specified iops removed from that driver. It's fine
Hi,
On Thursday 26 July 2007, Guennadi Liakhovetski wrote:
> On Wed, 25 Jul 2007, Alan Cox wrote:
>
> > > driver to using platform-device. I got a reply, that it's not worth it
> > > now
> > > that IDE is slowly becoming obsolete, and the pata_platform serves the
> > > perpose perfectly well.
Guennadi Liakhovetski wrote:
This is now very similar to pata_platform.c, they both use
same platform data structure and same resources.
To achieve that, byte_lanes_swapping platform data variable
and platform specified iops removed from that driver. It's fine,
since those wer
On Wed, 25 Jul 2007, Sergei Shtylyov wrote:
> Guennadi Liakhovetski wrote:
>
> > > This is now very similar to pata_platform.c, they both use
> > > same platform data structure and same resources.
>
> > > To achieve that, byte_lanes_swapping platform data variable
> > > and platform specified io
On Wed, 25 Jul 2007, Vitaly Bordug wrote:
>
> This is now very similar to pata_platform.c, they both use
> same platform data structure and same resources.
>
> To achieve that, byte_lanes_swapping platform data variable
> and platform specified iops removed from that driver. It's fine,
> since t
Hello.
Scott Wood wrote:
>>> +hwif->hw.io_ports[IDE_DATA_OFFSET] = port;
>>> +
>>> +port += (1 << pdata->ioport_shift);
>>> +for (i = IDE_ERROR_OFFSET; i <= IDE_STATUS_OFFSET;
>>> + i++, port += (1 << pdata->ioport_shift))
>>
>>
>>
>> Looks like shift doesn't buy as anythi
Vitaly Bordug wrote:
> This is now very similar to pata_platform.c, they both use
> same platform data structure and same resources.
> To achieve that, byte_lanes_swapping platform data variable
> and platform specified iops removed from that driver. It's fine,
> since those were never used anywa
Sergei Shtylyov wrote:
>>+ hwif->hw.io_ports[IDE_DATA_OFFSET] = port;
>>+
>>+ port += (1 << pdata->ioport_shift);
>>+ for (i = IDE_ERROR_OFFSET; i <= IDE_STATUS_OFFSET;
>>+ i++, port += (1 << pdata->ioport_shift))
>
>
> Looks like shift doesn't buy as anything, why not ju
23 matches
Mail list logo