* Re: mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE
[not found] <E1CdAeF-0007RX-Fq@phoenix.infradead.org>
@ 2004-12-11 18:05 ` David Woodhouse
2004-12-11 18:21 ` Artem B. Bityuckiy
0 siblings, 1 reply; 3+ messages in thread
From: David Woodhouse @ 2004-12-11 18:05 UTC (permalink / raw)
To: Artem Bityuckiy; +Cc: linux-mtd
On Sat, 2004-12-11 at 17:03 +0000, Artem Bityuckiy wrote:
> Remove headers frome here. Instead join them and use one <linux/jffs3.h>
> header.
No. Keep it in the fs/jffs3 directory not in include/linux/
--
dwmw2
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE
2004-12-11 18:05 ` mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE David Woodhouse
@ 2004-12-11 18:21 ` Artem B. Bityuckiy
2004-12-11 21:31 ` David Woodhouse
0 siblings, 1 reply; 3+ messages in thread
From: Artem B. Bityuckiy @ 2004-12-11 18:21 UTC (permalink / raw)
To: David Woodhouse; +Cc: MTD List
On Sat, 11 Dec 2004, David Woodhouse wrote:
> On Sat, 2004-12-11 at 17:03 +0000, Artem Bityuckiy wrote:
> > Remove headers frome here. Instead join them and use one <linux/jffs3.h>
> > header.
>
> No. Keep it in the fs/jffs3 directory not in include/linux/
>
> --
> dwmw2
>
Ok, but just interesting why?
:-)
You may not commit it to BK. patchin.sh doesn't install it unles it was
explicitly asked...
--
Best Regards,
Artem B. Bityuckiy,
St.-Petersburg, Russia.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE
2004-12-11 18:21 ` Artem B. Bityuckiy
@ 2004-12-11 21:31 ` David Woodhouse
0 siblings, 0 replies; 3+ messages in thread
From: David Woodhouse @ 2004-12-11 21:31 UTC (permalink / raw)
To: Artem B. Bityuckiy; +Cc: MTD List
On Sat, 11 Dec 2004, Artem B. Bityuckiy wrote:
> > No. Keep it in the fs/jffs3 directory not in include/linux/
> >
> Ok, but just interesting why?
It's private to the file system. It doesn't need to live in include/linux
where other people can see it. The only reason we _used_ to put them
there is because of the inode and superblock unions.
There _was_ a reason I moved it out of include/linux/ in the first place.
--
dwmw2
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-12-11 21:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <E1CdAeF-0007RX-Fq@phoenix.infradead.org>
2004-12-11 18:05 ` mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE David Woodhouse
2004-12-11 18:21 ` Artem B. Bityuckiy
2004-12-11 21:31 ` David Woodhouse
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox