* Integrating 2000 and Millenium drivers
@ 2000-08-14 22:38 Miguel Freitas
2000-08-15 0:37 ` Brendan J Simon
2000-08-15 7:29 ` David Woodhouse
0 siblings, 2 replies; 3+ messages in thread
From: Miguel Freitas @ 2000-08-14 22:38 UTC (permalink / raw)
To: mtd
Hi,
I'm fixing some little things on DOC2000 driver, and I'm wondering
if we could integrate 2k and Millenium drivers in a single file. There
is so many common code in these two drivers that I guess we could
do it like m-sys do: a single driver with a flag to enable all pipeline
stuff from millenium and registers. A single variable pointing to the
IO register would also be nice.
I agree that from a first version perspective it's better to develop
two drivers. We can work in all 2k or Mil little tricks without breaking
the other. But maybe we are reaching a stable low level routines
where the merge could be done.
I don't have the millenium hardware, so I would probably break
everything for millenium users if I make this merge. Is there
someone with knowledge of DoC Mil (and spare time!) to help me
with it?
BTW, I've just begin hacking DoC last month, so forgive me if I'm
missing some good reason why these drivers can't be merged...
Regards,
To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: Integrating 2000 and Millenium drivers
2000-08-14 22:38 Integrating 2000 and Millenium drivers Miguel Freitas
@ 2000-08-15 0:37 ` Brendan J Simon
2000-08-15 7:29 ` David Woodhouse
1 sibling, 0 replies; 3+ messages in thread
From: Brendan J Simon @ 2000-08-15 0:37 UTC (permalink / raw)
To: mtd
I don't know the code very well, but I would push for code reuse where ever
possibl
Maybe there should be three file ??? One that has the common/shared code,
one for Doc2000 specific code and one for DocMil specific code ???
Brendan Simon.
Miguel Freitas wrote:
> Hi,
> I'm fixing some little things on DOC2000 driver, and I'm wondering
> if we could integrate 2k and Millenium drivers in a single file. There
> is so many common code in these two drivers that I guess we could
> do it like m-sys do: a single driver with a flag to enable all pipeline
> stuff from millenium and registers. A single variable pointing to the
> IO register would also be nice.
> I agree that from a first version perspective it's better to develop
> two drivers. We can work in all 2k or Mil little tricks without breaking
> the other. But maybe we are reaching a stable low level routines
> where the merge could be done.
> I don't have the millenium hardware, so I would probably break
> everything for millenium users if I make this merge. Is there
> someone with knowledge of DoC Mil (and spare time!) to help me
> with it?
>
> BTW, I've just begin hacking DoC last month, so forgive me if I'm
> missing some good reason why these drivers can't be merged...
>
> Regards,
>
> To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org
To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Integrating 2000 and Millenium drivers
2000-08-14 22:38 Integrating 2000 and Millenium drivers Miguel Freitas
2000-08-15 0:37 ` Brendan J Simon
@ 2000-08-15 7:29 ` David Woodhouse
1 sibling, 0 replies; 3+ messages in thread
From: David Woodhouse @ 2000-08-15 7:29 UTC (permalink / raw)
To: miguel; +Cc: mtd
miguel@cetuc.puc-rio.br said:
>
> BTW, I've just begin hacking DoC last month, so forgive me if I'm
> missing some good reason why these drivers can't be merged...
There's no particularly good reason to keep them separate if we can merge
them without killing the performance. I split them up so that I could point
people at doc2001.c and say 'There. Make it work if you want to'.
Now that's done, I don't see any particular reason to keep them separate.
Code reuse is wonderful as long as it doesn't mean shitloads of conditional
branches in the inner loop of the access functions. I think we can merge
them without doing that, so why not?
--
dwmw2
To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2000-08-15 7:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2000-08-14 22:38 Integrating 2000 and Millenium drivers Miguel Freitas
2000-08-15 0:37 ` Brendan J Simon
2000-08-15 7:29 ` David Woodhouse
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox